Merge pull request #2472 from byod8/mainnet-staging

zosorock · web-flow · commit 714e9c3961d6 · 2024-11-14T12:47:25.000-05:00
[Chore] Update documentation in `atomic_speculate`
diff --git a/synthesizer/src/vm/finalize.rs b/synthesizer/src/vm/finalize.rs
@@ -426,7 +426,10 @@ impl<N: Network, C: ConsensusStorage<N>> VM<N, C> {
                                         }
                                     }
                                 }
+
                                 // This is a foundational bug - the caller is violating protocol rules.
+                                // It is possible that a `credits.aleo/split` transaction has no fee. However, it
+                                // is a simple transition without finalize operations and should not fail here.
                                 // Note: This will abort the entire atomic batch.
                                 None => Err("Rejected execute transaction has no fee".to_string()),
                             },

Original file line number	Diff line number	Diff line change
`@@ -426,7 +426,10 @@ impl<N: Network, C: ConsensusStorage<N>> VM<N, C> {`
`426`	`426`	`}`
`427`	`427`	`}`
`428`	`428`	`}`
	`429`	`+`
`429`	`430`	`// This is a foundational bug - the caller is violating protocol rules.`
	`431`	+ // It is possible that a `credits.aleo/split` transaction has no fee. However, it
	`432`	`+ // is a simple transition without finalize operations and should not fail here.`
`430`	`433`	`// Note: This will abort the entire atomic batch.`
`431`	`434`	`None => Err("Rejected execute transaction has no fee".to_string()),`
`432`	`435`	`},`