Unused imports

[code423n4]

Handle

robee

Vulnerability details

In the following files there are contract imports that aren't used.
Import of unnecessary files costs deployment gas (and is a bad coding practice that is important to ignore).
The following is a full list of all unused imports, we went through the whole code to find it :) :

    RebalanceManager.sol, line 7, import "@uniswap/v3-periphery/contracts/interfaces/IQuoterV2.sol";
    CallFacet.sol, line 7, import "../shared/Access/CallProtection.sol";
    LibERC20.sol, line 4, import "@openzeppelin/contracts/token/ERC20/IERC20.sol";
    IPangolinRebalanceManager.sol, line 4, import {IPangolinRouter} from "@pangolindex/exchange-contracts/contracts/pangolin-periphery/interfaces/IPangolinRouter.sol";
    IPangolinRebalanceManager.sol, line 5, import "./IExperiPie.sol";
    MintableERC20.sol, line 3, import {ERC20Upgradeable, ERC20PermitUpgradeable} from "@openzeppelin/contracts-upgradeable/token/ERC20/extensions/draft-ERC20PermitUpgradeable.sol";

[hemulin]

Unused imports are indeed bad practice.
However, they do not have impact on the generated byte code and gas used for deployment.

[0xleastwood]

I think they do increase the size of bytecode if I'm not mistaken