-
Notifications
You must be signed in to change notification settings - Fork 883
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
Fix #752: Reuse _computeSlot result in _accountDelta #753
Conversation
This saves a few hundred gas now but it will be a few thousand gas after https://eips.ethereum.org/EIPS/eip-7667 |
Sorry for the delay we've been having audits so the code was frozen. We are starting another set of audits this week so lets try to merge this... today if possible? |
Nice thanks for doing that. I've asked @snreynolds to take a look too which hopefully she will do today. |
The coverage error is 403 unauthorized.
It appears that you need to authorize Here is what it was trying to post:Forge code coverage:
|
Yep no worries! Its just because youre working from a fork branch and they cannot access our github secrets :) |
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
Reviewers @snreynolds @marktoda @hensha256
Fixes #752
Changes