xTRIBE contest - djxploit's results

Gas optimization issue in the for loop of xtribe.sol L95:

[1] Un-necessary initialisation of loop invariant :

Statement uint256 i=0 is not required, because all uninitialised uint variable, by default, are set to 0

[2] ++i is cheaper than i++ for unsigned integer [3] Value of accounts[i] is loaded twice, so instead load the value of accounts[i] once, store it in a local variable and then use it.

So the final optimised for loop is as follows:

for (uint256 i; i < size; ) {
            uint256 memory acc = accounts[i];
            emit DelegateVotesChanged(acc, 0, getVotes(acc));

            unchecked {
                ++i;
            }
        }