Phuture Finance contest - fatherOfBlocks's results

vToken.sol

• L45 - onlyRole could be a function this way the gas is optimized.
• L164 - _totalSupply - oldShares can be unckecked , create a variable and put it inside.
• L164 - _assetBalance - availableAssets can be unckecked , create a variable and put it inside.

UniswapV2PathPriceOracle.sol

• L34.49- i++ can be unchecked outside of the for, on the last line of the scope.

TopNMarketCapIndex.sol

• L48 - i++ can be unchecked outside of the for, on the last line of the scope.
• L55 - the require could be passed to line 51, so less code is executed.

PhuturePriceOracle.sol

• L37 - onlyRole could be a function this way the gas is optimized.

ManagedIndex.sol

• L30 - i++ can be unchecked outside of the for, on the last line of the scope.

BaseIndex.sol

• L28 - onlyRole could be a function this way the gas is optimized.
• L78 - i++ can be unchecked outside of the for, on the last line of the scope.

indexLogic.sol

• L48 - the subtraction can be unchecked, since it is impossible for it to generate underflow.
• L40.44.47.48.49 - create a variable and don't constantly call assets.at(i).
• L61.63.67 - create a variable and don't constantly call inactiveAssets.at.at(i)
• L88.117 - the subtraction can be unchecked, since it is impossible for it to generate underflow.
• L103.110.123.127 - you could create a local variable and not be constantly calling it.

ManagedIndexReweightingLogic.sol

• L39.40 - create a variable and don't constantly call assets.at(i).
• L80.82 - the subtraction can be unchecked, since it is impossible for it to generate underflow.

TopNMarketCapIndexReweightingLogic.sol

• L38.39 - create a variable and don't constantly call assets.at(i).
• L71.96.98 - the subtraction can be unchecked, since it is impossible for it to generate underflow.