From c7d2d69d95b263ee5f83511bc6fbe53acdc24ea3 Mon Sep 17 00:00:00 2001 From: Victor Stinner Date: Thu, 12 Mar 2020 08:38:11 +0100 Subject: [PATCH] bpo-39884: Add method name in "bad call flags" error (GH-18944) PyDescr_NewMethod() and PyCFunction_NewEx() now include the method name in the SystemError "bad call flags" error message to ease debug. --- .../NEWS.d/next/C API/2020-03-12-00-27-26.bpo-39884.CGOJBO.rst | 2 ++ Objects/descrobject.c | 3 ++- Objects/methodobject.c | 3 ++- 3 files changed, 6 insertions(+), 2 deletions(-) create mode 100644 Misc/NEWS.d/next/C API/2020-03-12-00-27-26.bpo-39884.CGOJBO.rst diff --git a/Misc/NEWS.d/next/C API/2020-03-12-00-27-26.bpo-39884.CGOJBO.rst b/Misc/NEWS.d/next/C API/2020-03-12-00-27-26.bpo-39884.CGOJBO.rst new file mode 100644 index 000000000000..c65dfdc21244 --- /dev/null +++ b/Misc/NEWS.d/next/C API/2020-03-12-00-27-26.bpo-39884.CGOJBO.rst @@ -0,0 +1,2 @@ +:c:func:`PyDescr_NewMethod` and :c:func:`PyCFunction_NewEx` now include the +method name in the SystemError "bad call flags" error message to ease debug. diff --git a/Objects/descrobject.c b/Objects/descrobject.c index 4ebbb74151a2..b448ec642683 100644 --- a/Objects/descrobject.c +++ b/Objects/descrobject.c @@ -888,7 +888,8 @@ PyDescr_NewMethod(PyTypeObject *type, PyMethodDef *method) vectorcall = method_vectorcall_O; break; default: - PyErr_SetString(PyExc_SystemError, "bad call flags"); + PyErr_Format(PyExc_SystemError, + "%s() method: bad call flags", method->ml_name); return NULL; } diff --git a/Objects/methodobject.c b/Objects/methodobject.c index 0d4570534b1a..16abded38546 100644 --- a/Objects/methodobject.c +++ b/Objects/methodobject.c @@ -56,7 +56,8 @@ PyCFunction_NewEx(PyMethodDef *ml, PyObject *self, PyObject *module) vectorcall = cfunction_vectorcall_O; break; default: - PyErr_SetString(PyExc_SystemError, "bad call flags"); + PyErr_Format(PyExc_SystemError, + "%s() method: bad call flags", ml->ml_name); return NULL; } -- 2.47.3