rust/ripgrep
1
0
Fork 0
mirror of https://github.com/BurntSushi/ripgrep.git synced 2024-12-07 11:13:17 +02:00
Code Issues Releases Activity

core: fix file separator bug

Browse Source 
I introduced a regression in the migration off of the clap by having
both the buffer writer and the printer be responsible for printing file
separators in multi-threaded search. The buffer writer owns that
responsibility in multi-threaded search.
This commit is contained in:
Andrew Gallant 2023-11-21 07:40:56 -05:00
parent 082245dadb
commit c81caa673b
1 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
crates/core/flags/hiargs.rs
View File

@ -592,7 +592,8 @@ impl HiArgs {
&self,
wtr: W,
) -> grep::printer::Standard<W> {
grep::printer::StandardBuilder::new()
let mut builder = grep::printer::StandardBuilder::new();
builder
.byte_offset(self.byte_offset)
.color_specs(self.colors.clone())
.column(self.column)
@ -615,10 +616,17 @@ impl HiArgs {
self.field_match_separator.clone().into_bytes(),
)
.separator_path(self.path_separator.clone())
.separator_search(self.file_separator.clone())
.stats(self.stats.is_some())
.trim_ascii(self.trim)
.build(wtr)
.trim_ascii(self.trim);
// When doing multi-threaded searching, the buffer writer is
// responsible for writing separators since it is the only thing that
// knows whether something has been printed or not. But for the single
// threaded case, we don't use a buffer writer and thus can let the
// printer own this.
if self.threads == 1 {
builder.separator_search(self.file_separator.clone());
}
builder.build(wtr)
}
/// Builds a "summary" printer where search results are aggregated on a