[PATCH 02/12] diff: emit_line_ws_markup() if/else style fix
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Nov 4, 2025, 02:09 UTC
- Message-ID
- <20251104020928.582199-3-gitster@pobox.com>
- In-Reply-To
- <20251104020928.582199-1-gitster@pobox.com>
Apply the simple rule: if you need {} in one arm of the if/else if/else... cascade, have {} in all of them.
Signed-off-by: Junio C Hamano <gitster@pobox.com> --- diff.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/diff.c b/diff.c index 74261b332a..9a24a0791c 100644 --- a/diff.c +++ b/diff.c @@ -1327,14 +1327,14 @@ static void emit_line_ws_markup(struct diff_options *o, ws = NULL; } - if (!ws && !set_sign) + if (!ws && !set_sign) { emit_line_0(o, set, NULL, 0, reset, sign, line, len); - else if (!ws) { + } else if (!ws) { emit_line_0(o, set_sign, set, !!set_sign, reset, sign, line, len); - } else if (blank_at_eof) + } else if (blank_at_eof) { /* Blank line at EOF - paint '+' as well */ emit_line_0(o, ws, NULL, 0, reset, sign, line, len); - else { + } else { /* Emit just the prefix, then the rest. */ emit_line_0(o, set_sign ? set_sign : set, NULL, !!set_sign, reset, sign, "", 0);
-- 2.52.0-rc0