Eric Smith <e...@trueblade.com> added the comment:

Indeed, the code that introduced this bug was in r68489, so it was
written after Guido went through the module and cleaned it up to use
try/finally. And since r68489 was a merge of r68487, the change was
originally made on the 2.x version (which doesn't use try/finally) to
the 3.x version (which does).

Which makes me think that we should go back and add try/finally to all
the popen calls in the 2.x version, so that it serves as a better
example when/if new popen calls are added. I'm willing to do that once
this issue is resolved.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue6882>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to