I'm hunting down performance issue in our code and spotted this in passing. As far I can tell nothing seems to need callWithLogger.
I ran our 6k+ tests with this patch applied and everything worked. Does anything in the twisted world need it? Barry diff --git a/src/twisted/internet/pollreactor.py b/src/twisted/internet/ pollreactor.py index 6db1660b9..6901e5c95 100644 --- a/src/twisted/internet/pollreactor.py +++ b/src/twisted/internet/pollreactor.py @@ -165,7 +165,7 @@ class PollReactor(posixbase.PosixReactorBase, posixbase._PollLikeMixin): # Handles the infrequent case where one selectable's # handler disconnects another. continue - log.callWithLogger(selectable, _drdw, selectable, fd, event) + _drdw(selectable, fd, event) doIteration = doPoll _______________________________________________ Twisted-Python mailing list Twisted-Python@twistedmatrix.com https://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python