Platform: Code4rena
Start Date: 05/05/2022
Pot Size: $125,000 DAI
Total HM: 17
Participants: 62
Period: 14 days
Judge: leastwood
Total Solo HM: 15
Id: 120
League: ETH
Rank: 56/62
Findings: 1
Award: $89.43
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: IllIllI
Also found by: 0v3rf10w, 0x1f8b, 0x4non, 0xDjango, 0xNazgul, 0xf15ers, 0xkatana, 0xsomeone, AlleyCat, BowTiedWardens, Cityscape, Fitraldys, Funen, GimelSec, Hawkeye, JC, MaratCerby, MiloTruck, Randyyy, TerrierLover, Tomio, UnusualTurtle, WatchPug, Waze, _Adam, augustg, bobirichman, catchup, csanuragjain, ellahi, fatherOfBlocks, hake, hansfriese, horsefacts, ignacio, joestakey, kenta, mics, oyc_109, robee, samruna, sashik_eth, sikorico, simon135, throttle
89.4325 DAI - $89.43
Recommendations I will mention one example if examples are more than 30.
use != 0 instead of > 0 (more than 30 examples) AlchemistV2.sol#L1267
use ++i instead of i++, i+=1 (more than 30 examples) AlchemistV2.sol#L990
no need to initialize variables with default values (more than 30 examples) AlchemistV2.sol#L990
an array’s length should be cached to save gas in for-loops (more than 30 examples) AlchemistV2.sol#L990
non-strict inequalities are cheaper than strict ones AlchemistV2.sol#L729 AlchemistV2.sol#L773 AlchemistV2.sol#L838 EthAssetManager.sol#L553 EthAssetManager.sol#L711 EthAssetManager.sol#L721 ThreePoolAssetManager.sol#L379 ThreePoolAssetManager.sol#L755 ThreePoolAssetManager.sol#L1027 ThreePoolAssetManager.sol#L1037 Limiters.sol#L100