gh-112087: Make list.sort to be thread-safe for PEP 703. by corona10 · Pull Request #116553 · python/cpython
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 34.4k
Merged
corona10 merged 3 commits intopython:mainfrom
Mar 10, 2024Merged
gh-112087: Make list.sort to be thread-safe for PEP 703.#116553
corona10 merged 3 commits intopython:mainfrom
gh-112087: Make list.sort to be thread-safe for PEP 703.#116553
corona10 merged 3 commits intopython:mainfrom
Conversation
Copy link Copy Markdown
Member
corona10
added
the
topic-free-threading
label
corona10
requested a review
from colesbury
bedevere-app
bot
added
the
awaiting core review
label
bedevere-app
bot
mentioned this pull request
4 tasks
corona10
added
the
skip news
label
Copy link Copy Markdown
Member Author
corona10
commented
Mar 9, 2024
corona10 commented
Mar 9, 2024- No resizing in here
- QSBR was already applied:
Lines 2854 to 2861 in 17d31bf
#ifdef Py_GIL_DISABLED bool use_qsbr = _PyObject_GC_IS_SHARED(self); #else bool use_qsbr = false; #endif free_list_items(final_ob_item, use_qsbr); } return Py_XNewRef(result);
corona10
changed the title
gh-112087: Make list.sort to be threaf-safe for PEP 703.
gh-112087: Make list.sort to be thread-safe for PEP 703.
colesbury reviewed Mar 10, 2024
corona10
requested a review
from colesbury
colesbury approved these changes Mar 10, 2024
bedevere-app
bot
added
awaiting merge
and removed
awaiting core review
labels
corona10
enabled auto-merge (squash)
corona10
merged commit
5b2f21f
into
python:main
bedevere-app
bot
removed
the
awaiting merge
label
corona10
deleted the
gh-112087-sort
branch
adorilson pushed a commit to adorilson/cpython that referenced this pull request
Mar 25, 2024diegorusso pushed a commit to diegorusso/cpython that referenced this pull request
Apr 17, 2024This 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment