[Python-Dev] PEP 471 (scandir): Add a new DirEntry.inode() method? (original) (raw)

Ben Hoyt benhoyt at gmail.com
Sat Feb 14 21:32:07 CET 2015

+1 we need to provide the inode (we shouldn't be throwing anything from the underlying directory entry away when possible). But...

I think the "or None" semantics are a bad idea. It'd be better for this to raise AttributeError on Windows so that someone can't write the most natural form of code assuming that inode is valid and have it appear to work on Windows when in fact it'd do the wrong thing.

+1 for inode support. I agree with the above -- it should either raise AttributeError on Windows if it's not going to be set ... or it should be more like Victor's original proposal where .inode() is a method that calls stat on Windows. I don't have strong feelings.

-Ben

More information about the Python-Dev mailing list