[issue6095] os.curdir as the default argument for os.listdir

Amaury Forgeot d'Arc report at bugs.python.org
Mon Jun 15 01:12:39 CEST 2009


Amaury Forgeot d'Arc <amauryfa at gmail.com> added the comment:

Some comments about the patch:

- It seems wasteful to allocate a new PyUnicode object for the "."; the 
posix implements does it right: a simple wcscopy should be enough (and 
reduces the chances of refcount mistakes)

- the last block is not correctly indented; this code uses tabs.

- no need to test for hasattr(posix, 'listdir'): all supported platforms 
have directories and the posix module always exposes this function.

----------
nosy: +amaury.forgeotdarc

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue6095>
_______________________________________


More information about the Python-bugs-list mailing list