perf(anvil): avoid unnecessary transaction vector cloning in Block::new (#12393)

* Update block.rs

* chore: impl encodeable2718

---------

Co-authored-by: Matthias Seitz <[email protected]>

Author: CreeptoGengar

crates/anvil/core/src/eth/block.rs

@@ -32,9 +32,7 @@ impl Block {
         T: Into<Transaction>,
     {
         let transactions: Vec<_> = transactions.into_iter().map(Into::into).collect();
-        let transactions1: Vec<super::transaction::TypedTransaction> =
-            transactions.clone().into_iter().map(Into::into).collect();
-        let transactions_root = calculate_transaction_root(&transactions1);
+        let transactions_root = calculate_transaction_root(&transactions);
 
         Self {
             header: Header {

crates/anvil/core/src/eth/transaction/mod.rs

@@ -206,6 +206,12 @@ pub struct MaybeImpersonatedTransaction {
     pub impersonated_sender: Option<Address>,
 }
 
+impl Typed2718 for MaybeImpersonatedTransaction {
+    fn ty(&self) -> u8 {
+        self.transaction.ty()
+    }
+}
+
 impl MaybeImpersonatedTransaction {
     /// Creates a new wrapper for the given transaction
     pub fn new(transaction: TypedTransaction) -> Self {
@@ -279,6 +285,16 @@ impl MaybeImpersonatedTransaction {
     }
 }
 
+impl Encodable2718 for MaybeImpersonatedTransaction {
+    fn encode_2718_len(&self) -> usize {
+        self.transaction.encode_2718_len()
+    }
+
+    fn encode_2718(&self, out: &mut dyn BufMut) {
+        self.transaction.encode_2718(out)
+    }
+}
+
 impl Encodable for MaybeImpersonatedTransaction {
     fn encode(&self, out: &mut dyn bytes::BufMut) {
         self.transaction.encode(out)