feat: fix gas optimisation

catalystdao · Feb 22, 2024 · 27bdaf1 · 27bdaf1
1 parent a92176e
commit 27bdaf1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/evm/src/CatalystVaultVolatile.sol b/evm/src/CatalystVaultVolatile.sol
@@ -232,6 +232,9 @@ contract CatalystVaultVolatile is CatalystVaultCommon, IntegralsVolatile {
                 // If the weight has already been reached, skip the mathematics.
                 if (currentWeight == targetWeight) {
                     wsum += targetWeight;
+                    unchecked {
+                        ++it;
+                    }
                     continue;
                 }
 

diff --git a/evm/test/CatalystVault/1Volatile/VolatileOneVault.t.sol b/evm/test/CatalystVault/1Volatile/VolatileOneVault.t.sol
@@ -30,7 +30,7 @@ import { TestSecurityLimitLiquiditySwap } from "../SecurityLimit.ReceiveLiquidit
 import { TestWithdrawEverything } from "../Withdraw/WithdrawEverything.t.sol";
 import {Token} from "../../mocks/token.sol";
 
-contract TestVolatileInvariant is TestInvariant, TestLocalswap, TestCrossChainInterfaceOnly, TestLocalswapMinout, TestPoolTokenInterface, TestSetup, TestSetupFinish, TestSetVaultFee, TestSetGovernanceFee, TestSetWeights, TestLocalswapFees, TestSwapWorthlessTokenLocal, TestEscrow, TestWithdrawInvariant, TestWithdrawComparison,  TestCompareDepositWithWithdraw, TestWithdrawNothing, TestWithdrawUnbalanced, TestSelfSwap, TestVaultConnections, TestSecurityLimitAssetSwap, TestSecurityLimitLiquiditySwap, TestWithdrawEverything { //,TestEvilRouterExploitVolatile  {
+contract TestVolatileInvariant is TestInvariant, TestLocalswap, TestCrossChainInterfaceOnly, TestLocalswapMinout, TestPoolTokenInterface, TestSetup, TestSetupFinish, TestSetVaultFee, TestSetGovernanceFee, TestLocalswapFees, TestSwapWorthlessTokenLocal, TestEscrow, TestWithdrawInvariant, TestWithdrawComparison,  TestCompareDepositWithWithdraw, TestWithdrawNothing, TestWithdrawUnbalanced, TestSelfSwap, TestVaultConnections, TestSecurityLimitAssetSwap, TestSecurityLimitLiquiditySwap, TestWithdrawEverything, TestSetWeights { // TestEvilRouterExploitVolatile  {
 
     address[] _vaults;