-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
gh-135379: Add back const cast to _PyLong_IsCompact #135706
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
Conversation
!buildbot AMD64 Debian root 3.x |
The regex 'AMD64 Debian root 3.x' did not match any buildbot builder. Is the requested builder in the list of stable builders? |
!buildbot AMD64 Debian root 3 |
The regex 'AMD64 Debian root 3' did not match any buildbot builder. Is the requested builder in the list of stable builders? |
!buildbot AMD64 CentOS9 |
🤖 New build scheduled with the buildbot fleet by @Fidget-Spinner for commit 991c098 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F135706%2Fmerge The command will test the builders whose names match following regular expression: The builders matched are:
|
!buildbot AMD64 CentOS9 NoGIL PR |
🤖 New build scheduled with the buildbot fleet by @Fidget-Spinner for commit 219d9d1 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F135706%2Fmerge The command will test the builders whose names match following regular expression: The builders matched are:
|
@@ -125,9 +125,9 @@ _PyLong_IsCompact(const PyLongObject* op) { | |||
} | |||
|
|||
static inline int | |||
PyLong_CheckCompact(const PyObject *op) | |||
PyLong_CheckCompact(PyObject *op) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to use PyObject *
here because PyLong_CheckExact
casts to PyObject *
which discards the const qualifier.
Uh oh!
There was an error while loading. Please reload this page.