remove unused args

Author: danyalprout

crates/builder/op-rbuilder/src/builders/builder_tx.rs

@@ -145,23 +145,18 @@ pub trait BuilderTransactions<ExtraCtx: Debug + Default = (), Extra: Debug + Def
     // changes to the db so call new_simulation_state to simulate on a new copy of the state
     fn simulate_builder_txs(
         &self,
-        state_provider: impl StateProvider + Clone,
-        info: &mut ExecutionInfo<Extra>,
         ctx: &OpPayloadBuilderCtx<ExtraCtx>,
         db: &mut State<impl Database + DatabaseRef>,
-        top_of_block: bool,
     ) -> Result<Vec<BuilderTransactionCtx>, BuilderTransactionError>;
 
     fn simulate_builder_txs_with_state_copy(
         &self,
         state_provider: impl StateProvider + Clone,
-        info: &mut ExecutionInfo<Extra>,
         ctx: &OpPayloadBuilderCtx<ExtraCtx>,
         db: &State<impl Database>,
-        top_of_block: bool,
     ) -> Result<Vec<BuilderTransactionCtx>, BuilderTransactionError> {
-        let mut simulation_state = self.new_simulation_state(state_provider.clone(), db);
-        self.simulate_builder_txs(state_provider, info, ctx, &mut simulation_state, top_of_block)
+        let mut simulation_state = self.new_simulation_state(state_provider, db);
+        self.simulate_builder_txs(ctx, &mut simulation_state)
     }
 
     fn add_builder_txs(
@@ -173,13 +168,8 @@ pub trait BuilderTransactions<ExtraCtx: Debug + Default = (), Extra: Debug + Def
         top_of_block: bool,
     ) -> Result<Vec<BuilderTransactionCtx>, BuilderTransactionError> {
         {
-            let builder_txs = self.simulate_builder_txs_with_state_copy(
-                state_provider,
-                info,
-                builder_ctx,
-                db,
-                top_of_block,
-            )?;
+            let builder_txs =
+                self.simulate_builder_txs_with_state_copy(state_provider, builder_ctx, db)?;
 
             let mut evm =
                 builder_ctx.evm_config.evm_with_env(&mut *db, builder_ctx.evm_env.clone());

crates/builder/op-rbuilder/src/builders/flashblocks/builder_tx.rs

@@ -8,7 +8,6 @@ use alloy_rpc_types_eth::TransactionInput;
 use alloy_sol_types::{SolCall, SolEvent, sol};
 use op_alloy_rpc_types::OpTransactionRequest;
 use reth_evm::{ConfigureEvm, precompiles::PrecompilesMap};
-use reth_provider::StateProvider;
 use reth_revm::State;
 use revm::{DatabaseRef, inspector::NoOpInspector};
 use tracing::warn;
@@ -22,7 +21,6 @@ use crate::{
         flashblocks::payload::{FlashblocksExecutionInfo, FlashblocksExtraCtx},
         get_nonce,
     },
-    primitives::reth::ExecutionInfo,
     tx_signer::Signer,
 };
 
@@ -70,11 +68,8 @@ impl FlashblocksBuilderTx {
 impl BuilderTransactions<FlashblocksExtraCtx, FlashblocksExecutionInfo> for FlashblocksBuilderTx {
     fn simulate_builder_txs(
         &self,
-        _state_provider: impl StateProvider + Clone,
-        _info: &mut ExecutionInfo<FlashblocksExecutionInfo>,
         ctx: &OpPayloadBuilderCtx<FlashblocksExtraCtx>,
         db: &mut State<impl Database + DatabaseRef>,
-        _top_of_block: bool,
     ) -> Result<Vec<BuilderTransactionCtx>, BuilderTransactionError> {
         let mut builder_txs = Vec::<BuilderTransactionCtx>::new();
 
@@ -166,11 +161,8 @@ impl BuilderTransactions<FlashblocksExtraCtx, FlashblocksExecutionInfo>
 {
     fn simulate_builder_txs(
         &self,
-        _state_provider: impl StateProvider + Clone,
-        _info: &mut ExecutionInfo<FlashblocksExecutionInfo>,
         ctx: &OpPayloadBuilderCtx<FlashblocksExtraCtx>,
         db: &mut State<impl Database + DatabaseRef>,
-        _top_of_block: bool,
     ) -> Result<Vec<BuilderTransactionCtx>, BuilderTransactionError> {
         let mut builder_txs = Vec::<BuilderTransactionCtx>::new();
 

crates/builder/op-rbuilder/src/builders/standard/builder_tx.rs

@@ -1,7 +1,6 @@
 use core::fmt::Debug;
 
 use alloy_evm::Database;
-use reth_provider::StateProvider;
 use reth_revm::State;
 use revm::DatabaseRef;
 
@@ -10,7 +9,6 @@ use crate::{
         BuilderTransactionCtx, BuilderTransactionError, BuilderTransactions,
         builder_tx::BuilderTxBase, context::OpPayloadBuilderCtx,
     },
-    primitives::reth::ExecutionInfo,
     tx_signer::Signer,
 };
 
@@ -30,11 +28,8 @@ impl StandardBuilderTx {
 impl BuilderTransactions for StandardBuilderTx {
     fn simulate_builder_txs(
         &self,
-        _state_provider: impl StateProvider + Clone,
-        _info: &mut ExecutionInfo,
         ctx: &OpPayloadBuilderCtx,
         db: &mut State<impl Database + DatabaseRef>,
-        _top_of_block: bool,
     ) -> Result<Vec<BuilderTransactionCtx>, BuilderTransactionError> {
         let mut builder_txs = Vec::<BuilderTransactionCtx>::new();
         let standard_builder_tx = self.base_builder_tx.simulate_builder_tx(ctx, &mut *db)?;