Bruce/bp
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix issue with replacement text printing before its line number

Browse Source
This commit is contained in:
Bruce Hill 2024-06-07 14:45:03 -04:00
parent 0e9f324955
commit 31e414af50
1 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
printmatch.c
View File

@ -232,6 +232,16 @@ public int fprint_match(FILE *out, const char *file_start, bp_match_t *m, print_
}
}
// Bugfix: if pattern matches at the start of a line and first printed character is from
// the replacement string, we need to be sure that the line number gets printed.
// Regression test: `seq 3 | bp '{..$}' -r '#@0'` (should not print "#" before line numbers)
if (r == text) {
if (opts && opts->fprint_between) {
printed += opts->fprint_between(out, m->start, m->start, opts->match_color);
if (opts->replace_color) printed += fprintf(out, "%s", opts->replace_color);
}
}
if (*r == '\\') {
++r;
if (*r == 'N') { // \N (nodent)