<div dir="ltr">Hi,<div><br></div><div>Right now  twisted/runner/inetdtap.py has the following errors reported by pyflakes:</div><div><br></div><div><div>$ pyflakes twisted/runner/inetdtap.py</div><div>twisted/runner/inetdtap.py:58: undefined name 'rpcVersions'<br></div><div>twisted/runner/inetdtap.py:59: undefined name 'name'</div><div>twisted/runner/inetdtap.py:62: undefined name 'p'</div><div>twisted/runner/inetdtap.py:152: undefined name 'proto'</div></div><div><br></div><div>From what I can see from reading the code, RPCServer.startService is broken.</div><div><br></div><div>makeService is also broken for rpc/* services .<br></div><div><br></div><div>There are no tests for all this code.</div><div><br></div><div>Link for the code : </div><div><a href="https://github.com/twisted/twisted/blob/trunk/twisted/runner/inetdtap.py">https://github.com/twisted/twisted/blob/trunk/twisted/runner/inetdtap.py</a><br clear="all"><div><br></div><div>----------</div><div><br></div><div>I am working on getting the Twisted code clean of pyflakes errors/warnings so that contributors could run pyflakes check on their own branch, without relying on buildbot.</div><div><br></div><div>I would like to clean the inetdtap.py module of pylakes warning but since it is broken and has no tests, I have no idea what this code is expected to do and how end users are expecting to use it. </div><div><br></div><div>Does someone volunteer to fix it?</div><div><br class="">Otherwise, has anyone any objections against deprecating twisted/runner/inetdtap.py?<br></div><div>Since it is broken, can we just remove it?</div><div><br></div><div>Thanks!</div>-- <br><div class="gmail_signature">Adi Roiban</div>
</div></div>