STINNER Victor added the comment:

del-detach.patch: I don't understand the change in _pyio.py. I think that the 
change on socket.py is no more need since the change 46329eec5515 (issue 
#26590).

del-flush.patch:
* change on IOBase.__del__() change the behaviour compared to the io module, I 
don't think that it's correct. I may also break backward compatibility :-/
* change on fileio.c implements a finalizer for FileIO: io.IOBase already 
implements a finalizer which calls close(), I'm not sure that this change is 
needed?
* change on iobase.c removes io.IOBase finalizer. I don't understand this. It 
breaks ResourceWarning on io.Buffered* and io.TextIOWrapper classes, no?

Well, I guess that these two patches were written before Serhiy reimplemented 
the FileIO in pure Python, and this change outdated your two patches.

----------

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

Reply via email to