]> git.ipfire.org Git - thirdparty/git.git/commit
drop trailing newline from warning/error/die messages
authorJeff King <peff@peff.net>
Thu, 5 Sep 2024 08:51:49 +0000 (04:51 -0400)
committerJunio C Hamano <gitster@pobox.com>
Thu, 5 Sep 2024 16:07:12 +0000 (09:07 -0700)
commit1a60f2066aadf68391aed69f4d5914d914b60f5b
tree3f95d066dcae08cb04c0f51f91fc3d5168ffc679
parent2e7b89e038c0c888acf61f1b4ee5a43d4dd5e94c
drop trailing newline from warning/error/die messages

Our error reporting routines append a trailing newline, and the strings
we pass to them should not include them (otherwise we get an extra blank
line after the message).

These cases were all found by looking at the results of:

  git grep -P '[^_](error|error_errno|warning|die|die_errno)\(.*\\n"[,)]' '*.c'

Note that we _do_ sometimes include a newline in the middle of such
messages, to create multiline output (hence our grep matching "," or ")"
after we see the newline, so we know we're at the end of the string).

It's possible that one or more of these cases could intentionally be
including a blank line at the end, but having looked at them all
manually, I think these are all just mistakes.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
12 files changed:
builtin/bisect.c
builtin/fetch.c
builtin/stash.c
builtin/submodule--helper.c
fetch-pack.c
loose.c
negotiator/skipping.c
send-pack.c
serve.c
t/helper/test-path-utils.c
t/helper/test-progress.c
t/helper/test-reach.c