Skip to content

Commit

Permalink
Fix off-by-one capacity error when sizing target containers (#450)
Browse files Browse the repository at this point in the history
Signed-off-by: Moritz Hoffmann <[email protected]>
  • Loading branch information
antiguru committed Dec 20, 2023
1 parent d25f911 commit 8c23284
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/trace/implementations/merge_batcher.rs
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,10 @@ where
vals += 1;
}
upds += 1;
} else {
keys += 1;
vals += 1;
upds += 1;
}
prev_keyval = Some((key, val));
}
Expand Down
4 changes: 4 additions & 0 deletions src/trace/implementations/merge_batcher_col.rs
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,10 @@ where
vals += 1;
}
upds += 1;
} else {
keys += 1;
vals += 1;
upds += 1;
}
prev_keyval = Some((key, val));
}
Expand Down

0 comments on commit 8c23284

Please sign in to comment.