fix: prefix incrementor

.forge-snapshots/Base-DutchOrder-BaseExecuteSingleWithFee.snap

@@ -1 +1 @@
-181978
+150412

.forge-snapshots/Base-DutchOrder-ExecuteBatch.snap

@@ -1 +1 @@
-196879
+196869

.forge-snapshots/Base-DutchOrder-ExecuteBatchMultipleOutputs.snap

@@ -1 +1 @@
-206650
+206640

.forge-snapshots/Base-DutchOrder-ExecuteBatchMultipleOutputsDifferentTokens.snap

@@ -1 +1 @@
-260307
+260297

.forge-snapshots/Base-DutchOrder-ExecuteBatchNativeOutput.snap

@@ -1 +1 @@
-190411
+190401

.forge-snapshots/Base-DutchOrder-ExecuteSingle.snap

@@ -1 +1 @@
-148194
+148189

.forge-snapshots/Base-DutchOrder-ExecuteSingleNativeOutput.snap

@@ -1 +1 @@
-133756
+133751

.forge-snapshots/Base-DutchOrder-ExecuteSingleValidation.snap

@@ -1 +1 @@
-137605
+157500

.forge-snapshots/Base-ExclusiveDutchOrder-BaseExecuteSingleWithFee.snap

@@ -1 +1 @@
-150633
+150956

.forge-snapshots/Base-ExclusiveDutchOrder-ExecuteBatch.snap

@@ -1 +1 @@
-197283
+197273

.forge-snapshots/Base-ExclusiveDutchOrder-ExecuteBatchMultipleOutputs.snap

@@ -1 +1 @@
-207049
+207039

.forge-snapshots/Base-ExclusiveDutchOrder-ExecuteBatchMultipleOutputsDifferentTokens.snap

@@ -1 +1 @@
-260720
+260710

.forge-snapshots/Base-ExclusiveDutchOrder-ExecuteBatchNativeOutput.snap

@@ -1 +1 @@
-190809
+190799

.forge-snapshots/Base-ExclusiveDutchOrder-ExecuteSingle.snap

@@ -1 +1 @@
-148401
+148396

.forge-snapshots/Base-ExclusiveDutchOrder-ExecuteSingleNativeOutput.snap

@@ -1 +1 @@
-95568
+133963

.forge-snapshots/Base-ExclusiveDutchOrder-ExecuteSingleValidation.snap

@@ -1 +1 @@
-157716
+157711

.forge-snapshots/Base-LimitOrderReactor-BaseExecuteSingleWithFee.snap

@@ -1 +1 @@
-178116
+178111

.forge-snapshots/Base-LimitOrderReactor-ExecuteBatch.snap

@@ -1 +1 @@
-189056
+189046

.forge-snapshots/Base-LimitOrderReactor-ExecuteBatchMultipleOutputs.snap

@@ -1 +1 @@
-197889
+197879

.forge-snapshots/Base-LimitOrderReactor-ExecuteBatchMultipleOutputsDifferentTokens.snap

@@ -1 +1 @@
-250617
+250607

.forge-snapshots/Base-LimitOrderReactor-ExecuteBatchNativeOutput.snap

@@ -1 +1 @@
-182582
+182572

.forge-snapshots/Base-LimitOrderReactor-ExecuteSingle.snap

@@ -1 +1 @@
-144333
+144328

.forge-snapshots/Base-LimitOrderReactor-ExecuteSingleNativeOutput.snap

@@ -1 +1 @@
-129895
+129890

.forge-snapshots/Base-LimitOrderReactor-ExecuteSingleValidation.snap

@@ -1 +1 @@
-133743
+153638

.forge-snapshots/EthOutputTest3OrdersWithEthAndERC20Outputs.snap

@@ -1 +1 @@
-363478
+363463

.forge-snapshots/EthOutputTestEthOutput.snap

@@ -1 +1 @@
-156223
+156218

.forge-snapshots/ProtocolFeesGasComparisonTest-InterfaceAndProtocolFee.snap

@@ -1 +1 @@
-176915
+176910

.forge-snapshots/ProtocolFeesGasComparisonTest-InterfaceAndProtocolFeeEthOutput.snap

@@ -1 +1 @@
-162907
+162902

.forge-snapshots/ProtocolFeesGasComparisonTest-InterfaceFee.snap

@@ -1 +1 @@
-166046
+166041

.forge-snapshots/ProtocolFeesGasComparisonTest-InterfaceFeeEthOutput.snap

