Skip to content

gh-141510: Update PyDict_Copy documentation with note on frozendict - #145249

Merged
ZeroIntensity merged 2 commits into
python:mainfrom
eendebakpt:pydict_copy_docs
Feb 26, 2026
Merged

gh-141510: Update PyDict_Copy documentation with note on frozendict#145249
ZeroIntensity merged 2 commits into
python:mainfrom
eendebakpt:pydict_copy_docs

Conversation

@eendebakpt

@eendebakpteendebakpt commented Feb 26, 2026•
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

@bedevere-appbedevere-appBot mentioned this pull request Feb 26, 2026
Closed
@bedevere-appbedevere-appBot added docs Documentation in the Doc dir skip news labels Feb 26, 2026
ZeroIntensity
ZeroIntensity reviewed Feb 26, 2026
View reviewed changes
Comment threadDoc/c-api/dict.rst Outdated
@ZeroIntensity
ZeroIntensity merged commit 4401f23 into python:mainFeb 26, 2026
33 checks passed
@github-project-automationgithub-project-automationBot moved this from Todo to Done in Docs PRsFeb 26, 2026
brijkapadia pushed a commit to brijkapadia/cpython that referenced this pull request Feb 28, 2026
@eendebakpt@brijkapadia
pythongh-141510: UpdatePyDict_Copydocumentation with note on `fro…
50beac6
…zendict` (pythonGH-145249)
@vstinner

vstinner commented Mar 4, 2026

Copy link
Copy Markdown
Member

I'm not sure that it's a good idea to return a frozendict if PyDict_Copy() is called on a frozendict.

I wrote #145517 to change PyDict_Copy() API to always return a dict.

ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
@eendebakpt@ljfp
pythongh-141510: UpdatePyDict_Copydocumentation with note on `fro…
284bbd7
…zendict` (pythonGH-145249)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ZeroIntensityZeroIntensityZeroIntensity approved these changes

Assignees

No one assigned

Labels

docsDocumentation in the Doc dirskip news

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@eendebakpt@vstinner@ZeroIntensity