chore: gas tweak

.forge-snapshots/BinHookTest#testBurnSucceedsWithHook.snap

@@ -1 +1 @@
-142950
+143903

.forge-snapshots/BinHookTest#testDonateSucceedsWithHook.snap

@@ -1 +1 @@
-134297
+135247

.forge-snapshots/BinHookTest#testInitializeSucceedsWithHook.snap

@@ -1 +1 @@
-108533
+109118

.forge-snapshots/BinHookTest#testMintSucceedsWithHook.snap

@@ -1 +1 @@
-300345
+301295

.forge-snapshots/BinHookTest#testSwapSucceedsWithHook.snap

@@ -1 +1 @@
-138390
+139320

.forge-snapshots/BinPoolManagerTest#testBurnNativeCurrency.snap

@@ -1 +1 @@
-93346
+90617

.forge-snapshots/BinPoolManagerTest#testGasBurnHalfBin.snap

@@ -1 +1 @@
-68539
+65810

.forge-snapshots/BinPoolManagerTest#testGasBurnNineBins.snap

@@ -1 +1 @@
-152077
+149365

.forge-snapshots/BinPoolManagerTest#testGasBurnOneBin.snap

@@ -1 +1 @@
-69718
+66989

.forge-snapshots/BinPoolManagerTest#testGasDonate.snap

@@ -1 +1 @@
-55313
+52582

.forge-snapshots/BinPoolManagerTest#testGasMintNneBins-1.snap

@@ -1 +1 @@
-971345
+968582

.forge-snapshots/BinPoolManagerTest#testGasMintNneBins-2.snap

@@ -1 +1 @@
-122859
+120096

.forge-snapshots/BinPoolManagerTest#testGasMintOneBin-1.snap

@@ -1 +1 @@
-340528
+337772

.forge-snapshots/BinPoolManagerTest#testGasMintOneBin-2.snap

@@ -1 +1 @@
-57589
+54833

.forge-snapshots/BinPoolManagerTest#testGasSwapMultipleBins.snap

@@ -1 +1 @@
-92098
+89370

.forge-snapshots/BinPoolManagerTest#testGasSwapOverBigBinIdGate.snap

@@ -1 +1 @@
-94083
+91355

.forge-snapshots/BinPoolManagerTest#testGasSwapSingleBin.snap

@@ -1 +1 @@
-73587
+70859

.forge-snapshots/BinPoolManagerTest#testMintNativeCurrency.snap

@@ -1 +1 @@
-322235
+319479

.forge-snapshots/BinPoolManagerTest#testNoOpGas_Burn.snap

@@ -1 +1 @@
-44517
+41726

.forge-snapshots/BinPoolManagerTest#testNoOpGas_Donate.snap

@@ -1 +1 @@
-20863
+19494

.forge-snapshots/BinPoolManagerTest#testNoOpGas_Initialize.snap

@@ -1 +1 @@
-42510
+37255

.forge-snapshots/BinPoolManagerTest#testNoOpGas_Mint.snap

@@ -1 +1 @@
-42175
+39359

.forge-snapshots/BinPoolManagerTest#testNoOpGas_Swap.snap

@@ -1 +1 @@
-25539
+22739

.forge-snapshots/BinPoolManagerTest#testSetProtocolFee.snap

@@ -1 +1 @@
-8153
+8148

.forge-snapshots/CLPoolManagerTest#addLiquidity_fromEmpty.snap

@@ -1 +1 @@
-351744
+349002

.forge-snapshots/CLPoolManagerTest#addLiquidity_fromNonEmpty.snap

@@ -1 +1 @@
-62292
+59550

.forge-snapshots/CLPoolManagerTest#addLiquidity_nativeToken.snap

@@ -1 +1 @@
-245337
+242595

.forge-snapshots/CLPoolManagerTest#donateBothTokens.snap

@@ -1 +1 @@
-85388
+82660

.forge-snapshots/CLPoolManagerTest#gasDonateOneToken.snap

@@ -1 +1 @@
-55350
+52622

.forge-snapshots/CLPoolManagerTest#initializeWithoutHooks.snap

@@ -1 +1 @@
-41952
+36693

.forge-snapshots/CLPoolManagerTest#removeLiquidity_toNonEmpty.snap

@@ -1 +1 @@
-45262
+42520

.forge-snapshots/CLPoolManagerTest#swap_againstLiquidity.snap

@@ -1 +1 @@
-57550
+54818

.forge-snapshots/CLPoolManagerTest#swap_leaveSurplusTokenInVault.snap

@@ -1 +1 @@
-103626
+100894

.forge-snapshots/CLPoolManagerTest#swap_runOutOfLiquidity.snap

@@ -1 +1 @@
-25043336
+25040604

.forge-snapshots/CLPoolManagerTest#swap_simple.snap

