Skip to content

[3.10] bpo-46006: Revert "bpo-40521: Per-interpreter interned strings (GH-20085)" (GH-30422) - #30433

Closed
vstinner wants to merge 1 commit into
python:3.10from
vstinner:revert_interned310_copy
Closed

[3.10] bpo-46006: Revert "bpo-40521: Per-interpreter interned strings (GH-20085)" (GH-30422)#30433
vstinner wants to merge 1 commit into
python:3.10from
vstinner:revert_interned310_copy

Conversation

@vstinner

@vstinnervstinner commented Jan 6, 2022

Copy link
Copy Markdown
Member

This reverts commit ea25180.

Keep "assert(interned == NULL);" in _PyUnicode_Fini(), but only for
the main interpreter.

Keep _PyUnicode_ClearInterned() changes avoiding the creation of a
temporary Python list object.

Leave PyInterpreterState structure unchanged to keep the ABI backward
compatibility with Python 3.10.0: just rename "interned" to
"unused_interned".

(cherry picked from commit 35d6540)

https://bugs.python.org/issue46006

)" (GH-30422)
This reverts commit ea25180.
Keep "assert(interned == NULL);" in _PyUnicode_Fini(), but only for
the main interpreter.
Keep _PyUnicode_ClearInterned() changes avoiding the creation of a
temporary Python list object.
Leave PyInterpreterState structure unchanged to keep the ABI backward
compatibility with Python 3.10.0: just rename "interned" to
"unused_interned".
(cherry picked from commit 35d6540)
@vstinner

Copy link
Copy Markdown
MemberAuthor

I cannot merge #30425 because of an "unresolved conversation", but I can no longer access the conversation, since it's gone after a git push --force. So I created this new PR.

@vstinnervstinner changed the title bpo-46006: Revert "bpo-40521: Per-interpreter interned strings (GH-20085)" (GH-30422)[3.10] bpo-46006: Revert "bpo-40521: Per-interpreter interned strings (GH-20085)" (GH-30422)Jan 6, 2022
@vstinner

Copy link
Copy Markdown
MemberAuthor

I merged #30425

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@vstinner@the-knights-who-say-ni@bedevere-bot