Skip to content

gh-108444: Replace _PyLong_AsInt() with PyLong_AsInt()#108459

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.

Sign up for GitHub

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

Jump to bottom
Merged
vstinner merged 1 commit into from
Aug 24, 2023
Merged

gh-108444: Replace _PyLong_AsInt() with PyLong_AsInt() #108459

vstinner merged 1 commit into from
Aug 24, 2023

Conversation

@vstinner
Copy link
Member

@vstinnervstinner commented Aug 24, 2023•
edited by bedevere-bot
Loading

Change generated by the command:

sed -i -e 's!_PyLong_AsInt!PyLong_AsInt!g' \ $(find -name "*.c" -o -name "*.h")

@vstinner
pythongh-108444: Replace _PyLong_AsInt() with PyLong_AsInt()
1dd3358
Change generated by the command: sed -i -e 's!_PyLong_AsInt!PyLong_AsInt!g' \ $(find -name "*.c" -o -name "*.h")
@bedevere-botbedevere-bot mentioned this pull request Aug 24, 2023
Closed
corona10
corona10 approved these changes Aug 24, 2023
View reviewed changes
Copy link
Member

@corona10corona10 left a comment

Choose a reason for hiding this comment

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

lgtm!

@vstinnervstinner merged commit b32d4ca into python:mainAug 24, 2023
@vstinnervstinner deleted the use_pylong_asint branch August 24, 2023 23:01
@vstinner
Copy link
MemberAuthor

vstinner commented Aug 24, 2023

Thanks for the review @corona10.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@corona10corona10corona10 approved these changes

@berkerpeksagberkerpeksagAwaiting requested review from berkerpeksagberkerpeksag is a code owner

@erlend-aaslanderlend-aaslandAwaiting requested review from erlend-aaslanderlend-aasland is a code owner

@pgansslepganssleAwaiting requested review from pgansslepganssle is a code owner

@abalkinabalkinAwaiting requested review from abalkinabalkin is a code owner

@markshannonmarkshannonAwaiting requested review from markshannonmarkshannon is a code owner

@iritkatrieliritkatrielAwaiting requested review from iritkatrieliritkatriel is a code owner

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vstinner@corona10@bedevere-bot