Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork33.7k
gh-116738: Make _json module safe in the free-threading build#119438
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Merged
Changes from1 commit
Commits
Show all changes
53 commits Select commitHold shift + click to select a range
da0e917 Make the _json module thread safe
eendebakpt3797dfa Update Modules/_json.c
eendebakpt366654c handle goto and return statements
eendebakpt5b72cdf Apply suggestions from code review
eendebakptc4c24c3 Update Include/internal/pycore_critical_section.h
eendebakpt370191b rename macro
eendebakpt93c4466 Merge branch 'main' into json_ft
eendebakpteafd3c1 fix typo
eendebakptdaeec46 Merge branch 'json_ft' of github.com:eendebakpt/cpython into json_ft
eendebakptd54baf2 fix missing to exit critical section
eendebakpte5fa305 revert changes to tests
eendebakptd4ddf5d 📜🤖 Added by blurb_it.
blurb-it[bot]67d942f Merge branch 'main' into json_ft
eendebakpt4ffc1b2 Merge branch 'main' into json_ft
eendebakpt384ca59 sync with main
eendebakpt64e20aa sync with main
eendebakpte6ce9c9 update news entry
eendebakpt34885a0 fix normal build
eendebakpt2fe760b Merge branch 'main' into json_ft
eendebakpteebccac add lock around result of PyMapping_Items
eendebakptdb8947c add tests
eendebakptc19ad14 fix argument of Py_END_CRITICAL_SECTION_SEQUENCE_FAST
eendebakpt8b12e0f Merge branch 'main' into json_ft
eendebakpt78d3595 avoid Py_EXIT_CRITICAL_SECTION_SEQUENCE_FAST
eendebakpt6e8615f use barriers in test
eendebakpt39ebc00 typo
eendebakpt7c5b185 whitespace
eendebakptadf78c7 Merge branch 'main' into json_ft
eendebakptacd0ad1 resolve merge conflicts
eendebakpt41e3dee Update Misc/NEWS.d/next/Core_and_Builtins/2024-06-04-20-26-21.gh-issu…
eendebakpt75884cb cleanup
eendebakpt0424c58 Merge branch 'json_ft' of github.com:eendebakpt/cpython into json_ft
eendebakpt9c964f9 Merge branch 'main' into json_ft
eendebakpt5acc999 Merge branch 'main' into json_ft
eendebakpt5173373 format
eendebakpt7d00562 only use items locally in encoder_listencode_dict
eendebakpt00f4d7f Merge branch 'main' into json_ft
eendebakptb95c07f use strong references in iteration
eendebakptcedc2c3 Merge branch 'main' into json_ft
eendebakpt0ca453d replace Py_IncRef with Py_INCREF
eendebakpt45544f3 Update Modules/_json.c
eendebakptdcb088e Apply suggestion from @kumaraditya303
kumaraditya3035f8ad07 add header
kumaraditya30361bc8d1 Apply suggestions from code review
eendebakptf85b695 review comments
eendebakpt2c0cd18 use _Py_CRITICAL_SECTION_ASSERT_OBJECT_LOCKED
eendebakptc133b71 remove testing code
eendebakpt7609117 Merge branch 'main' into json_ft
eendebakpt7656a2a add incref/decref to fast seq iteration
eendebakpt1920aed Merge branch 'json_ft' of github.com:eendebakpt/cpython into json_ft
eendebakptd5d65ab fix
eendebakpt4a16e18 fix
eendebakpt00139f3 Update Modules/_json.c
kumaraditya303File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
fix missing to exit critical section
- Loading branch information
Uh oh!
There was an error while loading.Please reload this page.
commitd54baf2049a71956b26c45e67fe31bf4601517e8
There are no files selected for viewing
4 changes: 4 additions & 0 deletionsInclude/internal/pycore_critical_section.h
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
4 changes: 3 additions & 1 deletionLib/test/test_json/test_dump.py
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
4 changes: 3 additions & 1 deletionModules/_json.c
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
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.