Skip to content

🐞 Fix Base58 decodeWord - #1545

Merged
atarpara merged 2 commits into
mainfrom
base58-fix
Aug 2, 2026
Merged

🐞 Fix Base58 decodeWord#1545
atarpara merged 2 commits into
mainfrom
base58-fix

Conversation

@atarpara

@atarpara atarpara commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator

Description

Describe the changes made in your pull request here.

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.

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown

Gas Snapshot Comparison Report

Generated at commit : 6d69cfe, Compared to commit : ab96a83

Contract Name Test Name Main Gas PR Gas Diff
Base58Test testBase58EncodeDecode() 2005919 2006851 932
testDecodeGas() 13574 13552 -22
testDecodeWordGas() 7004 7774 770
testDecodeWordInvalidCharacterReverts() 10646 11508 862
testDecodeWordOverflowsReverts() 43363 47474 4111
test__codesize() 18238 20571 2333
testDecodeWordLowCharacterReverts() - 5786037 -

@atarpara
atarpara merged commit c251232 into main Aug 2, 2026
13 checks passed
@atarpara
atarpara deleted the base58-fix branch August 2, 2026 09:03
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