[Twisted-Python] Code review comment for http.py

Scott, Barry barry.scott at forcepoint.com
Mon Oct 7 04:36:15 MDT 2019


On Saturday, 5 October 2019 04:48:03 BST Glyph wrote:
> > On Oct 4, 2019, at 4:26 AM, Scott, Barry <barry.scott at forcepoint.com>
> > wrote:
> > 
> > I see that there are duplicate lines in http.py in
> > _GenericHTTPChannelProtocol.dataReceived()
> > 
> > Notice that around line 2960 two calls to setTimeout().
> > 
> >                networkProducer.unregisterProducer()
> >                
> >                # Cancel the old channel's timeout.
> >                self._channel.setTimeout(None)
> >                
> >                # Cancel the old channel's timeout.
> >                self._channel.setTimeout(None)
> >                
> >                transport = self._channel.transport
> > 
> > Barry
> 
> Mind filing a ticket for this?  Seems like a great thing to tag with "easy"
> for new contributors :).

https://twistedmatrix.com/trac/ticket/9709

Barry
> 
> -g
> _______________________________________________
> Twisted-Python mailing list
> Twisted-Python at twistedmatrix.com
> https://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python








More information about the Twisted-Python mailing list