Skip to content

Feat: cvx locker gas#17

Open
GalloDaSballo wants to merge 4 commits intoCitadel-DAO:mainfrom
GalloDaSballo:feat-cvx-locker-gas
Open

Feat: cvx locker gas#17
GalloDaSballo wants to merge 4 commits intoCitadel-DAO:mainfrom
GalloDaSballo:feat-cvx-locker-gas

Conversation

@GalloDaSballo
Copy link
Copy Markdown
Contributor

Minor gas savings for CVX locker, loop, some storage vars

NOTE: Honestly if there's no tests merging this changes will be scary AF

@GalloDaSballo
Copy link
Copy Markdown
Contributor Author

Usual suspects:

  • Cached storage vars if use more than once
  • unchecked { ++i; }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant