code-423n4 / 2021-09-wildcredit-findings

0 stars 0 forks source link

Unused imports #107

Open code423n4 opened 2 years ago

code423n4 commented 2 years ago

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: import './interfaces/IERC20.sol'; in InterestRateModel import './interfaces/IERC20.sol'; in LPTokenMaster import 'uniswap/uniswap-v3-core@1.0.0/contracts/libraries/FixedPoint128.sol'; and import 'uniswap/uniswap-v3-core@1.0.0/contracts/libraries/FullMath.sol'; and import './external/PositionKey.sol'; in UniswapV3Helper

Recommended Mitigation Steps

Consider removing unused imports mentioned above.

talegift commented 2 years ago

All of the imports mentioned in UniswapV3Helper are being used.

The rest is correct. Will remove them.