Canto contest - ak1's results

CToken.sol

sloc : 266, 267

uint principalTimesIndex = borrowSnapshot.principal * borrowIndex; return principalTimesIndex / borrowSnapshot.interestIndex;

we can avoid using extra variable, this could save gas

and can have like this

return (borrowSnapshot.principal * borrowIndex) / borrowSnapshot.interestIndex;

inside mintFresh function,

in line 214 , 217

require(getCashPrior() == actualMintAmount, "CNote::mintFresh: Error in doTransferIn, CNote reserves >= mint Amount"); //sanity check that Accountant has some thing to redeem
    
    //underlying.balanceOf(address(this)) == repayAmount;
    uint amtMinted = getCashPrior();
    uint err = _accountant.redeemMarket(amtMinted);
    if (err != 0) {
        revert AccountantRedeemError(amtMinted);
    }

getCashPrior() can be called once and assigned to a variable and that variable can be used. This will save gas. likewise some other places also could be checked.

Comptroller.sol

inside function claimComp, pre-increment operator can be used. declaring int j as common variable inside the function and use it wherever possible by proper initilization.

NoteInterest.sol

function getBorrowRate(uint cash, uint borrows, uint reserves) public view override returns (uint) { // Gets the Note/gUSDC TWAP in a given interval, as a mantissa (scaled by 1e18) // uint twapMantissa = getUnderlyingPrice(note); uint rand = uint(keccak256(abi.encodePacked(msg.sender))) % 100; uint ir = (100 - rand).mul(adjusterCoefficient).add(baseRatePerYear).mul(1e16); uint newRatePerYear = ir >= 0 ? ir : 0; // convert it to base rate per block uint newRatePerBlock = newRatePerYear.div(blocksPerYear); return newRatePerBlock; } line no - 96, value should be 1e18 not the 1e16.