Serhiy Storchaka added the comment:

I think so too. I just wanted someone to confirmed that it is not 
overcautiousness. For now glob1() is used only in one place in the stdlib 
(issue16620).

But there was other problem with previous patch, the same as with current 
implementation of os.walk() (issue25995). It makes glob to use a lot of file 
descriptors.

Updated patch lefts deprecated glob1() and glob2() and makes glob to consume 
all scandir iterator before starting to yield values (but the problem with fd 
leaks on non-refcounted implementations is still left, issue25994). This 
doesn't affect performance, but lefts the issue with delaying (issue22167).

----------
dependencies: +File descriptor leaks in os.scandir()
Added file: http://bugs.python.org/file41481/glob_scandir_2.patch

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

Reply via email to