Unnecessary imports #50
Labels
0 (Non-critical)
Code style, clarity, syntax, versioning, off-chain monitoring (events etc), exclude gas optimisation
bug
Warden finding
disagree with severity
Sponsor confirms validity, but disagrees with warden’s risk assessment (sponsor explain in comments)
resolved
Finding has been patched by sponsor (sponsor pls link to PR containing fix)
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
Handle
pauliax
Vulnerability details
Impact
There are imports that are not necessary. They will increase the size of deployment with no real benefit. Consider removing them to save some gas.
Examples of such imports are:
in contract DrawBuffer imported twice:
import "./interfaces/IDrawBeacon.sol";
in contract DrawCalculator:
import "./libraries/DrawRingBufferLib.sol";
import "./PrizeDistributor.sol";
in contract PrizeDistributor:
import "./interfaces/IDrawBeacon.sol";
Recommended Mitigation Steps
Consider removing imports that are not actually used in the contract.
The text was updated successfully, but these errors were encountered: