refactor: derive Clone for ExecutionResult

crates/primitives/src/result.rs

@@ -1,3 +1,5 @@
+use derive_where::derive_where;
+
 use crate::{Address, Bytes, ChainSpec, EvmState, Log, TransactionValidation, U256};
 use core::fmt::{self, Debug};
 use std::{boxed::Box, string::String, vec::Vec};
@@ -25,6 +27,7 @@ pub struct ResultAndState<ChainSpecT: ChainSpec> {
 
 /// Result of a transaction execution.
 #[derive(Debug, PartialEq, Eq, Hash)]
+#[derive_where(Clone; ChainSpecT::HaltReason)]
 pub enum ExecutionResult<ChainSpecT: ChainSpec> {
     /// Returned successfully
     Success {
@@ -44,34 +47,6 @@ pub enum ExecutionResult<ChainSpecT: ChainSpec> {
     },
 }
 
-impl<ChainSpecT: ChainSpec> Clone for ExecutionResult<ChainSpecT> {
-    fn clone(&self) -> Self {
-        match self {
-            Self::Success {
-                reason,
-                gas_used,
-                gas_refunded,
-                logs,
-                output,
-            } => Self::Success {
-                reason: *reason,
-                gas_used: *gas_used,
-                gas_refunded: *gas_refunded,
-                logs: logs.clone(),
-                output: output.clone(),
-            },
-            Self::Revert { gas_used, output } => Self::Revert {
-                gas_used: *gas_used,
-                output: output.clone(),
-            },
-            Self::Halt { reason, gas_used } => Self::Halt {
-                reason: reason.clone(),
-                gas_used: *gas_used,
-            },
-        }
-    }
-}
-
 impl<ChainSpecT: ChainSpec> ExecutionResult<ChainSpecT> {
     /// Returns if transaction execution is successful.
     /// 1 indicates success, 0 indicates revert.