Remove temporary sender variable (gas savings)

Author: mdehoog

src/Paymaster.sol

@@ -40,10 +40,9 @@ contract Paymaster is BasePaymaster {
     function getHash(UserOperation calldata userOp, uint48 validUntil, uint48 validAfter)
     public view returns (bytes32) {
         // can't use userOp.hash(), since it contains also the paymasterAndData itself.
-        address sender = userOp.getSender();
         return keccak256(
             abi.encode(
-                sender,
+                userOp.getSender(),
                 userOp.nonce,
                 keccak256(userOp.initCode),
                 keccak256(userOp.callData),

test/Paymaster.t.sol

@@ -59,7 +59,7 @@ contract PaymasterTest is Test {
         UserOperation memory userOp = createUserOp();
         signUserOp(userOp);
 
-        vm.expectRevert(createEncodedValidationResult(false, 57198));
+        vm.expectRevert(createEncodedValidationResult(false, 57193));
         entrypoint.simulateValidation(userOp);
     }
 
@@ -71,7 +71,7 @@ contract PaymasterTest is Test {
         userOp.paymasterAndData = abi.encodePacked(address(paymaster), abi.encode(MOCK_VALID_UNTIL, MOCK_VALID_AFTER), r, s, v);
         signUserOp(userOp);
 
-        vm.expectRevert(createEncodedValidationResult(false, 55198));
+        vm.expectRevert(createEncodedValidationResult(false, 55193));
         entrypoint.simulateValidation(userOp);
     }
 
@@ -81,7 +81,7 @@ contract PaymasterTest is Test {
         userOp.paymasterAndData = abi.encodePacked(address(paymaster), abi.encode(MOCK_VALID_UNTIL, MOCK_VALID_AFTER), r, s, v);
         signUserOp(userOp);
 
-        vm.expectRevert(createEncodedValidationResult(true, 57204));
+        vm.expectRevert(createEncodedValidationResult(true, 57199));
         entrypoint.simulateValidation(userOp);
     }