Uh oh!
There was an error while loading. Please reload this page.
gh-141510: Support frozendict in plistlib - #145590
Conversation
vstinner
left a comment
There was a problem hiding this comment.
LGTM.
@serhiy-storchaka@ronaldoussoren: Do you want to double check this plistlib change?
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
serhiy-storchaka
left a comment
There was a problem hiding this comment.
Please add a NEWS entry. Note that support of frozendict is half-hearted -- only for serialization. dict_type=frozendict does not work in load(). I am not sure that it is worth to add support for deserialization.
vstinner
left a comment
There was a problem hiding this comment.
Oh, can you also add plistlib to https://docs.python.org/dev/whatsnew/3.15.html#whatsnew315-frozendict?
vstinner
commented
Mar 16, 2026
I saw that and hesitated to propose a follow-up change to add |
serhiy-storchaka
commented
Mar 16, 2026
This can add an overhead and this would not work in case of cyclic references in binary Plist. Originally, So, there is very little value in such feature, and it is very difficult to do without breaking corner cases. But this should be explicitly stated that frozendicts are supported only in one way (like tuple). |
Co-authored-by: Victor Stinner <vstinner@python.org>
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: Victor Stinner <vstinner@python.org>
Co-authored-by: Victor Stinner <vstinner@python.org>
📚 Documentation preview 📚: https://cpython-previews--145590.org.readthedocs.build/