cache msg sender

Uniswap · Dec 14, 2023 · 3cf69de · 3cf69de
1 parent df5f006
commit 3cf69de
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/.forge-snapshots/poolManager bytecode size.snap b/.forge-snapshots/poolManager bytecode size.snap
@@ -1 +1 @@
-26357
+26366
diff --git a/.forge-snapshots/swap burn 6909 for input.snap b/.forge-snapshots/swap burn 6909 for input.snap
@@ -1 +1 @@
-129161
+129176
diff --git a/src/PoolManager.sol b/src/PoolManager.sol
@@ -349,10 +349,11 @@ contract PoolManager is IPoolManager, Fees, NoDelegateCall, Claims {
     function burn(Currency currency, address from, uint256 amount) external override noDelegateCall onlyByLocker {
         _accountDelta(currency, -(amount.toInt128()));
         uint256 id = currency.toId();
-        if (from != msg.sender && !isOperator[from][msg.sender]) {
-            uint256 senderAllowance = allowance[from][msg.sender][id];
+        address sender = msg.sender;
+        if (from != sender && !isOperator[from][sender]) {
+            uint256 senderAllowance = allowance[from][sender][id];
             if (senderAllowance != type(uint256).max) {
-                allowance[from][msg.sender][id] = senderAllowance - amount;
+                allowance[from][sender][id] = senderAllowance - amount;
             }
         }
         _burn(from, id, amount);