diff options
author | Steve Dower <steve.dower@microsoft.com> | 2016-09-10 00:33:37 (GMT) |
---|---|---|
committer | Steve Dower <steve.dower@microsoft.com> | 2016-09-10 00:33:37 (GMT) |
commit | ff70fc220653d8999d69bd3f3b986a07afc81b64 (patch) | |
tree | ec3a8b95c09b001e3446304a7342767475ad5eaa | |
parent | 1319236167041f01c4efd4f7cc1400d2e4a18275 (diff) | |
parent | 8dcc48ee3b6447cd63038e63ee11de36f6d85a68 (diff) | |
download | cpython-ff70fc220653d8999d69bd3f3b986a07afc81b64.zip cpython-ff70fc220653d8999d69bd3f3b986a07afc81b64.tar.gz cpython-ff70fc220653d8999d69bd3f3b986a07afc81b64.tar.bz2 |
Issue #25758: Prevents zipimport from unnecessarily encoding a filename (patch by Eryk Sun)
-rw-r--r-- | Lib/test/test_zipimport.py | 2 | ||||
-rw-r--r-- | Misc/NEWS | 3 | ||||
-rw-r--r-- | Modules/zipimport.c | 14 |
3 files changed, 8 insertions, 11 deletions
diff --git a/Lib/test/test_zipimport.py b/Lib/test/test_zipimport.py index a2482d4..a7e680d 100644 --- a/Lib/test/test_zipimport.py +++ b/Lib/test/test_zipimport.py @@ -615,7 +615,7 @@ class UncompressedZipImportTestCase(ImportHooksBaseTestCase): z.writestr(zinfo, test_src) z.close() try: - zipimport.zipimporter(filename) + zipimport.zipimporter(filename).load_module(TESTMOD) finally: os.remove(filename) @@ -10,6 +10,9 @@ What's New in Python 3.6.0 beta 1 Core and Builtins ----------------- +- Issue #25758: Prevents zipimport from unnecessarily encoding a filename + (patch by Eryk Sun) + - Issue #25856: The __module__ attribute of extension classes and functions now is interned. This leads to more compact pickle data with protocol 4. diff --git a/Modules/zipimport.c b/Modules/zipimport.c index 6d5c68a..92a82e6 100644 --- a/Modules/zipimport.c +++ b/Modules/zipimport.c @@ -1362,22 +1362,16 @@ normalize_line_endings(PyObject *source) static PyObject * compile_source(PyObject *pathname, PyObject *source) { - PyObject *code, *fixed_source, *pathbytes; - - pathbytes = PyUnicode_EncodeFSDefault(pathname); - if (pathbytes == NULL) - return NULL; + PyObject *code, *fixed_source; fixed_source = normalize_line_endings(source); if (fixed_source == NULL) { - Py_DECREF(pathbytes); return NULL; } - code = Py_CompileString(PyBytes_AsString(fixed_source), - PyBytes_AsString(pathbytes), - Py_file_input); - Py_DECREF(pathbytes); + code = Py_CompileStringObject(PyBytes_AsString(fixed_source), + pathname, Py_file_input, NULL, 1); + Py_DECREF(fixed_source); return code; } |