FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-36203: Check callback is callable in PyWeakref_NewRef by tekknolagi · Pull Request #26273 · python/cpython · GitHub

/ cpython Public

bpo-36203: Check callback is callable in PyWeakref_NewRef - #26273

Closed
tekknolagi wants to merge 2 commits into
python:mainfrom
tekknolagi:main
ConversationCommits (2)ChecksFiles changed
Closed

bpo-36203: Check callback is callable in PyWeakref_NewRef#26273
tekknolagi wants to merge 2 commits into
python:mainfrom
tekknolagi:main

Conversation

tekknolagi commented May 20, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor

The documentation says that PyWeakref_NewRef should raise a
TypeError if the callback is not callable, None, or NULL, but the code
does not reflect the documentation. This PR fixes that.

This should probably be backported to whatever version of Python
changed that documentation.

https://bugs.python.org/issue36203

Check callback is callable in PyWeakref_NewRef
a1d0810 
The documentation says that `PyWeakref_NewRef` should raise a
`TypeError` if the callback is not callable, None, or NULL, but the code
does not reflect the documentation. This PR fixes that.
tekknolagi mentioned this pull request May 20, 2021
Closed
tekknolagi changed the title Check callback is callable in PyWeakref_NewRef bpo-36203: Check callback is callable in PyWeakref_NewRef May 20, 2021

tekknolagi commented May 20, 2021

Copy link
Copy Markdown
Contributor Author

Tests failing seem entirely unrelated

github-actions Bot commented Jun 21, 2021

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

github-actions Bot added the stale Stale PR or inactive for long period of time. label Jun 21, 2021

tekknolagi commented Nov 9, 2021

Copy link
Copy Markdown
Contributor Author

bump? :)

MaxwellDupre approved these changes Mar 24, 2022
View reviewed changes

MaxwellDupre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Runs ok on my Ubuntu.

github-actions Bot removed the stale Stale PR or inactive for long period of time. label Aug 8, 2022
iritkatriel requested changes Dec 6, 2022
View reviewed changes

iritkatriel left a comment

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This has merge conflicts now.

bedevere-bot commented Dec 6, 2022

Copy link
Copy Markdown

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

ghost commented Dec 17, 2022

Copy link
Copy Markdown

The following commit authors need to sign the Contributor License Agreement:

Click the button to sign:

kumaraditya303 reviewed Dec 17, 2022
View reviewed changes
Comment thread Misc/NEWS.d/next/C API/2019-03-09-00-10-25.bpo-36203.yKKWd3.rst
@@ -0,0 +1 @@
Amend PyWeakref_New to raise a TypeError if the callback given is not NULL, None, or a callable object. The documentation previously falsely claimed this check occurred. No newline at end of file

kumaraditya303 Dec 17, 2022

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality
Suggested change
Amend PyWeakref_New to raise a TypeError if the callback given is not NULL, None, or a callable object. The documentation previously falsely claimed this check occurred.
Amend :c:func:`PyWeakref_New` to raise :exc:`TypeError` if the *callback* given is not NULL, None, or a callable object. The documentation previously falsely claimed this check occurred.

python-cla-bot Bot commented Apr 6, 2025 •
edited
Loading

Copy link
Copy Markdown

The following commit authors need to sign the Contributor License Agreement:

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 9, 2026

tekknolagi commented Apr 9, 2026

Copy link
Copy Markdown
Contributor Author

I regrettably have no time to work on this. Please feel free to take over or close as appropriate.

github-actions Bot removed the stale Stale PR or inactive for long period of time. label May 10, 2026
serhiy-storchaka self-requested a review June 4, 2026 13:52
tekknolagi mannequin mentioned this pull request Apr 10, 2022
Closed
serhiy-storchaka mentioned this pull request Jun 10, 2026
Merged

serhiy-storchaka commented Jun 10, 2026

Copy link
Copy Markdown
Member

Superseded by #151145.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

kumaraditya303 kumaraditya303 left review comments
iritkatriel iritkatriel requested changes
serhiy-storchaka Awaiting requested review from serhiy-storchaka
+1 more reviewer
MaxwellDupre MaxwellDupre approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

awaiting changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

Back | FazBrowse Home | New Git URL