[Python-checkins] bpo-43510: Fix emitting EncodingWarning from _io module. (GH-25146)
methane
webhook-mailer at python.org
Fri Apr 2 04:39:19 EDT 2021
https://github.com/python/cpython/commit/bec8c787ec72d73b39011bde3f3a93e9bb1174b7
commit: bec8c787ec72d73b39011bde3f3a93e9bb1174b7
branch: master
author: Inada Naoki <songofacandy at gmail.com>
committer: methane <songofacandy at gmail.com>
date: 2021-04-02T17:38:59+09:00
summary:
bpo-43510: Fix emitting EncodingWarning from _io module. (GH-25146)
I forget to check PyErr_WarnEx() return value. But it will fail when -Werror is used.
files:
M Modules/_io/_iomodule.c
M Modules/_io/textio.c
diff --git a/Modules/_io/_iomodule.c b/Modules/_io/_iomodule.c
index 652c2ce5b0d61..170dea41e8abd 100644
--- a/Modules/_io/_iomodule.c
+++ b/Modules/_io/_iomodule.c
@@ -532,8 +532,10 @@ _io_text_encoding_impl(PyObject *module, PyObject *encoding, int stacklevel)
if (encoding == NULL || encoding == Py_None) {
PyInterpreterState *interp = _PyInterpreterState_GET();
if (_PyInterpreterState_GetConfig(interp)->warn_default_encoding) {
- PyErr_WarnEx(PyExc_EncodingWarning,
- "'encoding' argument not specified", stacklevel);
+ if (PyErr_WarnEx(PyExc_EncodingWarning,
+ "'encoding' argument not specified", stacklevel)) {
+ return NULL;
+ }
}
Py_INCREF(_PyIO_str_locale);
return _PyIO_str_locale;
diff --git a/Modules/_io/textio.c b/Modules/_io/textio.c
index 6f89a879c9c2b..eb05ae1a16eb0 100644
--- a/Modules/_io/textio.c
+++ b/Modules/_io/textio.c
@@ -1085,6 +1085,19 @@ _io_TextIOWrapper___init___impl(textio *self, PyObject *buffer,
self->ok = 0;
self->detached = 0;
+ if (encoding == NULL) {
+ PyInterpreterState *interp = _PyInterpreterState_GET();
+ if (_PyInterpreterState_GetConfig(interp)->warn_default_encoding) {
+ if (PyErr_WarnEx(PyExc_EncodingWarning,
+ "'encoding' argument not specified", 1)) {
+ return -1;
+ }
+ }
+ }
+ else if (strcmp(encoding, "locale") == 0) {
+ encoding = NULL;
+ }
+
if (errors == Py_None) {
errors = _PyUnicode_FromId(&PyId_strict); /* borrowed */
if (errors == NULL) {
@@ -1123,17 +1136,6 @@ _io_TextIOWrapper___init___impl(textio *self, PyObject *buffer,
self->encodefunc = NULL;
self->b2cratio = 0.0;
- if (encoding == NULL) {
- PyInterpreterState *interp = _PyInterpreterState_GET();
- if (_PyInterpreterState_GetConfig(interp)->warn_default_encoding) {
- PyErr_WarnEx(PyExc_EncodingWarning,
- "'encoding' argument not specified", 1);
- }
- }
- else if (strcmp(encoding, "locale") == 0) {
- encoding = NULL;
- }
-
if (encoding == NULL) {
/* Try os.device_encoding(fileno) */
PyObject *fileno;
More information about the Python-checkins
mailing list