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

Changes by Éric Araujo : -- nosy: +michael.foord resolution: accepted -> fixed stage: patch review -> committed/rejected ___ Python tracker ___ ___

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

Tarek Ziadé added the comment: Thanks for looking into this ! -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

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

Martin v. Löwis added the comment: Thanks for the patch. Committed (with modifications) as r83078. -- resolution: -> accepted status: open -> closed ___ Python tracker ___ _

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

Tarek Ziadé added the comment: reopening again sorry (roundup bug) -- resolution: wont fix -> status: closed -> open ___ Python tracker ___ _

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

Changes by Tarek Ziadé : -- nosy: -gvanrossum resolution: -> wont fix status: open -> closed ___ Python tracker ___ ___ Python-bugs-l

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

Changes by Tarek Ziadé : -- resolution: wont fix -> status: closed -> open ___ Python tracker ___ ___ Python-bugs-list mailing list Un

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

Tarek Ziadé added the comment: Yes, sorry. I should have added the link when I opened the bug. Thanks for reopening it -- resolution: -> wont fix status: open -> closed ___ Python tracker

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

Martin v. Löwis added the comment: ok, so reopening. Linking to the mailing list would have helped. -- resolution: wont fix -> status: closed -> open ___ Python tracker ___

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

Changes by Éric Araujo : -- nosy: +merwok ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org

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

Tarek Ziadé added the comment: what do you mean by 'too trivial' ? I don't understand why this is now suddenly rejected. Raymond, Guido, and other people have +1 this on python-ideas. http://mail.python.org/pipermail/python-ideas/2009-May/004871.html People have worked on a patch, so I think

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

Martin v. Löwis added the comment: At Europython, we agreed that this functionality is too trivial to add to the standard library; for all practical purposes, os.listdir(".") will do the right thing. So rejecting this as won't fix. -- nosy: +loewis resolution: -> wont fix status: op

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

Virgil Dupras added the comment: Here's another one. I hadn't realized that it was useless to target the 2.x codebase. So I re-worked this on py3k. The change is non-trivial, since the non-windows/non-os2 part of the code has significantly changed in 3k. This time, since "oname" is released l

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

Changes by Giampaolo Rodola' : -- nosy: +giampaolo.rodola ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http:/

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

Virgil Dupras added the comment: Since I last submitted this patch, my leet C skills have improved. I'm submitting another patch, without the needless PyUnicode creation this time. (Moreover, I think the previous patch was wrong to insert code before variable declaration of the block) --

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

Changes by Ezio Melotti : -- nosy: +ezio.melotti versions: -Python 2.7 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubs

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

Changes by Ezio Melotti : -- priority: -> normal stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsu

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

Virgil Dupras added the comment: 1. Yeah, I know. At first, that's what I wanted to do, but it resulted in a lot of code duplication (alloc, memerror, copy), which I didn't much like. But then again, what I ended up writing (because I realized I had to decref the new "po") uses up more lines

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

Amaury Forgeot d'Arc 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 in

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

Virgil Dupras added the comment: I saw this ticket as a good way to get my feet wet (I almost never touched C) with Python's C API, so I went ahead and did the part for OS X. I also did the Windows part, but I'm not setup to compile Python on Windows, so I don't even know if it remotely works

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

Changes by Raymond Hettinger : -- versions: +Python 3.2 -Python 3.1 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscrib

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

Tarek Ziadé added the comment: After a deeper look; this change looks rather complex for a posixpath.c newbie like me. I probably won't be able to provide it for 3.1. If you want to do it for 3.1, please go ahead Raymond ! -- ___ Python tracker

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

Raymond Hettinger added the comment: If this is to go into Py3.1, it needs to happen quickly. -- nosy: +rhettinger ___ Python tracker ___

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

New submission from Tarek Ziadé : make os.listdir() path argument be os.curdir. I am working on a patch, I'll submit here -- components: Library (Lib) messages: 88256 nosy: tarek severity: normal status: open title: os.curdir as the default argument for os.listdir type: feature request