Opened 8 years ago

Last modified 4 years ago

#2092 enhancement new

Replace reactor.iterate calls with some callLater fu

Reported by: jml Owned by:
Priority: high Milestone:
Component: trial Keywords:
Cc: Branch:
Author: Launchpad Bug:

Description

Pending the changes in #2091, remove the calls to iterate() from do_cleanPending. This ticket does not require fixing any unclean tests. Only tests that actually break because of the changes will need to be fixed.

Change History (4)

comment:1 Changed 8 years ago by jml

  • Milestone Twisted-2.5 deleted

I think I may have been on crack when I wrote that description. I just ran the tests without the two iterate() calls and I got 196 errors.

I think this needs to wait for the next release.

It depends on #2124 being implemented, and a whole bunch of time spent ridding the world of the warnings it will generate.

comment:2 Changed 8 years ago by jml

  • Priority changed from normal to high

Oh, now I get it. I meant "Replace iterate calls with some callLater fu"

comment:3 Changed 4 years ago by <automation>

  • Owner jml deleted

comment:4 Changed 4 years ago by thijs

  • Summary changed from Remove reactor.iterate calls from do_cleanPending to Replace reactor.iterate calls with some callLater fu
Note: See TracTickets for help on using tickets.