refactor: change try from casm contract class to be by ref to avoid clones

crates/blockifier/src/execution/contract_class.rs

@@ -62,10 +62,10 @@ pub enum ContractClass {
     V1(ContractClassV1),
 }
 
-impl TryFrom<CasmContractClass> for ContractClass {
+impl TryFrom<&CasmContractClass> for ContractClass {
     type Error = ProgramError;
 
-    fn try_from(contract_class: CasmContractClass) -> Result<Self, Self::Error> {
+    fn try_from(contract_class: &CasmContractClass) -> Result<Self, Self::Error> {
         Ok(ContractClass::V1(contract_class.try_into()?))
     }
 }
@@ -274,7 +274,7 @@ impl ContractClassV1 {
 
     pub fn try_from_json_string(raw_contract_class: &str) -> Result<ContractClassV1, ProgramError> {
         let casm_contract_class: CasmContractClass = serde_json::from_str(raw_contract_class)?;
-        let contract_class: ContractClassV1 = casm_contract_class.try_into()?;
+        let contract_class: ContractClassV1 = (&casm_contract_class).try_into()?;
 
         Ok(contract_class)
     }
@@ -405,15 +405,12 @@ impl EntryPointV1 {
     }
 }
 
-impl TryFrom<CasmContractClass> for ContractClassV1 {
+impl TryFrom<&CasmContractClass> for ContractClassV1 {
     type Error = ProgramError;
 
-    fn try_from(class: CasmContractClass) -> Result<Self, Self::Error> {
-        let data: Vec<MaybeRelocatable> = class
-            .bytecode
-            .into_iter()
-            .map(|x| MaybeRelocatable::from(Felt::from(x.value)))
-            .collect();
+    fn try_from(class: &CasmContractClass) -> Result<Self, Self::Error> {
+        let data: Vec<MaybeRelocatable> =
+            class.bytecode.iter().map(|x| MaybeRelocatable::from(Felt::from(&x.value))).collect();
 
         let mut hints: HashMap<usize, Vec<HintParams>> = HashMap::new();
         for (i, hint_list) in class.hints.iter() {
@@ -452,19 +449,20 @@ impl TryFrom<CasmContractClass> for ContractClassV1 {
         let mut entry_points_by_type = HashMap::new();
         entry_points_by_type.insert(
             EntryPointType::Constructor,
-            convert_entry_points_v1(class.entry_points_by_type.constructor),
+            convert_entry_points_v1(&class.entry_points_by_type.constructor),
         );
         entry_points_by_type.insert(
             EntryPointType::External,
-            convert_entry_points_v1(class.entry_points_by_type.external),
+            convert_entry_points_v1(&class.entry_points_by_type.external),
         );
         entry_points_by_type.insert(
             EntryPointType::L1Handler,
-            convert_entry_points_v1(class.entry_points_by_type.l1_handler),
+            convert_entry_points_v1(&class.entry_points_by_type.l1_handler),
         );
 
         let bytecode_segment_lengths = class
             .bytecode_segment_lengths
+            .clone()
             .unwrap_or_else(|| NestedIntList::Leaf(program.data_len()));
         let compiler_version = CompilerVersion(
             Version::parse(&class.compiler_version)
@@ -505,16 +503,16 @@ fn hint_to_hint_params(hint: &cairo_lang_casm::hints::Hint) -> Result<HintParams
     })
 }
 
-fn convert_entry_points_v1(external: Vec<CasmContractEntryPoint>) -> Vec<EntryPointV1> {
+fn convert_entry_points_v1(external: &[CasmContractEntryPoint]) -> Vec<EntryPointV1> {
     external
-        .into_iter()
+        .iter()
         .map(|ep| EntryPointV1 {
-            selector: EntryPointSelector(Felt::from(ep.selector)),
+            selector: EntryPointSelector(Felt::from(&ep.selector)),
             offset: EntryPointOffset(ep.offset),
             builtins: ep
                 .builtins
-                .into_iter()
-                .map(|builtin| BuiltinName::from_str(&builtin).expect("Unrecognized builtin."))
+                .iter()
+                .map(|builtin| BuiltinName::from_str(builtin).expect("Unrecognized builtin."))
                 .collect(),
         })
         .collect()
@@ -528,10 +526,10 @@ pub struct ClassInfo {
     abi_length: usize,
 }
 
-impl TryFrom<starknet_api::contract_class::ClassInfo> for ClassInfo {
+impl TryFrom<&starknet_api::contract_class::ClassInfo> for ClassInfo {
     type Error = ProgramError;
 
-    fn try_from(class_info: starknet_api::contract_class::ClassInfo) -> Result<Self, Self::Error> {
+    fn try_from(class_info: &starknet_api::contract_class::ClassInfo) -> Result<Self, Self::Error> {
         let starknet_api::contract_class::ClassInfo {
             casm_contract_class,
             sierra_program_length,
@@ -540,8 +538,8 @@ impl TryFrom<starknet_api::contract_class::ClassInfo> for ClassInfo {
 
         Ok(Self {
             contract_class: casm_contract_class.try_into()?,
-            sierra_program_length,
-            abi_length,
+            sierra_program_length: *sierra_program_length,
+            abi_length: *abi_length,
         })
     }
 }

crates/blockifier/src/transaction/account_transaction.rs

@@ -77,24 +77,24 @@ pub enum AccountTransaction {
     Invoke(InvokeTransaction),
 }
 
-impl TryFrom<starknet_api::executable_transaction::Transaction> for AccountTransaction {
+impl TryFrom<&starknet_api::executable_transaction::Transaction> for AccountTransaction {
     type Error = TransactionExecutionError;
 
     fn try_from(
-        executable_transaction: starknet_api::executable_transaction::Transaction,
+        executable_transaction: &starknet_api::executable_transaction::Transaction,
     ) -> Result<Self, Self::Error> {
         match executable_transaction {
             starknet_api::executable_transaction::Transaction::Declare(declare_tx) => {
                 Ok(Self::Declare(declare_tx.try_into()?))
             }
             starknet_api::executable_transaction::Transaction::DeployAccount(deploy_account_tx) => {
                 Ok(Self::DeployAccount(DeployAccountTransaction {
-                    tx: deploy_account_tx,
+                    tx: deploy_account_tx.clone(),
                     only_query: false,
                 }))
             }
             starknet_api::executable_transaction::Transaction::Invoke(invoke_tx) => {
-                Ok(Self::Invoke(InvokeTransaction { tx: invoke_tx, only_query: false }))
+                Ok(Self::Invoke(InvokeTransaction { tx: invoke_tx.clone(), only_query: false }))
             }
         }
     }

crates/blockifier/src/transaction/transactions.rs

@@ -133,11 +133,11 @@ pub struct DeclareTransaction {
     pub class_info: ClassInfo,
 }
 
-impl TryFrom<starknet_api::executable_transaction::DeclareTransaction> for DeclareTransaction {
+impl TryFrom<&starknet_api::executable_transaction::DeclareTransaction> for DeclareTransaction {
     type Error = TransactionExecutionError;
 
     fn try_from(
-        declare_tx: starknet_api::executable_transaction::DeclareTransaction,
+        declare_tx: &starknet_api::executable_transaction::DeclareTransaction,
     ) -> Result<Self, Self::Error> {
         Self::new_from_executable_tx(declare_tx, false)
     }
@@ -191,14 +191,14 @@ impl DeclareTransaction {
     }
 
     fn new_from_executable_tx(
-        declare_tx: starknet_api::executable_transaction::DeclareTransaction,
+        declare_tx: &starknet_api::executable_transaction::DeclareTransaction,
         only_query: bool,
     ) -> Result<Self, TransactionExecutionError> {
         let starknet_api::executable_transaction::DeclareTransaction { tx, tx_hash, class_info } =
             declare_tx;
         let class_info: ClassInfo = class_info.try_into()?;
 
-        Self::create(tx, tx_hash, class_info, only_query)
+        Self::create(tx.clone(), *tx_hash, class_info, only_query)
     }
 
     implement_inner_tx_getter_calls!(

crates/gateway/src/rpc_state_reader.rs

@@ -135,7 +135,7 @@ impl BlockifierStateReader for RpcStateReader {
             serde_json::from_value(result).map_err(serde_err_to_state_err)?;
         match contract_class {
             CompiledContractClass::V1(contract_class_v1) => Ok(ContractClass::V1(
-                ContractClassV1::try_from(contract_class_v1).map_err(StateError::ProgramError)?,
+                ContractClassV1::try_from(&contract_class_v1).map_err(StateError::ProgramError)?,
             )),
             CompiledContractClass::V0(contract_class_v0) => Ok(ContractClass::V0(
                 ContractClassV0::try_from(contract_class_v0).map_err(StateError::ProgramError)?,

crates/gateway/src/rpc_state_reader_test.rs

@@ -178,7 +178,7 @@ async fn test_get_compiled_contract_class() {
             .await
             .unwrap()
             .unwrap();
-    assert_eq!(result, ContractClass::V1(expected_result.try_into().unwrap()));
+    assert_eq!(result, ContractClass::V1((&expected_result).try_into().unwrap()));
     mock.assert_async().await;
 }
 

crates/gateway/src/stateful_transaction_validator.rs

@@ -75,11 +75,7 @@ impl StatefulTransactionValidator {
         mut validator: V,
     ) -> StatefulTransactionValidatorResult<()> {
         let skip_validate = skip_stateful_validations(executable_tx, account_nonce);
-        let account_tx = AccountTransaction::try_from(
-            // TODO(Arni): create a try_from for &ExecutableTransaction.
-            executable_tx.clone(),
-        )
-        .map_err(|error| {
+        let account_tx = AccountTransaction::try_from(executable_tx).map_err(|error| {
             error!("Failed to convert executable transaction into account transaction: {}", error);
             GatewaySpecError::UnexpectedError { data: "Internal server error".to_owned() }
         })?;

crates/native_blockifier/src/py_block_executor_test.rs

@@ -20,7 +20,7 @@ use crate::test_utils::MockStorage;
 fn global_contract_cache_update() {
     // Initialize executor and set a contract class on the state.
     let casm = CasmContractClass { compiler_version: "0.1.0".to_string(), ..Default::default() };
-    let contract_class = ContractClass::V1(ContractClassV1::try_from(casm.clone()).unwrap());
+    let contract_class = ContractClass::V1(ContractClassV1::try_from(&casm).unwrap());
     let class_hash = class_hash!("0x1");
 
     let temp_storage_path = tempfile::tempdir().unwrap().into_path();

crates/native_blockifier/src/state_readers/papyrus_state.rs

@@ -63,7 +63,7 @@ impl PapyrusReader {
                      inconsistent.",
                 );
 
-            return Ok(ContractClass::V1(ContractClassV1::try_from(casm_contract_class)?));
+            return Ok(ContractClass::V1(ContractClassV1::try_from(&casm_contract_class)?));
         }
 
         let v0_contract_class = self

crates/papyrus_execution/src/execution_utils.rs

@@ -67,7 +67,7 @@ pub(crate) fn get_contract_class(
                 return Err(ExecutionUtilsError::CasmTableNotSynced);
             };
             return Ok(Some(BlockifierContractClass::V1(
-                ContractClassV1::try_from(casm).map_err(ExecutionUtilsError::ProgramError)?,
+                ContractClassV1::try_from(&casm).map_err(ExecutionUtilsError::ProgramError)?,
             )));
         }
         None => {}

crates/papyrus_execution/src/lib.rs

@@ -796,7 +796,7 @@ fn to_blockifier_tx(
             only_query,
         ) => {
             let class_v1 = BlockifierContractClass::V1(
-                compiled_class.try_into().map_err(BlockifierError::new)?,
+                (&compiled_class).try_into().map_err(BlockifierError::new)?,
             );
             let class_info =
                 ClassInfo::new(&class_v1, sierra_program_length, abi_length).map_err(|err| {
@@ -823,7 +823,7 @@ fn to_blockifier_tx(
             only_query,
         ) => {
             let class_v1 = BlockifierContractClass::V1(
-                compiled_class.try_into().map_err(BlockifierError::new)?,
+                (&compiled_class).try_into().map_err(BlockifierError::new)?,
             );
             let class_info =
                 ClassInfo::new(&class_v1, sierra_program_length, abi_length).map_err(|err| {

crates/papyrus_execution/src/state_reader.rs

@@ -85,7 +85,7 @@ impl BlockifierStateReader for ExecutionStateReader {
             .and_then(|pending_data| pending_data.classes.get_compiled_class(class_hash))
         {
             return Ok(BlockifierContractClass::V1(
-                ContractClassV1::try_from(pending_casm).map_err(StateError::ProgramError)?,
+                ContractClassV1::try_from(&pending_casm).map_err(StateError::ProgramError)?,
             ));
         }
         if let Some(ApiContractClass::DeprecatedContractClass(pending_deprecated_class)) = self

crates/papyrus_execution/src/state_reader_test.rs

@@ -49,8 +49,7 @@ fn read_state() {
     // The class is not used in the execution, so it can be default.
     let class0 = ContractClass::default();
     let casm0 = get_test_casm();
-    let blockifier_casm0 =
-        BlockifierContractClass::V1(ContractClassV1::try_from(casm0.clone()).unwrap());
+    let blockifier_casm0 = BlockifierContractClass::V1(ContractClassV1::try_from(&casm0).unwrap());
     let compiled_class_hash0 = CompiledClassHash(StarkHash::default());
 
     let class_hash1 = ClassHash(1u128.into());
@@ -64,8 +63,7 @@ fn read_state() {
     let compiled_class_hash2 = CompiledClassHash(StarkHash::TWO);
     let mut casm1 = get_test_casm();
     casm1.bytecode[0] = BigUintAsHex { value: 12345u32.into() };
-    let blockifier_casm1 =
-        BlockifierContractClass::V1(ContractClassV1::try_from(casm1.clone()).unwrap());
+    let blockifier_casm1 = BlockifierContractClass::V1(ContractClassV1::try_from(&casm1).unwrap());
     let nonce1 = Nonce(felt!(2_u128));
     let class_hash3 = ClassHash(567_u128.into());
     let class_hash4 = ClassHash(89_u128.into());