FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.13] gh-118789: Restore hidden `_PyWeakref_ClearRef` (GH-118797) by miss-islington · Pull Request #118903 · python/cpython · GitHub

/ cpython Public

[3.13] gh-118789: Restore hidden _PyWeakref_ClearRef (GH-118797) - #118903

Merged
encukou merged 1 commit into
python:3.13from
miss-islington:backport-db5af7d-3.13
May 10, 2024
Merged

[3.13] gh-118789: Restore hidden _PyWeakref_ClearRef (GH-118797)#118903
encukou merged 1 commit into
python:3.13from
miss-islington:backport-db5af7d-3.13

Conversation

miss-islington commented May 10, 2024
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

_PyWeakref_ClearRef was previously exposed in the public C-API, although
it begins with an underscore and is not documented. It's used by a few
C-API extensions. There is currently no alternative public API that can
replace its use.

_PyWeakref_ClearWeakRefsExceptCallbacks is the only thread-safe way to
use _PyWeakref_ClearRef in the free-threaded build. This exposes the C
symbol, but does not make the API public.
(cherry picked from commit db5af7d)

Co-authored-by: Sam Gross colesbury@gmail.com

_PyWeakref_ClearRef was previously exposed in the public C-API, although
it begins with an underscore and is not documented. It's used by a few
C-API extensions. There is currently no alternative public API that can
replace its use.

_PyWeakref_ClearWeakRefsExceptCallbacks is the only thread-safe way to
use _PyWeakref_ClearRef in the free-threaded build. This exposes the C
symbol, but does not make the API public.
(cherry picked from commit db5af7d)

Co-authored-by: Sam Gross <colesbury@gmail.com>
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL