Gas Optimizations #330
Labels
bug
Something isn't working
duplicate
This issue or pull request already exists
G (Gas Optimization)
POC
https://github.com/code-423n4/2022-05-aura/blob/0001f78d35f7e45b5244e9cf00e4c323417166e7/contracts/Aura.sol#L68
https://github.com/code-423n4/2022-05-aura/blob/085f573756b132b2a5992c5aa5d7b907cd11c289/contracts/AuraBalRewardPool.sol#L157
https://github.com/code-423n4/2022-05-aura/blob/0001f78d35f7e45b5244e9cf00e4c323417166e7/contracts/AuraLocker.sol#L259
Impact
Save gas use "!=" instead ">"
POC
https://github.com/code-423n4/2022-05-aura/blob/0001f78d35f7e45b5244e9cf00e4c323417166e7/contracts/AuraClaimZap.sol#L138
Impact
Caching array lenght can save gas.
#Navigation
uint256 tokenRewardContractsLen = tokenRewardContracts.length;
....
POC
https://github.com/code-423n4/2022-05-aura/blob/0001f78d35f7e45b5244e9cf00e4c323417166e7/contracts/AuraMerkleDrop.sol#L8
Impact
Save gas if import didnt use in contract.
POC
https://github.com/code-423n4/2022-05-aura/blob/0001f78d35f7e45b5244e9cf00e4c323417166e7/contracts/AuraLocker.sol#L657
Impact
Not defining "cutoffEpoch" can reduce gas cost
Navigation
in https://github.com/code-423n4/2022-05-aura/blob/0001f78d35f7e45b5244e9cf00e4c323417166e7/contracts/AuraLocker.sol#L669
if (lockEpoch < epochStart) {
if (lockEpoch > epochStart.sub(lockDuration)) {
amount = amount.add(locks[i - 1].amount);
} else {
.....
The text was updated successfully, but these errors were encountered: