Skip to content

Docs: PyModule_AddObjectRef should return -1 if value is NULL. #101944

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
c2170 opened this issue Feb 16, 2023 · 2 comments
Closed

Docs: PyModule_AddObjectRef should return -1 if value is NULL. #101944

c2170 opened this issue Feb 16, 2023 · 2 comments
Labels
docs Documentation in the Doc dir topic-C-API

Comments

@vstinner
Copy link
Member

I created a doc change to clarify the NULL case: PR gh-129433.

@picnixz picnixz removed the interpreter-core (Objects, Python, Grammar, and Parser dirs) label Jan 31, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Feb 3, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Feb 3, 2025
@vstinner
Copy link
Member

vstinner commented Feb 3, 2025

Documentation rephrased by change 04264a2.

@vstinner vstinner closed this as completed Feb 3, 2025
vstinner added a commit that referenced this issue Feb 3, 2025
…29433) (#129612)

gh-101944: Clarify PyModule_AddObjectRef() documentation (GH-129433)
(cherry picked from commit 04264a2)

Co-authored-by: Victor Stinner <[email protected]>
vstinner added a commit that referenced this issue Feb 3, 2025
…29433) (#129613)

gh-101944: Clarify PyModule_AddObjectRef() documentation (GH-129433)
(cherry picked from commit 04264a2)

Co-authored-by: Victor Stinner <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir topic-C-API
Projects
Status: Todo
Development

Successfully merging a pull request may close this issue.

5 participants