Skip to content

gh-71587: Establish global state in _datetime #110475

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
erlend-aasland merged 21 commits into from
Oct 12, 2023
Merged

gh-71587: Establish global state in _datetime #110475

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
cacf1e8
Consistency: use explicit initialiser for m_base
erlend-aasland Feb 15, 2023
c2eba8f
Add module state stub; establish global state on stack
erlend-aasland Feb 15, 2023
8d54ecd
Put conversion factors in state struct
erlend-aasland Feb 15, 2023
360397a
Move PyDateTime_TimeZone_UTC to state
erlend-aasland Feb 15, 2023
4f509f9
Move PyDateTime_Epoch to state struct
erlend-aasland Feb 15, 2023
6b14c8c
Remove datetime_strptime globals entry
erlend-aasland Feb 15, 2023
2eb3315
Apply Victor's suggestions from gh-102995
erlend-aasland Oct 6, 2023
f5ede1e
Apply PEP-7 to modified code
erlend-aasland Oct 6, 2023
f6ac875
Merge branch 'main' into isolate-datetime/part1-establish-global-state
erlend-aasland Oct 6, 2023
7139569
Fix check-c-globals
erlend-aasland Oct 6, 2023
26e6454
Merge PR updates
erlend-aasland Oct 6, 2023
227dd2c
Pull in main
erlend-aasland Oct 10, 2023
beb0977
Address review: use shorter names
erlend-aasland Oct 10, 2023
8525be6
Pull in main
erlend-aasland Oct 11, 2023
6863f76
Address review: rename global_state => _datetime_global_state
erlend-aasland Oct 11, 2023
0738205
Pull in main
erlend-aasland Oct 11, 2023
c56b5e1
Address review
erlend-aasland Oct 11, 2023
3ee48d2
Fix ref leak in init
erlend-aasland Oct 11, 2023
0a1d770
Pull in main
erlend-aasland Oct 11, 2023
a9e8d1a
Clean up initialisation
erlend-aasland Oct 11, 2023
c3ba7fe
Preserve comment
erlend-aasland Oct 11, 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
Next Next commit
Address review: rename global_state => _datetime_global_state
  • Loading branch information
@erlend-aasland
erlend-aasland committed Oct 11, 2023
commit 6863f763c07909fa3e921e933d2ed68091b603a6
4 changes: 2 additions & 2 deletions Modules/_datetimemodule.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,9 @@ typedef struct {
PyObject *epoch;
} datetime_state;

static datetime_state global_state;
static datetime_state _datetime_global_state;

#define GLOBAL_STATE() (&global_state)
#define GLOBAL_STATE() (&_datetime_global_state)
Comment thread
erlend-aasland marked this conversation as resolved.
Outdated

/*[clinic input]
module datetime
Expand Down
2 changes: 1 addition & 1 deletion Tools/c-analyzer/cpython/globals-to-fix.tsv
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -423,7 +423,7 @@ Modules/_datetimemodule.c datetime_strptime module -
Modules/_ctypes/_ctypes.c - _ctypes_ptrtype_cache -
Modules/_ctypes/_ctypes.c - global_state -
Modules/_ctypes/ctypes.h - global_state -
Modules/_datetimemodule.c - global_state -
Modules/_datetimemodule.c - _datetime_global_state -
Modules/_tkinter.c - tcl_lock -
Modules/_tkinter.c - excInCmd -
Modules/_tkinter.c - valInCmd -
Expand Down