copy

coinbase · Mar 14, 2024 · dfac046 · dfac046
1 parent 80bc421
commit dfac046
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/.gas-snapshot b/.gas-snapshot
@@ -1,11 +1,11 @@
-PostOpTest:test_RevertsIfPostOpFailed(uint256,uint256,uint256) (runs: 256, μ: 95004, ~: 97095)
+PostOpTest:test_RevertsIfPostOpFailed(uint256,uint256,uint256) (runs: 256, μ: 94902, ~: 97095)
 PostOpTest:test_entryPointAddStake(uint112,uint32) (runs: 256, μ: 56233, ~: 56233)
-PostOpTest:test_entryPointDeposit(uint112) (runs: 256, μ: 32516, ~: 32962)
+PostOpTest:test_entryPointDeposit(uint112) (runs: 256, μ: 32405, ~: 32962)
 PostOpTest:test_entryPointUnlockStake() (gas: 54910)
 PostOpTest:test_entryPointWithdraw(uint112) (runs: 256, μ: 63200, ~: 65300)
 PostOpTest:test_entryPointWithdrawStake() (gas: 71825)
 PostOpTest:test_paymasterPaysForOp() (gas: 208402)
-PostOpTest:test_transfersExcess(uint256,uint256,uint256,uint256) (runs: 256, μ: 110196, ~: 111227)
+PostOpTest:test_transfersExcess(uint256,uint256,uint256,uint256) (runs: 256, μ: 110036, ~: 111224)
 Simulate:test() (gas: 12114)
 ValidatePaymasterUserOpTest:test_emitsCorrectly(address,uint256,uint256) (runs: 256, μ: 102602, ~: 103835)
 ValidatePaymasterUserOpTest:test_recordsNonceUsed(uint256) (runs: 256, μ: 109936, ~: 111025)
@@ -18,9 +18,9 @@ ValidateTest:test_receive() (gas: 14687)
 WithdrawGasExcess:test_RevertsIfNoExcess(uint256) (runs: 256, μ: 68914, ~: 69177)
 WithdrawGasExcess:test_transferExcess(uint256,uint256,uint256) (runs: 256, μ: 102074, ~: 102074)
 WithdrawTest:test_emitsCorrectly(address,uint256,uint256) (runs: 256, μ: 116118, ~: 119073)
-WithdrawTest:test_emitsCorrectlyERC20Withdraw(address,uint256,uint256) (runs: 256, μ: 158167, ~: 160513)
+WithdrawTest:test_emitsCorrectlyERC20Withdraw(address,uint256,uint256) (runs: 256, μ: 158012, ~: 160513)
 WithdrawTest:test_recordsNonceUsed(uint256) (runs: 256, μ: 109078, ~: 110167)
-WithdrawTest:test_revertsIfExpired(uint48,uint256) (runs: 256, μ: 59414, ~: 59535)
+WithdrawTest:test_revertsIfExpired(uint48,uint256) (runs: 256, μ: 59425, ~: 59535)
 WithdrawTest:test_revertsIfNonceUsed() (gas: 93506)
 WithdrawTest:test_revertsIfWrongSignature() (gas: 61536)
 WithdrawTest:test_transfersERC20Successfully(uint256) (runs: 256, μ: 136385, ~: 138825)

diff --git a/src/MagicSpend.sol b/src/MagicSpend.sol
@@ -145,7 +145,7 @@ contract MagicSpend is Ownable, IPaymaster {
         onlyEntryPoint
     {
         // `PostOpMode.postOpReverted` should be impossible.
-        // Only possible cause would be if this contract does not enough ETH to transfer
+        // Only possible cause would be if this contract does not own enough ETH to transfer
         // but this is checked at the validation step.
         assert(mode != PostOpMode.postOpReverted);
 
@@ -228,7 +228,7 @@ contract MagicSpend is Ownable, IPaymaster {
     /// @dev Reverts if not called by the owner of the contract.
     ///
     /// @param amount              The amount to stake in the Entrypoint.
-    /// @param unstakeDelaySeconds XXX
+    /// @param unstakeDelaySeconds The duration for which the stake cannot be withdrawn.
     function entryPointAddStake(uint256 amount, uint32 unstakeDelaySeconds) external payable onlyOwner {
         IEntryPoint(entryPoint()).addStake{value: amount}(unstakeDelaySeconds);
     }