GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-141510: Update PyDict_Copy documentation with note on frozendict by eendebakpt · Pull Request #145249 · python/cpython

Conversation

@eendebakpt

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

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor
  • Issue: PEP 814: Add built-in frozendict type #141510

📚 Documentation preview 📚: https://cpython-previews--145249.org.readthedocs.build/

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@eendebakpt
Update PyDict_Copy documentation with note on frozendict
2b71234
@eendebakpt
eendebakpt requested a review from ZeroIntensity as a code owner February 26, 2026 11:09
@bedevere-app bedevere-app Bot added the awaiting review label Feb 26, 2026
@bedevere-app bedevere-app Bot mentioned this pull request Feb 26, 2026
PEP 814: Add built-in frozendict type #141510
Closed
@bedevere-app bedevere-app Bot added docs Documentation in the Doc dir skip news labels Feb 26, 2026
@github-project-automation github-project-automation Bot added this to Docs PRs Feb 26, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Docs PRs Feb 26, 2026
ZeroIntensity
ZeroIntensity reviewed Feb 26, 2026
View reviewed changes
Comment thread Doc/c-api/dict.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
@eendebakpt
review comments
c371baf
ZeroIntensity
ZeroIntensity approved these changes Feb 26, 2026
View reviewed changes
@bedevere-app bedevere-app Bot added awaiting merge and removed awaiting review labels Feb 26, 2026
@ZeroIntensity
ZeroIntensity merged commit 4401f23 into python:main Feb 26, 2026
33 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@github-project-automation github-project-automation Bot moved this from Todo to Done in Docs PRs Feb 26, 2026
@bedevere-app bedevere-app Bot removed the awaiting merge label Feb 26, 2026
brijkapadia pushed a commit to brijkapadia/cpython that referenced this pull request Feb 28, 2026
@eendebakpt @brijkapadia
pythongh-141510: Update PyDict_Copy documentation 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.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
@eendebakpt @ljfp
pythongh-141510: Update PyDict_Copy documentation with note on `fro…
284bbd7 
…zendict` (pythonGH-145249)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ZeroIntensity ZeroIntensity ZeroIntensity approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip news

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@eendebakpt @vstinner @ZeroIntensity
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.