-
Notifications
You must be signed in to change notification settings - Fork 38
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
Convert for to do while to save gas on multiple runs #10
base: main
Are you sure you want to change the base?
Conversation
* Fix readme example * Use solidity highlighting * Update
Can you rebase and post the gas difference before and after? |
Consistent usage of `.` at end of sentences
It seems the commit does not pass the test, the following should be changed I combined this idea with my PR with a gas difference before and after here |
[PASS] testGetTokenDataForChipSignature() (gas: 127921 -> 127892) |
After adding laubs unchecked to do while cut gas down significantly #13 [PASS] testGetTokenDataForChipSignature() (gas: 127921 -> 127752) |
No description provided.