[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