Submission Details
Severity:
Modifier gas optimization for inState modifier
Summary
Modifier gas optimization for inState modifier
It is recommended to move the modifiers checks into an internal virtual function. This reduces the size of compiled contracts that use the modifiers. Putting the check in an internal function decreases contract size when a modifier is used multiple times.
- modifier inState(State expectedState) {
- if (s_state != expectedState) {
- revert Escrow__InWrongState(s_state, expectedState);
- }
- _;
- }
+ modifier inState(State expectedState) {
+ checkState(expectedState);
+ _;
+ }
+ function checkState(State expectedState) internal view virtual {
+ if (s_state != expectedState) {
+ revert Escrow__InWrongState(s_state, expectedState);
+ }
+ }
Prize pool breakdown
Total prize
40,000 USDC
nSLOC
186215 USDC / LOC
37,000 USDC
3,000 USDC
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.