gh-102960: Make frames weak-referenceable - #152838
Merged
ambv merged 4 commits intoJul 14, 2026
Merged
Conversation
Copy link
Copy Markdown
Contributor
Sorry, something went wrong.
Add an explicit f_weakreflist field to the frame object, following the same pattern as generators and coroutines, including free-threading-safe weakref clearing via FT_CLEAR_WEAKREFS() in frame_dealloc(). Py_TPFLAGS_MANAGED_WEAKREF is not used because static builtin types must not carry it (see init_static_type()) and the pre-header would cost two extra words per frame instead of one. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
ambv
requested review from
AA-Turner,
kumaraditya303 and
markshannon
as code owners
July 2, 2026 01:12
Copy link
Copy Markdown
Documentation build overview6 files changed · ± 6 modified ± Modified |
Sorry, something went wrong.
Exceptions raised in worker threads don't propagate to the unittest result, so report weakref mismatches via a shared list instead. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Under --parallel-threads a concurrent collection can turn a test's own gc.collect() into a no-op, leaving its cycles uncollected. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Comment thread
Lib/test/test_frame.py
Drop tests that exercised generic weakref machinery already covered by test_weakref (proxy, shared refs, WeakValueDictionary, coroutine frames) and fold the callback assertion into the basic test. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
kumaraditya303
approved these changes
Jul 10, 2026
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
Reviewers
2 participants
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.
Frame objects now support weak references, stored in an explicit f_weakreflist field, following the same pattern as generators and coroutines. Weakref clearing in frame_dealloc() uses FT_CLEAR_WEAKREFS(), which handles the free-threaded build correctly (unconditional PyObject_ClearWeakRefs() there, NULL-check fast path on the default build).
Py_TPFLAGS_MANAGED_WEAKREF was considered instead, but static builtin types must not carry it (see init_static_type()), and the managed pre-header would cost two extra words per frame object instead of one.
This is a redo of GH-102962 against current main. Compared to that PR it adds free-threading-safe weakref clearing, a more comprehensive test suite, and docs.
Tested on the default, free-threaded (--disable-gil), and JIT (--enable-experimental-jit) builds.
🤖 Generated with Claude Code