fix tx_convert in witness (#261)

* fix tx_convert in witness * simplify code
privacy-scaling-explorations · Jan 5, 2022 · 0c9cf1c · 0c9cf1c
1 parent 962bb76
commit 0c9cf1c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/bus-mapping/src/eth_types.rs b/bus-mapping/src/eth_types.rs
@@ -330,7 +330,7 @@ pub struct GethExecTrace {
 
 /// Truncate the memory in each step to the memory size before the step is
 /// executed (and before the memory is expanded).  This is required because geth
-/// sets the memory in each step as the memroy before execution but after
+/// sets the memory in each step as the memory before execution but after
 /// expansion.
 pub fn fix_geth_trace_memory_size(trace: &mut [GethExecStep]) {
     let mut mem_sizes = vec![0; trace.len()];

diff --git a/zkevm-circuits/src/evm_circuit/witness.rs b/zkevm-circuits/src/evm_circuit/witness.rs
@@ -404,7 +404,7 @@ fn tx_convert(
     tx: &bus_mapping::circuit_input_builder::Transaction,
     ops_len: (usize, usize, usize),
 ) -> Transaction<Fp> {
-    let mut result: Transaction<Fp> = Transaction::<Fp> {
+    Transaction::<Fp> {
         calls: vec![Call {
             id: 1,
             is_root: true,
@@ -414,12 +414,13 @@ fn tx_convert(
                 randomness,
             ),
         }],
+        steps: tx
+            .steps()
+            .iter()
+            .map(|step| step_convert(step, ops_len))
+            .collect(),
         ..Default::default()
-    };
-    let _ = tx.steps().iter().map(|step| {
-        result.steps.push(step_convert(step, ops_len));
-    });
-    result
+    }
 }
 
 pub fn block_convert(