Opened 4 years ago

Closed 4 years ago

#6381 defect closed fixed (fixed)

There are some duplicate implementations of (Clock, MemoryReactor) combiner

Reported by: exarkun Owned by: exarkun
Priority: normal Milestone:
Component: core Keywords:
Cc: Branch: branches/clockmemoryreactor-6381
branch-diff, diff-cov, branch-cov, buildbot
Author: exarkun


We have a bunch of subclasses of Clock and MemoryReactor in various places. We should remove the duplication.

Change History (5)

comment:1 Changed 4 years ago by exarkun

  • Author set to exarkun
  • Branch set to branches/clockmemoryreactor-6381

(In [37659]) Branching to 'clockmemoryreactor-6381'

comment:2 Changed 4 years ago by exarkun

  • Keywords review added
  • Owner exarkun deleted

Pretty simple. Here are the build results. pyflakes results seem a bit nonsensical, not sure what's going on there.

comment:3 Changed 4 years ago by exarkun

And by pyflakes I meant twistedchecker, I guess.

comment:4 Changed 4 years ago by therve

  • Keywords review removed
  • Owner set to exarkun

Wonderful, please merge.

comment:5 Changed 4 years ago by exarkun

  • Resolution set to fixed
  • Status changed from new to closed

(In [37671]) Merge clockmemoryreactor-6381

Author: exarkun Reviewer: therve Fixes: #6381

Eliminate the various duplicate versions of a Clock / MemoryReactor subclass used in a number of different tests.

Note: See TracTickets for help on using tickets.