GalloDaSballo
commented
3 years ago Unused imports do not increase the size of the bytecode as the compiler removes them
That said they are a code-smell, changing to non-critical
Open code423n4 opened 3 years ago
GalloDaSballo
commented
3 years ago Unused imports do not increase the size of the bytecode as the compiler removes them
That said they are a code-smell, changing to non-critical
Handle
pauliax
Vulnerability details
Impact
There are unused imports. They will increase the size of deployment with no real benefit. Consider removing unused imports to save some gas.
Examples of such imports are:
in contract PoolKeeper import "@chainlink/contracts/src/v0.8/interfaces/AggregatorV2V3Interface.sol"; import "@openzeppelin/contracts/token/ERC20/IERC20.sol"; import "@openzeppelin/contracts/proxy/Clones.sol";
imported twice: import "../interfaces/IERC20DecimalsWrapper.sol";
in contract PoolCommitter import "../interfaces/IOracleWrapper.sol";
Recommended Mitigation Steps
Remove unnecessary imports.