-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
gh-99138: Isolate _zoneinfo #99218
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.
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
gh-99138: Isolate _zoneinfo #99218
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
a5d7a79
Convert zone info type to heap type and add it to global state
45ad0b6
Remove unneeded tp_base placeholder
erlend-aasland accb561
Add io_open to global state
erlend-aasland 61530f0
Add _tzpath_find_tzfile to global state
erlend-aasland 8639a03
Add _common_mod to global state
erlend-aasland e728631
Add TIMEDELTA_CACHE to global state
erlend-aasland 96241c9
Add ZONEINFO_WEAK_CACHE to global state
erlend-aasland 39a0b7c
Move ZONEINFO_STRONG_CACHE to global state
erlend-aasland 4749890
Move ZONEINFO_STRONG_CACHE_MAX_SIZE to constants and make it const
erlend-aasland 5e71905
Add NO_TTINFO to global state
erlend-aasland 4c31528
Add module arg to zoneinfo_get_state()
erlend-aasland 0f1f1ea
Global state to module state
erlend-aasland 573aec1
Style nit
erlend-aasland 773a836
Address review: add comment above cache pointers
erlend-aasland 6e69e93
Sync with main
erlend-aasland 3e4f429
Merge branch 'main' into isolate-zoneinfo
erlend-aasland d4101af
Add NEWS
erlend-aasland 1ef23da
Sync with main
erlend-aasland 23cbc43
Simplify cache init
erlend-aasland ae5c549
Purge zoneinfo from c analyzer todo
erlend-aasland 8adb727
Sync with main
erlend-aasland 7c8836e
Sync with main
erlend-aasland 6d5216e
Address reviews
erlend-aasland b7944dd
Pull in main
erlend-aasland 71392bb
Pull in main
erlend-aasland 244c537
Address review: remove warning about process-wide global state
erlend-aasland c32d46d
Pull in main
erlend-aasland 75f519b
Revert "Address review: remove warning about process-wide global state"
erlend-aasland 3b460ad
Address review: document that initialize_caches() is not idempotent
erlend-aasland a9dbc4d
Address review: reword cache warning only slightly
erlend-aasland File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1 change: 1 addition & 0 deletions
1
Misc/NEWS.d/next/Library/2023-01-02-22-41-44.gh-issue-99138.17hp9U.rst
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
Apply :pep:`687` to :mod:`zoneinfo`. Patch by Erlend E. Aasland. |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.