Omniscia Impact Market Audit
PACTTimelock Code Style Findings
PACTTimelock Code Style Findings
PAT-01C: Redundant Usage of SafeMath
| Type | Severity | Location |
|---|---|---|
| Language Specific |  | PACTTimelock.sol:L2, L7 |
Description:
The linked contract utilizes SafeMath when it is compiled with a pragma version of 0.8.X.
Example:
contracts/governor/PACTTimelock.sol
2pragma solidity 0.8.4;3
4import "@openzeppelin/contracts/utils/math/SafeMath.sol";5
6contract PACTTimelock {7 using SafeMath for uint256;Recommendation:
Given that safe arithmetics are toggled on by default in these versions, we advise the usage of SafeMath to be omitted from the codebase as it incurs an additional gas cost at verbosal benefit.
Alleviation:
The redundant usage of SafeMath has been omitted across the system.