Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-135004: rewrite and cleanupblake2module.c#135006

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
picnixz merged 14 commits intopython:mainfrompicnixz:ref/hashlib/blake-2-new-135004
Jun 9, 2025

Conversation

picnixz
Copy link
Member

@picnixzpicnixz commentedJun 1, 2025
edited
Loading

Strictly speaking, we're gaining a bit as well:

Modules/blake2module.c | 7231 file changed, 357 insertions(+), 366 deletions(-)

I've updatedhmacmodule.c as well so that my two functions for detecting SIMD features have the same body. I could have a separate header, but it's easier that way (ideally, I'd like to rely on#125022 for SIMD detection in the future).

@picnixzpicnixz changed the titlegh-135004: rewrite and cleanup_blake2module.cgh-135004: rewrite and cleanupblake2module.cJun 1, 2025
@picnixz
Copy link
MemberAuthor

!buildbot FIPS

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by@picnixz for commit17365bd 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F135006%2Fmerge

The command will test the builders whose names match following regular expression:FIPS

The builders matched are:

  • AMD64 RHEL8 FIPS Only Blake2 Builtin Hash PR
  • AMD64 RHEL8 FIPS No Builtin Hashes PR
  • AMD64 CentOS9 FIPS No Builtin Hashes PR
  • AMD64 CentOS9 FIPS Only Blake2 Builtin Hash PR

@picnixzpicnixz added the 🔨 test-with-buildbotsTest PR w/ buildbots; report in status section labelJun 1, 2025
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by@picnixz for commit4b4b66c 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F135006%2Fmerge

If you want to schedule another build, you need to add the🔨 test-with-buildbots label again.

@bedevere-botbedevere-bot removed the 🔨 test-with-buildbotsTest PR w/ buildbots; report in status section labelJun 1, 2025
@picnixzpicnixzforce-pushed theref/hashlib/blake-2-new-135004 branch from4b4b66c toce0e848CompareJune 7, 2025 13:34
@picnixzpicnixzforce-pushed theref/hashlib/blake-2-new-135004 branch from70ea98e tob16e301CompareJune 8, 2025 09:30
@picnixzpicnixz added the type-refactorCode refactoring (with no changes in behavior) labelJun 8, 2025
@picnixzpicnixz self-assigned thisJun 8, 2025
@picnixzpicnixz merged commit3cb1097 intopython:mainJun 9, 2025
45 checks passed
@picnixzpicnixz deleted the ref/hashlib/blake-2-new-135004 branchJune 9, 2025 07:17
lkollar pushed a commit to lkollar/cpython that referenced this pull requestJun 19, 2025
* Cleanup imports and update module docstring.* Simplify detection of SIMD support.* Correctly guard `update()` cases.* Rewrite `py_blake2b_or_s_new` and rename it to `py_blake2_new`.* Rewrite `blake2_blake2b_copy_locked` and `py_blake2_clear`.* Refactor computations of `digest` and `hexdigest`.* Simplify `py_blake2b_get_name` and `py_blake2b_get_block_size`.* Add `hacl_get_blake2_info` to extract static BLAKE-2 information.   This new helper is used by `py_blake2b_get_digest_size`, but can   be later used to expose `key_length` more easily.
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@gpsheadgpsheadgpshead approved these changes

@tirantiranAwaiting requested review from tirantiran is a code owner

Assignees

@picnixzpicnixz

Labels
type-refactorCode refactoring (with no changes in behavior)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@picnixz@bedevere-bot@gpshead

[8]ページ先頭

©2009-2025 Movatter.jp