Gas Optimizations

[code423n4]

Using requirement instead of modifier for authorization consumes more gas:

Seven functions at :https://github.com/code-423n4/2022-05-aura/blob/main/contracts/AuraStakingProxy.sol contract checks if the msg.sender is the owner using the require instead of using modifier this consumes gas more,

Uses modifier will consumes less gas, one of the functions as example:

    function setPendingOwner(address _po) external {
        require(msg.sender == owner, "!auth");
        pendingOwner = _po;
    }

change it to :

    modifier isOwner() {
       require(msg.sender==owner,"not owner");
}

    function setPendingOwner(address _po) external isOwner{
        pendingOwner = _po;
    }

[0xMaharishi]

This is so insignifcant - it's an admin fn so has no impact

[dmvt]

Grouping this with the warden’s gas report, #109