Éric Araujo added the comment:
Thank you for your effort! Note that distutils uses try:... finally:
fp.close(), so it tries to avoid the FD leak issues.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22825
Serhiy Storchaka added the comment:
So you suggest just close this issue?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22825
___
___
Éric Araujo added the comment:
Yes.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22825
___
___
Python-bugs-list mailing list
Unsubscribe:
Serhiy Storchaka added the comment:
Thank you for your response. This was a part of large patch about migration to
use of with (issue22826 and issue22831 are other parts).
--
resolution: - rejected
stage: patch review - resolved
status: open - closed
Éric Araujo added the comment:
I would not touch any internal parts of distutils, such as TextFile. It is not
a public class meant for general usage. The code is not ideal but not broken.
--
___
Python tracker rep...@bugs.python.org
New submission from Serhiy Storchaka:
Proposed patch makes distutils.text_file.TextFile support context management
and iterator protocols. It makes the use of TextFile simpler. The patch also
includes other minor modernizations.
--
components: Distutils
files: text_file.diff
keywords: