Opened 11 years ago

Last modified 6 years ago

#2092 enhancement new

Replace reactor.iterate calls with some callLater fu

Reported by: Jonathan Lange Owned by:
Priority: high Milestone:
Component: trial Keywords:
Cc: Branch:


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 11 years ago by Jonathan Lange

Milestone: Twisted-2.5

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 11 years ago by Jonathan Lange

Priority: normalhigh

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

comment:3 Changed 7 years ago by <automation>

Owner: Jonathan Lange deleted

comment:4 Changed 6 years ago by Thijs Triemstra

Summary: Remove reactor.iterate calls from do_cleanPendingReplace reactor.iterate calls with some callLater fu
Note: See TracTickets for help on using tickets.