Skip to content

gh-117953: Cleanups For fix_up_extension() in import.c #118192

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
ericsnowcurrently merged 11 commits into from
Apr 24, 2024
+163 −97
Merged

gh-117953: Cleanups For fix_up_extension() in import.c #118192

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
d87d212
Factor out fix_up_extension_for_interpreter().
ericsnowcurrently Apr 22, 2024
b85c5af
Pass tstate through to fix_up_extension().
ericsnowcurrently Apr 17, 2024
319c461
We already know the def is okay.
ericsnowcurrently Apr 19, 2024
e6ea054
Let the modules arg to fix_up_extension_for_interpreter() be NULL.
ericsnowcurrently Apr 19, 2024
439661f
Always expect a valid modules dict passed to fix_up_extension().
ericsnowcurrently Apr 17, 2024
0d28e0f
Do interpreter fixups after the extension cache, not before.
ericsnowcurrently Apr 23, 2024
abd69f4
Call fix_up_extension_for_interpreter() separately.
ericsnowcurrently Apr 23, 2024
f347f8d
fix_up_extension_for_interpreter() -> finish_singlephase_extension()
ericsnowcurrently Apr 23, 2024
4976089
fix_up_extension() -> update_global_state_for_extension()
ericsnowcurrently Apr 23, 2024
5627ad9
Update the comment about extensions.
ericsnowcurrently Apr 23, 2024
8b4e216
Make sure __file__ is set before m_copy gets populated.
ericsnowcurrently Apr 23, 2024
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
2 changes: 2 additions & 0 deletions Include/internal/pycore_import.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,12 @@ extern int _PyImport_ReleaseLock(PyInterpreterState *interp);

// This is used exclusively for the sys and builtins modules:
extern int _PyImport_FixupBuiltin(
PyThreadState *tstate,
PyObject *mod,
const char *name, /* UTF-8 encoded string */
PyObject *modules
);
// We could probably drop this:
extern int _PyImport_FixupExtensionObject(PyObject*, PyObject *,
PyObject *, PyObject *);

Expand Down
Loading