Skip to content

gh-116946: Revert GC protocol for immutable empty heap types (GH-138266, GH-138288, GH-138289) - #138338

Merged
kumaraditya303 merged 3 commits into
python:mainfrom
picnixz:fix/315/gc/revert-for-immutable-empty-types
Sep 1, 2025
Merged

gh-116946: Revert GC protocol for immutable empty heap types (GH-138266, GH-138288, GH-138289)#138338
kumaraditya303 merged 3 commits into
python:mainfrom
picnixz:fix/315/gc/revert-for-immutable-empty-types

Conversation

@picnixz

@picnixzpicnixz commented Sep 1, 2025

Copy link
Copy Markdown
Member

This reverts commits 9be91f6, 3ea16f9 and 6f1dd95. I mistakenly assumed that immutable empty types also needed to support the GC protocol but this is not the case.

@picnixzpicnixz changed the title gh-116946: Revert GC protocol for immutable empty heap types (GH-138266, GH-138288)gh-116946: Revert GC protocol for immutable empty heap types (GH-138266, GH-138288, 138289)Sep 1, 2025
@picnixzpicnixz changed the title gh-116946: Revert GC protocol for immutable empty heap types (GH-138266, GH-138288, 138289)gh-116946: Revert GC protocol for immutable empty heap types (GH-138266, GH-138288, GH-138289)Sep 1, 2025
@kumaraditya303
kumaraditya303 merged commit c9b252c into python:mainSep 1, 2025
47 checks passed
@picnixz
picnixz deleted the fix/315/gc/revert-for-immutable-empty-types branch September 1, 2025 15:49
@picnixz

Copy link
Copy Markdown
MemberAuthor

Thank you and sorry for the noise.

lkollar pushed a commit to lkollar/cpython that referenced this pull request Sep 9, 2025
…ythonGH-138266, pythonGH-138288, pythonGH-138289) (python#138338)
* Revert "pythongh-116946: fully implement GC protocol for `bz2` objects (python#138266)"
This reverts commit 9be91f6.
* Revert "pythongh-116946: fully implement GC protocol for `lzma` objects (python#138288)"
This reverts commit 3ea16f9.
* Revert "pythongh-116946: fully implement GC protocol for `_hashlib` objects (python#138289)"
This reverts commit 6f1dd95.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@picnixz@kumaraditya303