Skip to content

Comments

gh-145119: Allow frozendict to be assigned to instance __dict__#145123

Open
mohsinm-dev wants to merge 1 commit intopython:mainfrom
mohsinm-dev:gh-145119-frozendict-instance-dict
Open

gh-145119: Allow frozendict to be assigned to instance __dict__#145123
mohsinm-dev wants to merge 1 commit intopython:mainfrom
mohsinm-dev:gh-145119-frozendict-instance-dict

Conversation

@mohsinm-dev
Copy link
Contributor

@mohsinm-dev mohsinm-dev commented Feb 22, 2026

Allow frozendict (and its subclasses) to be assigned to an instance's __dict__, enabling immutable instances.

Changes:

  • _PyObject_SetDict: accept frozendict via PyAnyDict_Check
  • _PyDict_SetItem_LockHeld: raise TypeError on set/delete instead of SystemError
  • object___dir___impl: convert frozendict to a mutable dict for merging

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6b702cf72b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

_PyObject_SetDict(PyObject *obj, PyObject *value)
{
if (value != NULL && !PyDict_Check(value)) {
if (value != NULL && !PyAnyDict_Check(value)) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Avoid frozendict dict on objects mutating their own dict

Allowing _PyObject_SetDict() to accept PyAnyDict_Check here makes frozendict assignable to every PyObject_GenericSetDict user, but some of those types still perform internal writes with PyDict_SetItem/PyDict_DelItem (for example descriptor_get_wrapped_attribute and descriptor_set_wrapped_attribute in Objects/funcobject.c for classmethod/staticmethod annotation caching). PyDict_SetItem still rejects frozendict with PyErr_BadInternalCall, so after obj.__dict__ = frozendict(...), operations like reading or updating __annotations__ can now raise SystemError instead of behaving normally or reporting an intentional immutability error.

Useful? React with 👍 / 👎.

@mohsinm-dev mohsinm-dev force-pushed the gh-145119-frozendict-instance-dict branch from 6b702cf to 8d6ec8d Compare February 22, 2026 19:22
Sign up for free to 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.

1 participant