Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix save mem calls #95

Closed
wants to merge 2 commits into from
Closed

fix save mem calls #95

wants to merge 2 commits into from

Conversation

invocamanman
Copy link
Contributor

  • save mem was not properly handled in the call parameters

ignasirv
ignasirv approved these changes Jul 12, 2022
View reviewed changes
Copy link
Contributor

@ignasirv ignasirv left a comment

Choose a reason for hiding this comment

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

💯

@invocamanman
Copy link
Contributor Author

Added here #96

hecmas added a commit that referenced this pull request Jan 31, 2024
@hecmas
Merge pull request #95 from 0xPolygonHermez/feature/ecrecover-refactor
7b41b2f 
Ecrecover refactor, various optimizations and ecAdd, ecMul and ecPairing counters added
@hecmas hecmas mentioned this pull request Jan 31, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ignasirv ignasirv ignasirv approved these changes

@laisolizq laisolizq Awaiting requested review from laisolizq laisolizq is a code owner

@krlosMata krlosMata Awaiting requested review from krlosMata krlosMata is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@invocamanman @ignasirv