◐ Shell
reader mode source ↗
Skip to content

gh-141004: Document PyDict_GET_SIZE#141078

Merged
ZeroIntensity merged 1 commit into
python:mainfrom
ZeroIntensity:document-pydict-get-size
Nov 5, 2025
Merged

gh-141004: Document PyDict_GET_SIZE#141078
ZeroIntensity merged 1 commit into
python:mainfrom
ZeroIntensity:document-pydict-get-size

Conversation

@ZeroIntensity

@ZeroIntensity ZeroIntensity commented Nov 5, 2025 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member

@bedevere-app bedevere-app Bot added awaiting core review docs Documentation in the Doc dir skip news labels Nov 5, 2025
@github-project-automation github-project-automation Bot moved this to Todo in Docs PRs Nov 5, 2025
@bedevere-app bedevere-app Bot mentioned this pull request Nov 5, 2025
Open
StanFromIreland
StanFromIreland reviewed Nov 5, 2025
View reviewed changes

@StanFromIreland StanFromIreland left a comment •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

LGTM

Tangent: I must say it is a curious duplication (albeit demoting checks to an assert(PyDict_Check(op));), but I did not look much into it, there may be some good reasoning. Although it seems like something that should be deprecated.

@ZeroIntensity

ZeroIntensity commented Nov 5, 2025 •
edited
Loading

Copy link
Copy Markdown
Member Author

We have similar variations for other APIs, such as PyList_GET_SIZE and PyTuple_GET_SIZE. When you're certain that the object is of the correct type, it's faster to use these. PyDict_GET_SIZE seems to be used in quite a few third-party packages, so I'm opposed to deprecating it.

Thanks for the review!

@ZeroIntensity ZeroIntensity added bugs and security fixes needs backport to 3.14 bugs and security fixes labels Nov 5, 2025
Hide details View details @ZeroIntensity ZeroIntensity merged commit f0ab07f into python:main Nov 5, 2025
39 checks passed
@ZeroIntensity ZeroIntensity deleted the document-pydict-get-size branch November 5, 2025 22:32
@github-project-automation github-project-automation Bot moved this from Todo to Done in Docs PRs Nov 5, 2025
@miss-islington-app

miss-islington-app Bot commented Nov 5, 2025

Copy link
Copy Markdown

Thanks @ZeroIntensity for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 5, 2025
@ZeroIntensity @miss-islington
pythongh-141004: Document PyDict_GET_SIZE (pythonGH-141078) …
0a25ded 
(cherry picked from commit f0ab07f)

Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
@bedevere-app

bedevere-app Bot commented Nov 5, 2025

Copy link
Copy Markdown

GH-141082 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Nov 5, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 5, 2025
@ZeroIntensity @miss-islington
pythongh-141004: Document PyDict_GET_SIZE (pythonGH-141078) …
cd508d4 
(cherry picked from commit f0ab07f)

Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
@bedevere-app

bedevere-app Bot commented Nov 5, 2025

Copy link
Copy Markdown

GH-141083 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Nov 5, 2025
ZeroIntensity added a commit that referenced this pull request Nov 5, 2025
@miss-islington @ZeroIntensity
[3.14] gh-141004: Document PyDict_GET_SIZE (GH-141078) (GH-141082) …
5561837 
gh-141004: Document `PyDict_GET_SIZE` (GH-141078)
(cherry picked from commit f0ab07f)

Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
ZeroIntensity added a commit that referenced this pull request Nov 5, 2025
@miss-islington @ZeroIntensity
[3.13] gh-141004: Document PyDict_GET_SIZE (GH-141078) (GH-141083) …
fc059db 
gh-141004: Document `PyDict_GET_SIZE` (GH-141078)
(cherry picked from commit f0ab07f)

Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
StanFromIreland pushed a commit to StanFromIreland/cpython that referenced this pull request Dec 6, 2025
@ZeroIntensity @StanFromIreland
pythongh-141004: Document PyDict_GET_SIZE (pythonGH-141078)
0bbcd58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@StanFromIreland StanFromIreland StanFromIreland approved these changes

Assignees

No one assigned

Labels

docs Documentation in the Doc dir

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ZeroIntensity @StanFromIreland