Hi Patrick,
On Fri, 10 Jan 2025, Patrick Steinhardt wrote:
> diff --git a/builtin/blame.c b/builtin/blame.c
> index d7630ac89cb7bd6e9ce5d72c6a98aa433b3b12da..7555c445abe7ca2fa54670ac8fee1d95a6dbafe3 100644
> --- a/builtin/blame.c
> +++ b/builtin/blame.c
> @@ -489,9 +489,9 @@ static void emit_other(struct blame_scoreboard *sb, struct blame_entry *ent, int
> fputs(color, stdout);
>
> if (suspect->commit->object.flags & UNINTERESTING) {
> - if (blank_boundary)
> - memset(hex, ' ', length);
> - else if (!(opt & OUTPUT_ANNOTATE_COMPAT)) {
> + if (blank_boundary) {
> + memset(hex, ' ', strlen(hex));
Using `strlen()` is a neat trick.
I could have done without slipping in a style change (introducing
curlies), but the most important thing is that it fixes the bug.
Thank you,
Johannes
> + } else if (!(opt & OUTPUT_ANNOTATE_COMPAT)) {
> length--;
> putchar('^');
> }
