Skip to content
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

gh-92839: fixed typo in _bisectmodule.c (line 131) #92849

Merged
merged 5 commits into from May 28, 2022

Conversation

oda-gitso
Copy link
Contributor

@oda-gitso oda-gitso commented May 16, 2022

Issue #92839. Line 131 says if (x == NULL). It appears if (key_x == NULL) was intended.

I am not sure exactly what the procedure is here. The same typo exists in earlier versions of Python as well. Do we need to do anything about that?

@oda-gitso oda-gitso requested a review from rhettinger as a code owner May 16, 2022
@cpython-cla-bot
Copy link

@cpython-cla-bot cpython-cla-bot bot commented May 16, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented May 16, 2022

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

@rhettinger
Copy link
Contributor

@rhettinger rhettinger commented May 16, 2022

Please add the OP's test case. And also a Misc/NEWS entry.

@oda-gitso
Copy link
Contributor Author

@oda-gitso oda-gitso commented May 16, 2022

@rhettinger Sure I can do this.

I noticed insort_left (line 259) has the same typo. I have added another commit.

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented May 16, 2022

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

@oda-gitso
Copy link
Contributor Author

@oda-gitso oda-gitso commented May 21, 2022

@rhettinger Are these changes okay?

@rhettinger rhettinger merged commit 7fa9b7d into python:main May 28, 2022
13 checks passed
@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented May 28, 2022

Thanks @oda-gitso for the PR, and @rhettinger for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒🤖 I'm not a witch! I'm not a witch!

@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented May 28, 2022

Sorry @oda-gitso and @rhettinger, I had trouble checking out the 3.11 backport branch.
Please backport using cherry_picker on command line.
cherry_picker 7fa9b7daa5a8bb5760724ac2d94f5298c60dc905 3.11

miss-islington pushed a commit to miss-islington/cpython that referenced this issue May 28, 2022
)

(cherry picked from commit 7fa9b7d)

Co-authored-by: oda-gitso <105083118+oda-gitso@users.noreply.github.com>
@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented May 28, 2022

GH-93321 is a backport of this pull request to the 3.10 branch.

@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented May 28, 2022

Thanks @oda-gitso for the PR, and @rhettinger for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒🤖

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented May 28, 2022

GH-93322 is a backport of this pull request to the 3.11 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this issue May 28, 2022
)

(cherry picked from commit 7fa9b7d)

Co-authored-by: oda-gitso <105083118+oda-gitso@users.noreply.github.com>
miss-islington added a commit that referenced this issue May 28, 2022
(cherry picked from commit 7fa9b7d)

Co-authored-by: oda-gitso <105083118+oda-gitso@users.noreply.github.com>
@oda-gitso oda-gitso deleted the fix-issue-92839 branch May 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants