observations.length is already computed in numObservationsuint256 numObs = observations.length;can be changed into uint256 numObs = numObservations; as suggested by its preceding comment, because its length is already calculated during construction.
PRICE.sol#L212
Pre-incrementing is cheaper than post-incrementing Consider replacing e.g.
i++with++i.Instances: KernelUtils.sol: 49 64
Function not called by the contract can be
externalinstead ofpublicInstances: Kernel.sol: 95 100 439 451 TRSRY.sol: 47 79 MINTR.sol: 20 25 33 37 RANGE.sol: 110 115 219 PRICE.sol: 108 113 VOTES.sol: 22 27 45 55 INSTR.sol: 23 28 37 Governance.sol: 145 151
Pre-compute
.lengthbefore repeated usage, especially in loopsInstances: Governance.sol#L278
observations.lengthis already computed innumObservationsuint256 numObs = observations.length;can be changed intouint256 numObs = numObservations;as suggested by its preceding comment, because its length is already calculated during construction. PRICE.sol#L212