From 3d90a211ff1a0b0bb3fcebe103d3d375b636a211 Mon Sep 17 00:00:00 2001 From: Lee Bousfield Date: Wed, 10 May 2023 09:52:43 -0600 Subject: [PATCH] Comment that gasEstimateL1Component doesn't pad its estimate --- src/node-interface/NodeInterface.sol | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/node-interface/NodeInterface.sol b/src/node-interface/NodeInterface.sol index c62972aed1..f74c5a4c8c 100644 --- a/src/node-interface/NodeInterface.sol +++ b/src/node-interface/NodeInterface.sol @@ -98,8 +98,10 @@ interface NodeInterface { /** * @notice Estimates a transaction's l1 costs. * @dev Use eth_call to call. - * This method is exactly like gasEstimateComponents, but doesn't include the l2 component + * This method is similar to gasEstimateComponents, but doesn't include the l2 component * so that the l1 component can be known even when the tx may fail. + * This method also doesn't pad the estimate as gas estimation normally does. + * If using this value to submit a transaction, we'd recommend first padding it by 10%. * @param data the tx's calldata. Everything else like "From" and "Gas" are copied over * @param to the tx's "To" (ignored when contractCreation is true) * @param contractCreation whether "To" is omitted