[Python-checkins] gh-106182: sys: Intern getfilesystemencoding() and getfilesystemencodeerrors() (#106183)
methane
webhook-mailer at python.org
Fri Jun 30 00:02:07 EDT 2023
https://github.com/python/cpython/commit/f1034ba7f67400e7ed7e299dc8b22521c4e43246
commit: f1034ba7f67400e7ed7e299dc8b22521c4e43246
branch: main
author: Inada Naoki <songofacandy at gmail.com>
committer: methane <songofacandy at gmail.com>
date: 2023-06-30T13:02:03+09:00
summary:
gh-106182: sys: Intern getfilesystemencoding() and getfilesystemencodeerrors() (#106183)
sys: Intern getfilesystemencoding() and getfilesystemencodeerrors()
files:
A Misc/NEWS.d/next/Core and Builtins/2023-06-28-15-19-59.gh-issue-106182.cDSFi0.rst
M Python/sysmodule.c
diff --git a/Misc/NEWS.d/next/Core and Builtins/2023-06-28-15-19-59.gh-issue-106182.cDSFi0.rst b/Misc/NEWS.d/next/Core and Builtins/2023-06-28-15-19-59.gh-issue-106182.cDSFi0.rst
new file mode 100644
index 0000000000000..ca2116b00a665
--- /dev/null
+++ b/Misc/NEWS.d/next/Core and Builtins/2023-06-28-15-19-59.gh-issue-106182.cDSFi0.rst
@@ -0,0 +1,2 @@
+:func:`sys.getfilesystemencoding` and :mod:`sys.getfilesystemencodeerrors`
+now return interned Unicode object.
diff --git a/Python/sysmodule.c b/Python/sysmodule.c
index 681024064ad40..8e3fbe3314a95 100644
--- a/Python/sysmodule.c
+++ b/Python/sysmodule.c
@@ -863,6 +863,13 @@ sys_exit_impl(PyObject *module, PyObject *status)
}
+static PyObject *
+get_utf8_unicode(void)
+{
+ _Py_DECLARE_STR(utf_8, "utf-8");
+ PyObject *ret = &_Py_STR(utf_8);
+ return Py_NewRef(ret);
+}
/*[clinic input]
sys.getdefaultencoding
@@ -874,9 +881,7 @@ static PyObject *
sys_getdefaultencoding_impl(PyObject *module)
/*[clinic end generated code: output=256d19dfcc0711e6 input=d416856ddbef6909]*/
{
- _Py_DECLARE_STR(utf_8, "utf-8");
- PyObject *ret = &_Py_STR(utf_8);
- return Py_NewRef(ret);
+ return get_utf8_unicode();
}
/*[clinic input]
@@ -891,7 +896,17 @@ sys_getfilesystemencoding_impl(PyObject *module)
{
PyInterpreterState *interp = _PyInterpreterState_GET();
const PyConfig *config = _PyInterpreterState_GetConfig(interp);
- return PyUnicode_FromWideChar(config->filesystem_encoding, -1);
+
+ if (wcscmp(config->filesystem_encoding, L"utf-8") == 0) {
+ return get_utf8_unicode();
+ }
+
+ PyObject *u = PyUnicode_FromWideChar(config->filesystem_encoding, -1);
+ if (u == NULL) {
+ return NULL;
+ }
+ _PyUnicode_InternInPlace(interp, &u);
+ return u;
}
/*[clinic input]
@@ -906,7 +921,12 @@ sys_getfilesystemencodeerrors_impl(PyObject *module)
{
PyInterpreterState *interp = _PyInterpreterState_GET();
const PyConfig *config = _PyInterpreterState_GetConfig(interp);
- return PyUnicode_FromWideChar(config->filesystem_errors, -1);
+ PyObject *u = PyUnicode_FromWideChar(config->filesystem_errors, -1);
+ if (u == NULL) {
+ return NULL;
+ }
+ _PyUnicode_InternInPlace(interp, &u);
+ return u;
}
/*[clinic input]
More information about the Python-checkins
mailing list