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
Gas Optimizations #550
Labels
bug
Something isn't working
G (Gas Optimization)
G-04
grade-a
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
Comments
1- State variables only set in the constructor should be declared immutable :14.7k At least 14.7k |
C4-Staff
added a commit
that referenced
this issue
Jan 6, 2023
GainsGoblin marked the issue as sponsor confirmed |
c4-sponsor
added
the
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
label
Jan 9, 2023
2- Variables inside struct should be packed to save gas :On average it will save one Cold Slot SLOAD, 2k 3- Multiple address/IDs mappings can be combined into a single mapping of an address/id to a struct :Valid but looks like marginal savings Rest is minor let's say 300 gas |
17000 |
GalloDaSballo marked the issue as grade-a |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
Something isn't working
G (Gas Optimization)
G-04
grade-a
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
See the markdown file with the details of this report here.
The text was updated successfully, but these errors were encountered: