Skip to content

bpo-40865: Remove unused insint() macro from hash modules #20627

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

Merged
merged 2 commits into from
Jun 4, 2020

Conversation

erlend-aasland
Copy link
Contributor

@erlend-aasland erlend-aasland commented Jun 4, 2020

https://bugs.python.org/issue40865

Automerge-Triggered-By: @tiran

@erlend-aasland erlend-aasland changed the title bpo-40865: Remove unused insint() macro from sha512 module bpo-40865: Remove unused insint() macro from hash modules Jun 4, 2020
Copy link
Member

@tiran tiran left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the cleanup!

@miss-islington miss-islington merged commit 6ed578f into python:master Jun 4, 2020
@miss-islington
Copy link
Contributor

Thanks @erlend-aasland for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 4, 2020
…0627)

Automerge-Triggered-By: @tiran
(cherry picked from commit 6ed578f)

Co-authored-by: Erlend Egeberg Aasland <[email protected]>
@bedevere-bot
Copy link

GH-20633 is a backport of this pull request to the 3.9 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Jun 4, 2020
@erlend-aasland erlend-aasland deleted the fix-issue-40865 branch June 4, 2020 20:09
@erlend-aasland
Copy link
Contributor Author

Thanks for reviewing, @tiran!

miss-islington added a commit that referenced this pull request Jun 4, 2020
Automerge-Triggered-By: @tiran
(cherry picked from commit 6ed578f)

Co-authored-by: Erlend Egeberg Aasland <[email protected]>
arun-mani-j pushed a commit to arun-mani-j/cpython that referenced this pull request Jul 21, 2020
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.

5 participants