-
Notifications
You must be signed in to change notification settings - Fork 3
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
Feature merkle trie proof #6
Conversation
sircoon4
commented
Jul 17, 2024
•
edited
Loading
edited
- Added merkle trie inclusion proof for libplanet as precompiled contract.
- Included tests for 4 success cases and 4 failure(return false) cases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with some comments
- not sure core/vm/libplanet is fine path
- seems there's a convention with RequiredGas to be placed in protocol_params.go
Line 1197 in 4a64d62
return params.P256VerifyGas
op-geth/params/protocol_params.go
Line 169 in 4a64d62
P256VerifyGas uint64 = 3450 // secp256r1 elliptic curve signature verifier gas price
@eseiker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM