Skip to content

Conversation

@vstinner
Copy link
Member

@vstinnervstinner commented Jan 6, 2022

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
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
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