Skip to content

gh-89546: Clean up PyType_FromMetaclass #93686

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
miss-islington merged 19 commits into from
Jun 14, 2022
Merged

gh-89546: Clean up PyType_FromMetaclass #93686

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
d43d5ca
Raise invalid slot exceptions early
encukou May 31, 2022
5145034
Switch to `switch` for the slots
encukou May 31, 2022
87b28aa
Allocate & copy the docstring early
encukou Jun 1, 2022
9fb6b11
Prepare the type names early
encukou Jun 1, 2022
7fd0c6d
Move offset setting to before PyType_Ready
encukou Jun 1, 2022
06fe302
Add comment on preparing special slots
encukou Jun 1, 2022
8bb54f8
Comment that we don't want to call Python code while a type is half-b…
encukou Jun 1, 2022
6c25cfc
Add comment on the function's refcounting strategy
encukou Jun 1, 2022
7edb478
Check that base & offsets fit in the basicsize
encukou Jun 1, 2022
2075958
Add blurb for the new check
encukou Jun 10, 2022
763ea2d
Move variable declarations out of `switch` cases
encukou Jun 10, 2022
5ca1502
Fix ReST syntax (*sigh*)
encukou Jun 10, 2022
5b18552
Downcast to int to avoid a warning
encukou Jun 13, 2022
34edc2a
Reflow check_basicsize_includes_size_and_offsets to conform to style …
encukou Jun 13, 2022
e45f77f
Reword error messages to say "offset is out of bounds"
encukou Jun 13, 2022
9713fb4
Move modname down to improve scoping
encukou Jun 13, 2022
a997413
Move variables to smaller scopes
encukou Jun 13, 2022
b197bb5
Merge main branch to resolve conflicts
encukou Jun 13, 2022
f53d86c
One more PEP 7 nit!
encukou Jun 14, 2022
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
Next Next commit
Merge main branch to resolve conflicts
  • Loading branch information
@encukou
encukou committed Jun 13, 2022
commit b197bb5bf76ced3d10c501e7a62945d25cda7514
8 changes: 4 additions & 4 deletions Objects/typeobject.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3478,7 +3478,7 @@ PyType_FromMetaclass(PyTypeObject *metaclass, PyObject *module,
PyErr_SetString(
PyExc_SystemError,
"Multiple Py_tp_members slots are not supported.");
return NULL;
goto finally;
}
for (const PyMemberDef *memb = slot->pfunc; memb->name != NULL; memb++) {
nmembers++;
Expand Down Expand Up @@ -3509,7 +3509,7 @@ PyType_FromMetaclass(PyTypeObject *metaclass, PyObject *module,
PyErr_SetString(
PyExc_SystemError,
"Multiple Py_tp_doc slots are not supported.");
return NULL;
goto finally;
}
if (slot->pfunc == NULL) {
PyObject_Free(tp_doc);
Expand Down Expand Up @@ -3730,12 +3730,12 @@ PyType_FromMetaclass(PyTypeObject *metaclass, PyObject *module,
}

if (weaklistoffset) {
if (PyDict_DelItemString((PyObject *)type->tp_dict, "__weaklistoffset__") < 0) {
if (PyDict_DelItem((PyObject *)type->tp_dict, &_Py_ID(__weaklistoffset__)) < 0) {
goto finally;
}
}
if (dictoffset) {
if (PyDict_DelItemString((PyObject *)type->tp_dict, "__dictoffset__") < 0) {
if (PyDict_DelItem((PyObject *)type->tp_dict, &_Py_ID(__dictoffset__)) < 0) {
goto finally;
}
}
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.