ERC20 import

[code423n4]

Handle

pauliax

Vulnerability details

Impact

You don't need to import the implementation to interact with the contract, you can import only an interface, e.g. here:

 ERC20 public collateralToken;

Consider replacing ERC20 with IERC20 to reduce deployment costs.

[0xScotch]

#242

[GalloDaSballo]

This is not a duplicate of #242, whereas 242 is about Best Practices, this finding is saying that using ERC20 instead of IERC20 will increase the cost of the deployment.

I believe the optimizer will make the bytecode the same, but will mark this finding as valid because IERC20 would guarantee less work