Skip to content

gh-117398: datetime: Make use of C-API capsules for sub-interpreters #117413

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

Closed
neonene wants to merge 16 commits into from
Closed

gh-117398: datetime: Make use of C-API capsules for sub-interpreters #117413

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
a9dee17
do not cache sub interp capi
neonene Apr 1, 2024
4c9d200
Apply PEP 7
neonene Apr 1, 2024
836f5cf
non-null main on subinterp test
neonene Apr 1, 2024
34f5a6f
don't touch main-ptr during tests
neonene Apr 1, 2024
321ea7d
fix test_datetime_capi()
neonene Apr 1, 2024
39186c2
check if legacy module or not
neonene Apr 1, 2024
b8e545f
try again 836f5cf
neonene Apr 1, 2024
d2de01a
renames
neonene Apr 2, 2024
1f90272
globals-to-fix.tsv
neonene Apr 3, 2024
777fdbb
edit globals-to-fix
neonene Apr 3, 2024
c8c2d43
Merge branch 'main' into capsule
neonene Apr 28, 2024
da769ba
fix gil-disabled test
neonene Apr 28, 2024
fb2145c
use cache instead of interp state
neonene May 1, 2024
511bbf8
Merge branch 'main' into capsule
neonene May 1, 2024
041b1dd
Update globals-to-fix.tsv
neonene May 1, 2024
ab5b968
Merge branch 'main' into capsule
neonene May 4, 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
Prev Previous commit
Next Next commit
try again 836f5cf
  • Loading branch information
@neonene
neonene committed Apr 1, 2024
commit b8e545fd29ef6340c2b7de58f227a7a406c40731
7 changes: 6 additions & 1 deletion Modules/_testmultiphase.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -959,14 +959,19 @@ PyInit__test_shared_gil_only(void)
static int
datetime_capi_exec(PyObject *m)
{
int ismain = PyInterpreterState_Get() == PyInterpreterState_Main();
if (ismain) {
_pydatetimeapi_main = NULL;
}

PyDateTime_IMPORT;
if (PyDateTimeAPI == NULL) {
return -1;
}
if (PyDateTimeAPI != PyCapsule_Import(PyDateTime_CAPSULE_NAME, 0)) {
return -1;
}
if (PyInterpreterState_Get() == PyInterpreterState_Main()) {
if (ismain) {
if (PyDateTimeAPI != _pydatetimeapi_main) {
return -1;
}
Expand Down