Vyper Vested Claims

First Flight #34
Beginner FriendlyDeFi
100 EXP
View results
Submission Details
Severity: low
Invalid

[G-1] Store variables should be immutable to improve gas efficiency

Summary

The following store variables are initialized in the constructor but not changed after that:

vesting_start_time: public(uint256)
vesting_end_time: public(uint256)
token: public(address)
owner: public(address)

Immutables are not stored in the storage but as a part of the contract. So changing them to immutable could improve the gas efficiency.

Recommendations

- vesting_start_time: public(uint256)
- vesting_end_time: public(uint256)
- token: public(address)
- owner: public(address)
+ VESTING_START_TIME: public(immutable(uint256))
+ VESTING_END_TIME: public(immutable(uint256))
+ TOKEN: public(immutable(address))
+ OWNER: public(immutable(address))
Updates

Appeal created

bube Lead Judge 4 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.