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

gh-132775: Fix _PyFunctIon_VerifyStateless() by ericsnowcurrently · Pull Request #134900 · python/cpython · GitHub

/ cpython Public

gh-132775: Fix _PyFunctIon_VerifyStateless() - #134900

Merged
ericsnowcurrently merged 1 commit into
python:mainfrom
ericsnowcurrently:fix-pyfunction-verify-stateless
May 29, 2025
Merged

gh-132775: Fix _PyFunctIon_VerifyStateless()#134900
ericsnowcurrently merged 1 commit into
python:mainfrom
ericsnowcurrently:fix-pyfunction-verify-stateless

Conversation

ericsnowcurrently commented May 29, 2025
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

The problem we're fixing here is that we were using PyDict_Size() on "defaults", which it is actually a tuple. We're also adding some explicit type checks.

This is a follow-up to gh-133221/gh-133528.

Copy link
Copy Markdown

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 29, 2025
The problem we're fixing here is that we were using PyDict_Size() on "defaults",
which it is actually a tuple.  We're also adding some explicit type checks.

This is a follow-up to pythongh-133221/pythongh-133528.
(cherry picked from commit dafd141)

Co-authored-by: Eric Snow <ericsnowcurrently@gmail.com>

bedevere-app Bot commented May 29, 2025

Copy link
Copy Markdown

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

bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label May 29, 2025
ericsnowcurrently deleted the fix-pyfunction-verify-stateless branch May 29, 2025 20:16
ericsnowcurrently added a commit that referenced this pull request May 29, 2025
gh-132775: Fix _PyFunctIon_VerifyStateless() ()

The problem we're fixing here is that we were using PyDict_Size() on "defaults",
which it is actually a tuple.  We're also adding some explicit type checks.

This is a follow-up to gh-133221/gh-133528.

(cherry picked from commit dafd141, AKA gh-134900)

Co-authored-by: Eric Snow <ericsnowcurrently@gmail.com>
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
The problem we're fixing here is that we were using PyDict_Size() on "defaults",
which it is actually a tuple.  We're also adding some explicit type checks.

This is a follow-up to pythongh-133221/pythongh-133528.
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
The problem we're fixing here is that we were using PyDict_Size() on "defaults",
which it is actually a tuple.  We're also adding some explicit type checks.

This is a follow-up to pythongh-133221/pythongh-133528.
gostak-dd pushed a commit to gostak-dd/cpython that referenced this pull request Jun 2, 2026
The problem we're fixing here is that we were using PyDict_Size() on "defaults",
which it is actually a tuple.  We're also adding some explicit type checks.

This is a follow-up to pythongh-133221/pythongh-133528.
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL