Remove unused max steps

Author: matias-gonz

cli/src/main.rs

@@ -256,7 +256,6 @@ fn call_parser(
         &mut ExecutionResourcesManager::default(),
         &mut TransactionExecutionContext::default(),
         false,
-        block_context.invoke_tx_max_n_steps(),
     )?;
     Ok(call_info.retdata)
 }

fuzzer/src/main.rs

@@ -190,7 +190,6 @@ fn main() {
                         &mut resources_manager,
                         &mut tx_execution_context,
                         false,
-                        block_context.invoke_tx_max_n_steps()
                     )
                     .unwrap(),
                 expected_call_info

src/execution/execution_entry_point.rs

@@ -2,7 +2,6 @@ use crate::services::api::contract_classes::deprecated_contract_class::{
     ContractEntryPoint, EntryPointType,
 };
 use crate::state::cached_state::CachedState;
-use crate::state::StateDiff;
 use crate::{
     definitions::{block_context::BlockContext, constants::DEFAULT_ENTRY_POINT_SELECTOR},
     runner::StarknetRunner,
@@ -92,7 +91,6 @@ impl ExecutionEntryPoint {
         resources_manager: &mut ExecutionResourcesManager,
         tx_execution_context: &mut TransactionExecutionContext,
         support_reverted: bool,
-        max_steps: u64,
     ) -> Result<CallInfo, TransactionError>
     where
         T: StateReader,
@@ -111,35 +109,15 @@ impl ExecutionEntryPoint {
                 contract_class,
                 class_hash,
             ),
-            CompiledClass::Casm(contract_class) => {
-                let mut tmp_state = CachedState::new(
-                    state.state_reader.clone(),
-                    state.contract_classes.clone(),
-                    state.casm_contract_classes.clone(),
-                );
-                tmp_state.cache = state.cache.clone();
-
-                match self._execute(
-                    &mut tmp_state,
-                    resources_manager,
-                    block_context,
-                    tx_execution_context,
-                    contract_class,
-                    class_hash,
-                    support_reverted,
-                ) {
-                    Ok(call_info) => {
-                        let state_diff = StateDiff::from_cached_state(tmp_state)?;
-                        state.apply_state_update(&state_diff)?;
-                        Ok(call_info)
-                    }
-                    Err(e) => {
-                        let _n_reverted_steps =
-                            (max_steps as usize) - resources_manager.cairo_usage.n_steps;
-                        Err(e)
-                    }
-                }
-            }
+            CompiledClass::Casm(contract_class) => self._execute(
+                state,
+                resources_manager,
+                block_context,
+                tx_execution_context,
+                contract_class,
+                class_hash,
+                support_reverted,
+            ),
         }
     }
 

src/lib.rs

@@ -149,7 +149,6 @@ pub fn call_contract<T: StateReader>(
         &mut ExecutionResourcesManager::default(),
         &mut tx_execution_context,
         false,
-        block_context.invoke_tx_max_n_steps,
     )?;
 
     Ok(call_info.retdata)

src/syscalls/business_logic_syscall_handler.rs

@@ -245,7 +245,6 @@ impl<'a, S: StateReader> BusinessLogicSyscallHandler<'a, S> {
                 &mut self.resources_manager,
                 &mut self.tx_execution_context,
                 self.support_reverted,
-                self.block_context.invoke_tx_max_n_steps,
             )
             .map_err(|err| SyscallHandlerError::ExecutionError(err.to_string()))?;
 
@@ -347,7 +346,6 @@ impl<'a, S: StateReader> BusinessLogicSyscallHandler<'a, S> {
                 &mut self.resources_manager,
                 &mut self.tx_execution_context,
                 self.support_reverted,
-                self.block_context.invoke_tx_max_n_steps,
             )
             .map_err(|_| StateError::ExecutionEntryPoint())?;
 

src/syscalls/deprecated_business_logic_syscall_handler.rs

@@ -236,7 +236,6 @@ impl<'a, S: StateReader> DeprecatedBLSyscallHandler<'a, S> {
                 &mut self.resources_manager,
                 &mut self.tx_execution_context,
                 false,
-                self.block_context.invoke_tx_max_n_steps,
             )
             .map_err(|_| StateError::ExecutionEntryPoint())?;
         Ok(())
@@ -440,7 +439,6 @@ impl<'a, S: StateReader> DeprecatedBLSyscallHandler<'a, S> {
                 &mut self.resources_manager,
                 &mut self.tx_execution_context,
                 false,
-                self.block_context.invoke_tx_max_n_steps,
             )
             .map(|x| {
                 let retdata = x.retdata.clone();

src/testing/erc20.rs

@@ -125,7 +125,6 @@ fn test_erc20_cairo2() {
             &mut resources_manager,
             &mut tx_execution_context,
             false,
-            block_context.invoke_tx_max_n_steps,
         )
         .unwrap();
     let erc20_address = call_info.retdata.get(0).unwrap().clone();

src/testing/state.rs

@@ -153,7 +153,6 @@ impl StarknetState {
             &mut resources_manager,
             &mut tx_execution_context,
             false,
-            self.block_context.invoke_tx_max_n_steps,
         )?;
 
         let exec_info = ExecutionInfo::Call(Box::new(call_info.clone()));

src/transaction/declare.rs

@@ -229,7 +229,6 @@ impl Declare {
             resources_manager,
             &mut self.get_execution_context(block_context.invoke_tx_max_n_steps),
             false,
-            block_context.validate_max_n_steps,
         )?;
 
         verify_no_calls_to_other_contracts(&call_info)

src/transaction/declare_v2.rs

@@ -360,7 +360,6 @@ impl DeclareV2 {
                 resources_manager,
                 &mut tx_execution_context,
                 false,
-                block_context.validate_max_n_steps,
             )?)
         };
         let call_info = verify_no_calls_to_other_contracts(&call_info)?;