@@ -1 +1 @@
-146873
+146868

.forge-snapshots/ProtocolFeesGasComparisonTest-NoFees.snap

@@ -1 +1 @@
-149267
+149262

.forge-snapshots/ProtocolFeesGasComparisonTest-NoFeesEthOutput.snap

@@ -1 +1 @@
-124929
+124924

.forge-snapshots/SwapRouter02ExecutorExecute.snap

@@ -1 +1 @@
-262973
+262968

.forge-snapshots/SwapRouter02ExecutorExecuteAlreadyApproved.snap

@@ -1 +1 @@
-118092
+118087

.forge-snapshots/testExclusiveFillerSucceeds.snap

@@ -1 +1 @@
-174066
+174061

src/base/ProtocolFees.sol

@@ -52,7 +52,7 @@ abstract contract ProtocolFees is Owned {
         );
 
         unchecked {
-            for (uint256 i = 0; i < outputsLength; i++) {
+            for (uint256 i = 0; i < outputsLength; ++i) {
                 newOutputs[i] = order.outputs[i];
             }
         }
@@ -93,7 +93,7 @@ abstract contract ProtocolFees is Owned {
             newOutputs[outputsLength + i] = feeOutput;
 
             unchecked {
-                i++;
+                ++i;
             }
         }
 

src/lib/DutchDecayLib.sol

@@ -78,7 +78,7 @@ library DutchDecayLib {
         uint256 outputLength = outputs.length;
         result = new OutputToken[](outputLength);
         unchecked {
-            for (uint256 i = 0; i < outputLength; i++) {
+            for (uint256 i = 0; i < outputLength; ++i) {
                 result[i] = decay(outputs[i], decayStartTime, decayEndTime);
             }
         }

src/lib/DutchOrderLib.sol

@@ -84,7 +84,7 @@ library DutchOrderLib {
         unchecked {
             bytes memory packedHashes = new bytes(32 * outputs.length);
 
-            for (uint256 i = 0; i < outputs.length; i++) {
+            for (uint256 i = 0; i < outputs.length; ++i) {
                 bytes32 outputHash = hash(outputs[i]);
                 assembly {
                     mstore(add(add(packedHashes, 0x20), mul(i, 0x20)), outputHash)

src/lib/ExclusivityOverrideLib.sol

@@ -45,7 +45,7 @@ library ExclusivityOverrideLib {
             output.amount = output.amount.mulDivDown(BPS + exclusivityOverrideBps, BPS);
 
             unchecked {
-                i++;
+                ++i;
             }
         }
     }

src/reactors/BaseReactor.sol

@@ -61,7 +61,7 @@ abstract contract BaseReactor is IReactor, ReactorEvents, ProtocolFees, Reentran
         ResolvedOrder[] memory resolvedOrders = new ResolvedOrder[](ordersLength);
 
         unchecked {
-            for (uint256 i = 0; i < ordersLength; i++) {
+            for (uint256 i = 0; i < ordersLength; ++i) {
                 resolvedOrders[i] = resolve(orders[i]);
             }
         }
@@ -81,7 +81,7 @@ abstract contract BaseReactor is IReactor, ReactorEvents, ProtocolFees, Reentran
         ResolvedOrder[] memory resolvedOrders = new ResolvedOrder[](ordersLength);
 
         unchecked {
-            for (uint256 i = 0; i < ordersLength; i++) {
+            for (uint256 i = 0; i < ordersLength; ++i) {
                 resolvedOrders[i] = resolve(orders[i]);
             }
         }
@@ -96,7 +96,7 @@ abstract contract BaseReactor is IReactor, ReactorEvents, ProtocolFees, Reentran
     function _prepare(ResolvedOrder[] memory orders) internal {
         uint256 ordersLength = orders.length;
         unchecked {
-            for (uint256 i = 0; i < ordersLength; i++) {
+            for (uint256 i = 0; i < ordersLength; ++i) {
                 ResolvedOrder memory order = orders[i];
                 _injectFees(order);
                 order.validate(msg.sender);
@@ -112,7 +112,7 @@ abstract contract BaseReactor is IReactor, ReactorEvents, ProtocolFees, Reentran
         // attempt to transfer all currencies to all recipients
         unchecked {
             // transfer output tokens to their respective recipients
-            for (uint256 i = 0; i < ordersLength; i++) {
+            for (uint256 i = 0; i < ordersLength; ++i) {
                 ResolvedOrder memory resolvedOrder = orders[i];
                 uint256 outputsLength = resolvedOrder.outputs.length;
                 for (uint256 j = 0; j < outputsLength; j++) {

src/reactors/DutchOrderReactor.sol

@@ -66,7 +66,7 @@ contract DutchOrderReactor is BaseReactor {
 
         if (order.input.startAmount != order.input.endAmount) {
             unchecked {
-                for (uint256 i = 0; i < order.outputs.length; i++) {
+                for (uint256 i = 0; i < order.outputs.length; ++i) {
                     if (order.outputs[i].startAmount != order.outputs[i].endAmount) {
                         revert InputAndOutputDecay();
                     }

src/reactors/ExclusiveDutchOrderReactor.sol

@@ -77,7 +77,7 @@ contract ExclusiveDutchOrderReactor is BaseReactor {
 
         if (order.input.startAmount != order.input.endAmount) {
             unchecked {
-                for (uint256 i = 0; i < order.outputs.length; i++) {
+                for (uint256 i = 0; i < order.outputs.length; ++i) {
                     if (order.outputs[i].startAmount != order.outputs[i].endAmount) {
                         revert InputAndOutputDecay();
                     }

src/sample-executors/SwapRouter02Executor.sol

@@ -72,11 +72,11 @@ contract SwapRouter02Executor is IReactorCallback, Owned {
         ) = abi.decode(callbackData, (address[], address[], bytes[]));
 
         unchecked {
-            for (uint256 i = 0; i < tokensToApproveForSwapRouter02.length; i++) {
+            for (uint256 i = 0; i < tokensToApproveForSwapRouter02.length; ++i) {
                 ERC20(tokensToApproveForSwapRouter02[i]).safeApprove(address(swapRouter02), type(uint256).max);
             }
 
-            for (uint256 i = 0; i < tokensToApproveForReactor.length; i++) {
+            for (uint256 i = 0; i < tokensToApproveForReactor.length; ++i) {
                 ERC20(tokensToApproveForReactor[i]).safeApprove(address(reactor), type(uint256).max);
             }
         }
@@ -94,7 +94,7 @@ contract SwapRouter02Executor is IReactorCallback, Owned {
     /// @param tokensToApprove Max approve these tokens to swapRouter02
     /// @param multicallData Pass into swapRouter02.multicall()
     function multicall(ERC20[] calldata tokensToApprove, bytes[] calldata multicallData) external onlyOwner {
-        for (uint256 i = 0; i < tokensToApprove.length; i++) {
+        for (uint256 i = 0; i < tokensToApprove.length; ++i) {
             tokensToApprove[i].safeApprove(address(swapRouter02), type(uint256).max);
         }
         swapRouter02.multicall(type(uint256).max, multicallData);

test/base/BaseReactor.t.sol

@@ -79,7 +79,7 @@ abstract contract BaseReactorTest is GasSnapshot, ReactorEvents, Test, DeployPer
     {
         signedOrders = new SignedOrder[](requests.length);
         orderHashes = new bytes32[](requests.length);
-        for (uint256 i = 0; i < requests.length; i++) {
+        for (uint256 i = 0; i < requests.length; ++i) {
             (SignedOrder memory signed, bytes32 hash) = createAndSignOrder(requests[i]);
             signedOrders[i] = signed;
             orderHashes[i] = hash;

test/base/ProtocolFees.t.sol

@@ -103,7 +103,7 @@ contract ProtocolFeesTest is Test {
         vm.assume(feeBps <= 5);
         vm.assume(outputAmounts.length > 0);
         OutputToken[] memory outputs = new OutputToken[](outputAmounts.length);
-        for (uint256 i = 0; i < outputAmounts.length; i++) {
+        for (uint256 i = 0; i < outputAmounts.length; ++i) {
             outputs[i] = OutputToken(address(tokenOut), outputAmounts[i], RECIPIENT);
         }
         ResolvedOrder memory order = ResolvedOrder({
@@ -118,7 +118,7 @@ contract ProtocolFeesTest is Test {
         ResolvedOrder memory afterFees = fees.takeFees(order);
         assertGe(afterFees.outputs.length, outputs.length);
 
-        for (uint256 i = 0; i < outputAmounts.length; i++) {
+        for (uint256 i = 0; i < outputAmounts.length; ++i) {
             address tokenAddress = order.outputs[i].token;
             uint256 baseAmount = order.outputs[i].amount;
 

test/lib/ExclusivityOverrideLib.t.sol

@@ -134,7 +134,7 @@ contract ExclusivityOverrideLibTest is Test {
         vm.assume(exclusive != address(0));
         vm.assume(overrideAmt < 10000 && overrideAmt > 0);
         uint256[] memory amounts = new uint256[](fuzzAmounts.length);
-        for (uint256 i = 0; i < fuzzAmounts.length; i++) {
+        for (uint256 i = 0; i < fuzzAmounts.length; ++i) {
             amounts[i] = fuzzAmounts[i];
         }
 
@@ -143,7 +143,7 @@ contract ExclusivityOverrideLibTest is Test {
         vm.prank(caller);
         ResolvedOrder memory handled = exclusivity.handleOverride(order, exclusive, block.timestamp + 1, overrideAmt);
         // assert overrideAmt applied
-        for (uint256 i = 0; i < amounts.length; i++) {
+        for (uint256 i = 0; i < amounts.length; ++i) {
             assertEq(handled.outputs[i].amount, amounts[i] * (10000 + overrideAmt) / 10000);
             assertEq(handled.outputs[i].recipient, recipient);
         }

test/reactors/DutchOrderReactor.t.sol

@@ -413,7 +413,7 @@ contract DutchOrderReactorExecuteTest is PermitSignature, DeployPermit2, BaseRea
         returns (SignedOrder memory signedOrder, bytes32 orderHash)
     {
         DutchOutput[] memory outputs = new DutchOutput[](request.outputs.length);
-        for (uint256 i = 0; i < request.outputs.length; i++) {
+        for (uint256 i = 0; i < request.outputs.length; ++i) {
             OutputToken memory output = request.outputs[i];
             outputs[i] = DutchOutput({
                 token: output.token,
@@ -611,7 +611,7 @@ contract DutchOrderReactorExecuteTest is PermitSignature, DeployPermit2, BaseRea
 
     function generateSignedOrders(DutchOrder[] memory orders) private view returns (SignedOrder[] memory result) {
         result = new SignedOrder[](orders.length);
-        for (uint256 i = 0; i < orders.length; i++) {
+        for (uint256 i = 0; i < orders.length; ++i) {
             bytes memory sig = signOrder(swapperPrivateKey, address(permit2), orders[i]);
             result[i] = SignedOrder(abi.encode(orders[i]), sig);
         }

test/reactors/ExclusiveDutchOrderReactor.t.sol

@@ -45,7 +45,7 @@ contract ExclusiveDutchOrderReactorExecuteTest is PermitSignature, DeployPermit2
         returns (SignedOrder memory signedOrder, bytes32 orderHash)
     {
         DutchOutput[] memory outputs = new DutchOutput[](request.outputs.length);
-        for (uint256 i = 0; i < request.outputs.length; i++) {
+        for (uint256 i = 0; i < request.outputs.length; ++i) {
             OutputToken memory output = request.outputs[i];
             outputs[i] = DutchOutput({
                 token: output.token,
@@ -339,13 +339,13 @@ contract ExclusiveDutchOrderReactorExecuteTest is PermitSignature, DeployPermit2
         tokenIn.mint(address(swapper), amountIn);
         tokenIn.forceApprove(swapper, address(permit2), type(uint256).max);
         uint256 amountOutSum = 0;
-        for (uint256 i = 0; i < amountOuts.length; i++) {
+        for (uint256 i = 0; i < amountOuts.length; ++i) {
             amountOutSum += amountOuts[i] * (10000 + overrideAmt) / 10000;
         }
         tokenOut.mint(address(fillContract), uint256(amountOutSum));
 
         uint256[] memory amounts = new uint256[](amountOuts.length);
-        for (uint256 i = 0; i < amountOuts.length; i++) {
+        for (uint256 i = 0; i < amountOuts.length; ++i) {
             amounts[i] = amountOuts[i];
         }
 
@@ -377,7 +377,7 @@ contract ExclusiveDutchOrderReactorExecuteTest is PermitSignature, DeployPermit2
         returns (SignedOrder[] memory result)
     {
         result = new SignedOrder[](orders.length);
-        for (uint256 i = 0; i < orders.length; i++) {
+        for (uint256 i = 0; i < orders.length; ++i) {
             bytes memory sig = signOrder(swapperPrivateKey, address(permit2), orders[i]);
             result[i] = SignedOrder(abi.encode(orders[i]), sig);
         }