FastFlaunchZap.sol

The Fast Flaunch Zap allows for tokens to be flaunched with only a small number of the variables being customisable, whilst the majority are preset and immutable.

[INFO] Incorrect code commenting

Whilst reviewing the contract we noticed that there were some instances where code commenting did not make sense, potentially incorrectly inferred or had a typo:

• Code commenting infers that deployment fees are always removed, but this is a separate contract control
• FastFlaunchParams make note that the creator will receive "premined ERC20 tokens" but there is no premine logic
• "with a standardised value" -> "with standardised values"

Recommendations

Although not a bug, we feel that the CreatorFeeAllocation should be either a constant or upgradable. We would recommend making the constants upgradable as redeployment would also require an update of fee exemptions.