Omniscia Alliance Block Audit
LimitedAutoStake Code Style Findings
LimitedAutoStake Code Style Findings
LAS-01C: Variable Mutability Specifier
Type | Severity | Location |
---|---|---|
Gas Optimization | Informational | LimitedAutoStake.sol:L11, L15 |
Description:
The linked stakeLimit
variable is assigned to only once during the constructor
of the contract.
Example:
contracts/autostake-features/LimitedAutoStake.sol
13constructor(address token, uint256 _throttleRoundBlocks, uint256 _throttleRoundCap, uint256 stakeEnd, uint _stakeLimit) public AutoStake(token, _throttleRoundBlocks, _throttleRoundCap, stakeEnd) {14 require(_stakeLimit != 0 , "LimitedAutoStake:constructor::stake limit should not be 0");15 stakeLimit = _stakeLimit;16}
Recommendation:
We advise that the immutable
mutability specifier is introduced to the stakeLimit
variable greatly optimizing the gas cost involved in utilizing it.
Alleviation:
The linked variable was properly set to immutable
greatly optimizing the gas cost involved in utilizing it.