Omniscia Native Audit
PeripheryValidation Code Style Findings
PeripheryValidation Code Style Findings
PVN-01C: Redundant Method Implementation
Type | Severity | Location |
---|---|---|
Gas Optimization | PeripheryValidation.sol:L8-L10 |
Description:
The _blockTimestamp
method implementation is redundant as the block.timestamp
statement is more gas efficient.
Example:
contracts/libraries/PeripheryValidation.sol
6/// @dev Method that exists purely to be overridden for tests7/// @return The current block timestamp8function _blockTimestamp() internal view virtual returns (uint256) {9 return block.timestamp;10}
Recommendation:
We advise it to be safely omitted from the codebase and all its usage instances to be replaced by direct block.timestamp
statements.
Alleviation:
The _blockTimestamp
method is no longer present in the codebase and its invocations have been properly replaced by direct block.timestamp
statements.