8000 gh-71587: Establish global state in `_datetime` by erlend-aasland · Pull Request #110475 · python/cpython · GitHub
[go: up one dir, main page]
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
merged 21 commits into from
Oct 12, 2023
Merged

gh-71587: Establish global state in _datetime #110475

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: use shorter names
  • Loading branch information
@erlend-aasland
erlend-aasland committed Oct 10, 2023
commit beb0977b0e1924f2eaa784fd5efd81b9b2136be2
45 changes: 22 additions & 23 deletions Modules/_datetimemodule.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,10 +51,10 @@ typedef struct {
PyObject *seconds_per_day; // 3600 * 24 as Python int

/* The interned UTC timezone instance */
PyObject *PyDateTime_TimeZone_UTC;
PyObject *utc;

/* The interned Epoch datetime instance */
PyObject *PyDateTime_Epoch;
PyObject *epoch;
} datetime_state;

static datetime_state global_state;
Expand Down Expand Up @@ -1194,7 +1194,7 @@ new_timezone(PyObject *offset, PyObject *name)

if (name == NULL && delta_bool((PyDateTime_Delta *)offset) == 0) {
datetime_state *st = GLOBAL_STATE();
return Py_NewRef(st->PyDateTime_TimeZone_UTC);
return Py_NewRef(st->utc);
}
if ((GET_TD_DAYS(offset) == -1 &&
GET_TD_SECONDS(offset) == 0 &&
Expand Down Expand Up @@ -1408,7 +1408,7 @@ tzinfo_from_isoformat_results(int rv, int tzoffset, int tz_useconds)
// Create a timezone from offset in seconds (0 returns UTC)
if (tzoffset == 0) {
datetime_state *st = GLOBAL_STATE();
return Py_NewRef(st->PyDateTime_TimeZone_UTC);
return Py_NewRef(st->utc);
}

PyObject *delta = new_delta(0, tzoffset, tz_useconds, 1);
Expand Down Expand Up @@ -3953,7 +3953,7 @@ timezone_repr(PyDateTime_TimeZone *self)
const char *type_name = Py_TYPE(self)->tp_name;

datetime_state *st = GLOBAL_STATE();
if (((PyObject *)self) == st->PyDateTime_TimeZone_UTC) {
if (((PyObject *)self) == st->utc) {
return PyUnicode_FromFormat("%s.utc", type_name);
}

Expand All @@ -3976,7 +3976,7 @@ timezone_str(PyDateTime_TimeZone *self)
return Py_NewRef(self->name);
}
datetime_state *st = GLOBAL_STATE();
if ((PyObject *)self == st->PyDateTime_TimeZone_UTC ||
if ((PyObject *)self == st->utc ||
(GET_TD_DAYS(self->offset) == 0 &&
GET_TD_SECONDS(self->offset) == 0 &&
GET_TD_MICROSECONDS(self->offset) == 0))
Expand Down Expand Up @@ -6149,7 +6149,7 @@ local_timezone(PyDateTime_DateTime *utc_time)
PyObject *seconds;

datetime_state *st = GLOBAL_STATE();
delta = datetime_subtract((PyObject *)utc_time, st->PyDateTime_Epoch);
delta = datetime_subtract((PyObject *)utc_time, st->epoch);
if (delta == NULL)
return NULL;
one_second = new_delta(0, 1, 0, 0);
Expand Down Expand Up @@ -6273,7 +6273,7 @@ datetime_astimezone(PyDateTime_DateTime *self, PyObject *args, PyObject *kw)
DATE_GET_MINUTE(result),
DATE_GET_SECOND(result),
DATE_GET_MICROSECOND(result),
st->PyDateTime_TimeZone_UTC,
st->utc,
DATE_GET_FOLD(result),
Py_TYPE(result));
Py_DECREF(temp);
Expand All @@ -6282,7 +6282,7 @@ datetime_astimezone(PyDateTime_DateTime *self, PyObject *args, PyObject *kw)
}
else {
/* Result is already aware - just replace tzinfo. */
Py_SETREF(result->tzinfo, Py_NewRef(st->PyDateTime_TimeZone_UTC));
Py_SETREF(result->tzinfo, Py_NewRef(st->utc));
}

/* Attach new tzinfo and let fromutc() do the rest. */
Expand Down Expand Up @@ -6388,7 +6388,7 @@ datetime_timestamp(PyDateTime_DateTime *self, PyObject *Py_UNUSED(ignored))
if (HASTZINFO(self) && self->tzinfo != Py_None) {
datetime_state *st = GLOBAL_STATE();
PyObject *delta;
delta = datetime_subtract((PyObject *)self, st->PyDateTime_Epoch);
delta = datetime_subtract((PyObject *)self, st->epoch);
if (delta == NULL)
return NULL;
result = delta_total_seconds(delta, NULL);
Expand Down Expand Up @@ -6709,11 +6709,11 @@ get_datetime_capi(void)
capi->Date_FromTimestamp = datetime_date_fromtimestamp_capi;
capi->DateTime_FromDateAndTimeAndFold = new_datetime_ex2;
capi->Time_FromTimeAndFold = new_time_ex2;
// Make sure this function is called after PyDateTime_TimeZone_UTC has
// Make sure this function is called after utc has
// been initialized.
datetime_state *st = GLOBAL_STATE();
assert(st->PyDateTime_TimeZone_UTC != NULL);
capi->TimeZone_UTC = st->PyDateTime_TimeZone_UTC; // borrowed ref
assert(st->utc != NULL);
capi->TimeZone_UTC = st->utc; // borrowed ref
return capi;
}

Expand Down Expand Up @@ -6800,18 +6800,18 @@ _datetime_exec(PyObject *module)
return -1;
}

PyObject *x = create_timezone(delta, NULL);
PyObject *utc = create_timezone(delta, NULL);
Py_DECREF(delta);
if (x == NULL) {
if (utc == NULL) {
return -1;
}
if (PyDict_SetItemString(d, "utc", x) < 0) {
Py_DECREF(x);
if (PyDict_SetItemString(d, "utc", utc) < 0) {
Py_DECREF(utc);
return -1;
}

datetime_state *st = GLOBAL_STATE();
st->PyDateTime_TimeZone_UTC = x;
st->utc = utc;

/* bpo-37642: These attributes are rounded to the nearest minute for backwards
* compatibility, even though the constructor will accept a wider range of
Expand All @@ -6821,7 +6821,7 @@ _datetime_exec(PyObject *module)
return -1;
}

x = create_timezone(delta, NULL);
PyObject *x = create_timezone(delta, NULL);
Py_DECREF(delta);
DATETIME_ADD_MACRO(d, "min", x);

Expand All @@ -6835,9 +6835,8 @@ _datetime_exec(PyObject *module)
DATETIME_ADD_MACRO(d, "max", x);

/* Epoch */
st->PyDateTime_Epoch = new_datetime(1970, 1, 1, 0, 0, 0, 0,
st->PyDateTime_TimeZone_UTC, 0);
if (st->PyDateTime_Epoch == NULL) {
st->epoch = new_datetime(1970, 1, 1, 0, 0, 0, 0, st->utc, 0);
if (st->epoch == NULL) {
return -1;
}

Expand All @@ -6863,7 +6862,7 @@ _datetime_exec(PyObject *module)
return -1;
}

if (PyModule_AddObjectRef(module, "UTC", st->PyDateTime_TimeZone_UTC) < 0) {
if (PyModule_AddObjectRef(module, "UTC", st->utc) < 0) {
return -1;
}

Expand Down
0