Skip to content

GH-101578: Normalize the current exception #101607

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
markshannon merged 13 commits into from
Feb 8, 2023
Merged

GH-101578: Normalize the current exception #101607

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
87889bc
Make sure that the current exception is always normalized.
markshannon Feb 6, 2023
14f21ff
Remove redundant type and traceback fields for the current exception.
markshannon Feb 6, 2023
8b62323
Replace a some uses of PyErr_Fetch with PyErr_Fetch1.
markshannon Feb 6, 2023
508850f
Refactor assertion.
markshannon Feb 6, 2023
073409a
Document new C-API functions
markshannon Feb 6, 2023
d260b13
Add news
markshannon Feb 6, 2023
d5480e9
Fix typos
markshannon Feb 6, 2023
0c3dc7b
Update generated code.
markshannon Feb 6, 2023
aa70bad
Rename new functions and fix up docs.
markshannon Feb 7, 2023
1ae8eda
Fix up news entry
markshannon Feb 7, 2023
fe6c34f
Update stable ABI.
markshannon Feb 7, 2023
566f6e1
Restore behavior of PyErr_Restore, and add tests.
markshannon Feb 7, 2023
99f0880
Fix refleaks
markshannon Feb 7, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix refleaks
  • Loading branch information
@markshannon
markshannon committed Feb 7, 2023
commit 99f0880e0d72075850c9abfe91e9a9fd3c8312df
6 changes: 4 additions & 2 deletions Modules/_testcapi/heaptype.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,7 @@ test_from_spec_invalid_metatype_inheritance(PyObject *self, PyObject *Py_UNUSED(
PyObject *exc = NULL;
PyObject *result = NULL;
PyObject *message = NULL;
PyObject *args = NULL;

metaclass_a = PyType_FromSpecWithBases(&MinimalMetaclass_spec, (PyObject*)&PyType_Type);
if (metaclass_a == NULL) {
Expand Down Expand Up @@ -156,13 +157,13 @@ test_from_spec_invalid_metatype_inheritance(PyObject *self, PyObject *Py_UNUSED(
// Assert that the correct exception was raised
if (PyErr_ExceptionMatches(PyExc_TypeError)) {
exc = PyErr_GetRaisedException();
PyObject *args = PyException_GetArgs(exc);
args = PyException_GetArgs(exc);
if (!PyTuple_Check(args) || PyTuple_Size(args) != 1) {
PyErr_SetString(PyExc_AssertionError,
"TypeError args are not a one-tuple");
goto finally;
}
PyObject *message = Py_NewRef(PyTuple_GET_ITEM(args, 0));
message = Py_NewRef(PyTuple_GET_ITEM(args, 0));
meta_error_string = PyUnicode_FromString("metaclass conflict:");
if (meta_error_string == NULL) {
goto finally;
Expand All @@ -188,6 +189,7 @@ test_from_spec_invalid_metatype_inheritance(PyObject *self, PyObject *Py_UNUSED(
Py_XDECREF(message);
Py_XDECREF(class_a);
Py_XDECREF(class_b);
Py_XDECREF(args);
return result;
}

Expand Down
1 change: 1 addition & 0 deletions Python/errors.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,7 @@ _PyErr_Restore(PyThreadState *tstate, PyObject *type, PyObject *value,
}
else {
PyErr_SetString(PyExc_TypeError, "traceback must be a Traceback or None");
Py_XDECREF(value);
Py_DECREF(type);
Py_XDECREF(traceback);
return;
Expand Down