@@ -1 +1 @@
-38750
+36018

.forge-snapshots/CLPoolManagerTest#swap_useSurplusTokenAsInput.snap

@@ -1 +1 @@
-104388
+101656

.forge-snapshots/CLPoolManagerTest#swap_withHooks.snap

@@ -1 +1 @@
-44489
+41733

.forge-snapshots/CLPoolManagerTest#swap_withNative.snap

@@ -1 +1 @@
-38753
+36021

.forge-snapshots/CLPoolManagerTest#testNoOp_gas_Donate.snap

@@ -1 +1 @@
-22231
+19431

.forge-snapshots/CLPoolManagerTest#testNoOp_gas_Initialize.snap

@@ -1 +1 @@
-42902
+37643

.forge-snapshots/CLPoolManagerTest#testNoOp_gas_ModifyPosition.snap

@@ -1 +1 @@
-32427
+29618

.forge-snapshots/CLPoolManagerTest#testNoOp_gas_Swap.snap

@@ -1 +1 @@
-24680
+21883

src/pool-bin/BinPoolManager.sol

@@ -106,7 +106,7 @@ contract BinPoolManager is IBinPoolManager, Fees, Extsload {
         uint24 swapFee = key.fee.getInitialSwapFee();
         if (swapFee.isSwapFeeTooLarge(SwapFeeLibrary.TEN_PERCENT_FEE)) revert FeeTooLarge();
 
-        if (key.parameters.shouldCall(HOOKS_BEFORE_INITIALIZE_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_BEFORE_INITIALIZE_OFFSET, hooks)) {
             if (hooks.beforeInitialize(msg.sender, key, activeId, hookData) != IBinHooks.beforeInitialize.selector) {
                 revert Hooks.InvalidHookResponse();
             }
@@ -120,7 +120,7 @@ contract BinPoolManager is IBinPoolManager, Fees, Extsload {
         /// @notice Make sure the first event is noted, so that later events from afterHook won't get mixed up with this one
         emit Initialize(id, key.currency0, key.currency1, key.fee, binStep, hooks);
 
-        if (key.parameters.shouldCall(HOOKS_AFTER_INITIALIZE_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_AFTER_INITIALIZE_OFFSET, hooks)) {
             if (hooks.afterInitialize(msg.sender, key, activeId, hookData) != IBinHooks.afterInitialize.selector) {
                 revert Hooks.InvalidHookResponse();
             }
@@ -139,7 +139,7 @@ contract BinPoolManager is IBinPoolManager, Fees, Extsload {
         _checkPoolInitialized(id);
 
         IBinHooks hooks = IBinHooks(address(key.hooks));
-        if (key.parameters.shouldCall(HOOKS_BEFORE_SWAP_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_BEFORE_SWAP_OFFSET, hooks)) {
             bytes4 selector = hooks.beforeSwap(msg.sender, key, swapForY, amountIn, hookData);
             if (key.parameters.isValidNoOpCall(HOOKS_NO_OP_OFFSET, selector)) {
                 // Sentinel return value used to signify that a NoOp occurred.
@@ -170,7 +170,7 @@ contract BinPoolManager is IBinPoolManager, Fees, Extsload {
             emit Swap(id, msg.sender, delta.amount0(), delta.amount1(), activeId, swapFee, feeForProtocol);
         }
 
-        if (key.parameters.shouldCall(HOOKS_AFTER_SWAP_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_AFTER_SWAP_OFFSET, hooks)) {
             if (hooks.afterSwap(msg.sender, key, swapForY, amountIn, delta, hookData) != IBinHooks.afterSwap.selector) {
                 revert Hooks.InvalidHookResponse();
             }
@@ -241,7 +241,7 @@ contract BinPoolManager is IBinPoolManager, Fees, Extsload {
         _checkPoolInitialized(id);
 
         IBinHooks hooks = IBinHooks(address(key.hooks));
-        if (key.parameters.shouldCall(HOOKS_BEFORE_MINT_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_BEFORE_MINT_OFFSET, hooks)) {
             bytes4 selector = hooks.beforeMint(msg.sender, key, params, hookData);
             if (key.parameters.isValidNoOpCall(HOOKS_NO_OP_OFFSET, selector)) {
                 // Sentinel return value used to signify that a NoOp occurred.
@@ -274,7 +274,7 @@ contract BinPoolManager is IBinPoolManager, Fees, Extsload {
         /// @notice Make sure the first event is noted, so that later events from afterHook won't get mixed up with this one
         emit Mint(id, msg.sender, mintArray.ids, mintArray.amounts, compositionFee, feeForProtocol);
 
-        if (key.parameters.shouldCall(HOOKS_AFTER_MINT_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_AFTER_MINT_OFFSET, hooks)) {
             if (hooks.afterMint(msg.sender, key, params, delta, hookData) != IBinHooks.afterMint.selector) {
                 revert Hooks.InvalidHookResponse();
             }
@@ -292,7 +292,7 @@ contract BinPoolManager is IBinPoolManager, Fees, Extsload {
         _checkPoolInitialized(id);
 
         IBinHooks hooks = IBinHooks(address(key.hooks));
-        if (key.parameters.shouldCall(HOOKS_BEFORE_BURN_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_BEFORE_BURN_OFFSET, hooks)) {
             bytes4 selector = hooks.beforeBurn(msg.sender, key, params, hookData);
             if (key.parameters.isValidNoOpCall(HOOKS_NO_OP_OFFSET, selector)) {
                 // Sentinel return value used to signify that a NoOp occurred.
@@ -312,7 +312,7 @@ contract BinPoolManager is IBinPoolManager, Fees, Extsload {
         /// @notice Make sure the first event is noted, so that later events from afterHook won't get mixed up with this one
         emit Burn(id, msg.sender, binIds, amountRemoved);
 
-        if (key.parameters.shouldCall(HOOKS_AFTER_BURN_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_AFTER_BURN_OFFSET, hooks)) {
             if (hooks.afterBurn(msg.sender, key, params, delta, hookData) != IBinHooks.afterBurn.selector) {
                 revert Hooks.InvalidHookResponse();
             }
@@ -330,7 +330,7 @@ contract BinPoolManager is IBinPoolManager, Fees, Extsload {
         _checkPoolInitialized(id);
 
         IBinHooks hooks = IBinHooks(address(key.hooks));
-        if (key.parameters.shouldCall(HOOKS_BEFORE_DONATE_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_BEFORE_DONATE_OFFSET, hooks)) {
             bytes4 selector = hooks.beforeDonate(msg.sender, key, amount0, amount1, hookData);
             if (key.parameters.isValidNoOpCall(HOOKS_NO_OP_OFFSET, selector)) {
                 // Sentinel return value used to signify that a NoOp occurred.
@@ -347,7 +347,7 @@ contract BinPoolManager is IBinPoolManager, Fees, Extsload {
         /// @notice Make sure the first event is noted, so that later events from afterHook won't get mixed up with this one
         emit Donate(id, msg.sender, delta.amount0(), delta.amount1(), binId);
 
-        if (key.parameters.shouldCall(HOOKS_AFTER_DONATE_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_AFTER_DONATE_OFFSET, hooks)) {
             if (hooks.afterDonate(msg.sender, key, amount0, amount1, hookData) != IBinHooks.afterDonate.selector) {
                 revert Hooks.InvalidHookResponse();
             }

src/pool-cl/CLPoolManager.sol

@@ -101,7 +101,7 @@ contract CLPoolManager is ICLPoolManager, Fees, Extsload {
         uint24 swapFee = key.fee.getInitialSwapFee();
         if (swapFee.isSwapFeeTooLarge(SwapFeeLibrary.ONE_HUNDRED_PERCENT_FEE)) revert FeeTooLarge();
 
-        if (key.parameters.shouldCall(HOOKS_BEFORE_INITIALIZE_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_BEFORE_INITIALIZE_OFFSET, hooks)) {
             if (hooks.beforeInitialize(msg.sender, key, sqrtPriceX96, hookData) != ICLHooks.beforeInitialize.selector) {
                 revert Hooks.InvalidHookResponse();
             }
@@ -114,7 +114,7 @@ contract CLPoolManager is ICLPoolManager, Fees, Extsload {
         /// @notice Make sure the first event is noted, so that later events from afterHook won't get mixed up with this one
         emit Initialize(id, key.currency0, key.currency1, key.fee, tickSpacing, hooks);
 
-        if (key.parameters.shouldCall(HOOKS_AFTER_INITIALIZE_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_AFTER_INITIALIZE_OFFSET, hooks)) {
             if (
                 hooks.afterInitialize(msg.sender, key, sqrtPriceX96, tick, hookData)
                     != ICLHooks.afterInitialize.selector
@@ -138,17 +138,16 @@ contract CLPoolManager is ICLPoolManager, Fees, Extsload {
 
         ICLHooks hooks = ICLHooks(address(key.hooks));
 
-        if (params.liquidityDelta > 0 && key.parameters.shouldCall(HOOKS_BEFORE_ADD_LIQUIDITY_OFFSET, key.hooks)) {
+        if (params.liquidityDelta > 0 && key.parameters.shouldCall(HOOKS_BEFORE_ADD_LIQUIDITY_OFFSET, hooks)) {
             bytes4 selector = hooks.beforeAddLiquidity(msg.sender, key, params, hookData);
             if (key.parameters.isValidNoOpCall(HOOKS_NO_OP_OFFSET, selector)) {
                 // Sentinel return value used to signify that a NoOp occurred.
                 return BalanceDeltaLibrary.MAXIMUM_DELTA;
             } else if (selector != ICLHooks.beforeAddLiquidity.selector) {
                 revert Hooks.InvalidHookResponse();
             }
-        } else if (
-            params.liquidityDelta <= 0 && key.parameters.shouldCall(HOOKS_BEFORE_REMOVE_LIQUIDITY_OFFSET, key.hooks)
-        ) {
+        } else if (params.liquidityDelta <= 0 && key.parameters.shouldCall(HOOKS_BEFORE_REMOVE_LIQUIDITY_OFFSET, hooks))
+        {
             bytes4 selector = hooks.beforeRemoveLiquidity(msg.sender, key, params, hookData);
             if (key.parameters.isValidNoOpCall(HOOKS_NO_OP_OFFSET, selector)) {
                 // Sentinel return value used to signify that a NoOp occurred.
@@ -173,15 +172,14 @@ contract CLPoolManager is ICLPoolManager, Fees, Extsload {
         /// @notice Make sure the first event is noted, so that later events from afterHook won't get mixed up with this one
         emit ModifyLiquidity(id, msg.sender, params.tickLower, params.tickUpper, params.liquidityDelta);
 
-        if (params.liquidityDelta > 0 && key.parameters.shouldCall(HOOKS_AFTER_ADD_LIQUIDITY_OFFSET, key.hooks)) {
+        if (params.liquidityDelta > 0 && key.parameters.shouldCall(HOOKS_AFTER_ADD_LIQUIDITY_OFFSET, hooks)) {
             if (
                 hooks.afterAddLiquidity(msg.sender, key, params, delta, hookData) != ICLHooks.afterAddLiquidity.selector
             ) {
                 revert Hooks.InvalidHookResponse();
             }
-        } else if (
-            params.liquidityDelta <= 0 && key.parameters.shouldCall(HOOKS_AFTER_REMOVE_LIQUIDITY_OFFSET, key.hooks)
-        ) {
+        } else if (params.liquidityDelta <= 0 && key.parameters.shouldCall(HOOKS_AFTER_REMOVE_LIQUIDITY_OFFSET, hooks))
+        {
             if (
                 hooks.afterRemoveLiquidity(msg.sender, key, params, delta, hookData)
                     != ICLHooks.afterRemoveLiquidity.selector
@@ -204,7 +202,7 @@ contract CLPoolManager is ICLPoolManager, Fees, Extsload {
 
         ICLHooks hooks = ICLHooks(address(key.hooks));
 
-        if (key.parameters.shouldCall(HOOKS_BEFORE_SWAP_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_BEFORE_SWAP_OFFSET, hooks)) {
             bytes4 selector = hooks.beforeSwap(msg.sender, key, params, hookData);
             if (key.parameters.isValidNoOpCall(HOOKS_NO_OP_OFFSET, selector)) {
                 // Sentinel return value used to signify that a NoOp occurred.
@@ -247,7 +245,7 @@ contract CLPoolManager is ICLPoolManager, Fees, Extsload {
             state.protocolFee
         );
 
-        if (key.parameters.shouldCall(HOOKS_AFTER_SWAP_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_AFTER_SWAP_OFFSET, hooks)) {
             if (hooks.afterSwap(msg.sender, key, params, delta, hookData) != ICLHooks.afterSwap.selector) {
                 revert Hooks.InvalidHookResponse();
             }
@@ -266,7 +264,7 @@ contract CLPoolManager is ICLPoolManager, Fees, Extsload {
         _checkPoolInitialized(id);
 
         ICLHooks hooks = ICLHooks(address(key.hooks));
-        if (key.parameters.shouldCall(HOOKS_BEFORE_DONATE_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_BEFORE_DONATE_OFFSET, hooks)) {
             bytes4 selector = hooks.beforeDonate(msg.sender, key, amount0, amount1, hookData);
             if (key.parameters.isValidNoOpCall(HOOKS_NO_OP_OFFSET, selector)) {
                 // Sentinel return value used to signify that a NoOp occurred.
@@ -283,7 +281,7 @@ contract CLPoolManager is ICLPoolManager, Fees, Extsload {
         /// @notice Make sure the first event is noted, so that later events from afterHook won't get mixed up with this one
         emit Donate(id, msg.sender, amount0, amount1, tick);
 
-        if (key.parameters.shouldCall(HOOKS_AFTER_DONATE_OFFSET, key.hooks)) {
+        if (key.parameters.shouldCall(HOOKS_AFTER_DONATE_OFFSET, hooks)) {
             if (hooks.afterDonate(msg.sender, key, amount0, amount1, hookData) != ICLHooks.afterDonate.selector) {
                 revert Hooks.InvalidHookResponse();
             }