Skip to content

Conversation

@Vectorized
Copy link
Owner

Description

using LibBytes for bytes32;

Checklist

Ensure you completed all of the steps below before submitting your pull request:

  • Ran forge fmt?
  • Ran forge test?

Pull requests with an incomplete checklist will be thrown out.

@Vectorized Vectorized changed the title ♻️ Add bytes32 to address utils ✨ Add bytes32 to address utils Jul 21, 2025
@github-actions
Copy link

github-actions bot commented Jul 21, 2025

Gas Snapshot Comparison Report

Generated at commit : de83711, Compared to commit : a5bb996

Contract Name Test Name Main Gas PR Gas Diff
LibBytesTest testBytesInCalldata() 2786 2764 -22
testCmp() 2629 2607 -22
testDirectReturn() 9260 9283 23
testDynamicStructInCalldata() 4406 4517 111
testDynamicStructInCalldata2() 38901 39077 176
testIndexOfByte() 5456 5434 -22
testSliceCalldata() 3088 3131 43
testStaticStructInCalldata() 20870 20760 -110
test__codesize() 14432 14506 74

@Vectorized Vectorized merged commit 2718072 into main Jul 21, 2025
13 checks passed
@Vectorized Vectorized deleted the bytes32-to-address branch August 18, 2025 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants