Skip to content

Commit

Permalink
Review: fix up batch graph comments
Browse files Browse the repository at this point in the history
  • Loading branch information
Mirko-von-Leipzig committed Dec 6, 2024
1 parent 2d3d60d commit d504085
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion crates/block-producer/src/mempool/batch_graph.rs
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ impl BatchGraph {
///
/// # Returns
///
/// Returns all removes batches and their transactions.
/// Returns all removed batches and their transactions.
///
/// # Errors
///
Expand Down Expand Up @@ -185,6 +185,8 @@ impl BatchGraph {
&mut self,
batch_ids: BTreeSet<BatchJobId>,
) -> Result<Vec<TransactionId>, GraphError<BatchJobId>> {
// This clone could be elided by moving this call to the end. This would lose the atomic
// property of this method though its unclear what value (if any) that has.
self.inner.prune_processed(batch_ids.clone())?;
let mut transactions = Vec::new();

Expand Down

0 comments on commit d504085

Please sign in to comment.