[Python-Dev] PEP 471 "scandir" accepted

Ben Hoyt benhoyt at gmail.com
Tue Jul 22 23:07:37 CEST 2014


Makes sense, thanks. -Ben

On Tue, Jul 22, 2014 at 4:57 PM, Nick Coghlan <ncoghlan at gmail.com> wrote:
>
> On 23 Jul 2014 02:18, "Victor Stinner" <victor.stinner at gmail.com> wrote:
>>
>> 2014-07-22 17:52 GMT+02:00 Ben Hoyt <benhoyt at gmail.com>:
>> > However, given that we have to support this for listdir() anyway, I
>> > think it's worth reconsidering whether scandir()'s directory argument
>> > can be an integer FD. Given that listdir() already supports it, it
>> > will almost certainly be asked for later anyway for someone who's
>> > porting some listdir code that uses an FD. Thoughts, Victor?
>>
>> Please focus on what was accepted in the PEP. We should first test
>> os.scandir(). In a few months, with better feedbacks, we can consider
>> extending os.scandir() to support a file descriptor. There are
>> different issues which should be discussed and decided to implement it
>> (ex: handle the lifetime of the directory file descriptor).
>
> As Victor suggests, getting the core version working and incorporated first
> is a good way to go. Future enhancements (like accepting a file descriptor)
> and refactorings (like eliminating the code duplication with listdir) don't
> need to (and hence shouldn't) go into the initial patch.
>
> Cheers,
> Nick.
>
>>
>> Victor
>
>
>> _______________________________________________
>> Python-Dev mailing list
>> Python-Dev at python.org
>> https://mail.python.org/mailman/listinfo/python-dev
>> Unsubscribe:
>> https://mail.python.org/mailman/options/python-dev/ncoghlan%40gmail.com


More information about the Python-Dev mailing list