summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJust van Rossum <just@letterror.com>2003-02-25 21:42:15 (GMT)
committerJust van Rossum <just@letterror.com>2003-02-25 21:42:15 (GMT)
commit46c9784f68a8dbb4f13fa86ac780c45a4363b8a8 (patch)
treec52f1d1b5b66057eb089165f4639dc0038ac17a0
parent49833313855e1c7aec9c6654e81c416329298a49 (diff)
downloadcpython-46c9784f68a8dbb4f13fa86ac780c45a4363b8a8.zip
cpython-46c9784f68a8dbb4f13fa86ac780c45a4363b8a8.tar.gz
cpython-46c9784f68a8dbb4f13fa86ac780c45a4363b8a8.tar.bz2
Patch #683592: unicode support for os.listdir()
os.listdir() may now return unicode strings on platforms that set Py_FileSystemDefaultEncoding.
-rw-r--r--Misc/NEWS9
-rw-r--r--Modules/posixmodule.c24
2 files changed, 32 insertions, 1 deletions
diff --git a/Misc/NEWS b/Misc/NEWS
index fa7ef5e..750b2c3 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -25,7 +25,11 @@ Extension modules
Library
-------
-TBD
+- os.listdir() now returns Unicode strings on platforms that set
+ Py_FileSystemDefaultEncoding, for file names that are not representable
+ in ASCII. (This currently only affects MacOS X; on Windows versions
+ with wide file name support os.listdir() already returned Unicode
+ strings.)
Tools/Demos
-----------
@@ -61,6 +65,9 @@ TBD
Mac
---
+- os.listdir() now may return Unicode strings on MacOS X. See the general
+ news item under "Library".
+
- A new method MacOS.WMAvailable() returns true if it is safe to access
the window manager, false otherwise.
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index d203378..482bba9 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -1795,6 +1795,30 @@ posix_listdir(PyObject *self, PyObject *args)
d = NULL;
break;
}
+#ifdef Py_USING_UNICODE
+ if (Py_FileSystemDefaultEncoding != NULL) {
+ PyObject *w;
+
+ w = PyUnicode_FromEncodedObject(v,
+ Py_FileSystemDefaultEncoding,
+ "strict");
+ Py_DECREF(v);
+ v = w;
+ if (v == NULL) {
+ Py_DECREF(d);
+ d = NULL;
+ break;
+ }
+ /* attempt to convert to ASCII */
+ w = PyUnicode_AsASCIIString(v);
+ if (w != NULL) {
+ Py_DECREF(v);
+ v = w;
+ }
+ else
+ PyErr_Clear();
+ }
+#endif
if (PyList_Append(d, v) != 0) {
Py_DECREF(v);
Py_DECREF(d);