diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2021-12-07 19:26:30 +0100 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-12-07 13:25:15 -0800 |
commit | e081a7c3b7290c50c817598e5f2686d068540762 (patch) | |
tree | e1b23b79d8067b0d4fe4629c5bc8d8d787958aa2 /parse-options.c | |
parent | usage.c: add a die_message() routine (diff) | |
download | tgif-e081a7c3b7290c50c817598e5f2686d068540762.tar.xz |
usage.c API users: use die_message() for "fatal :" + exit 128
Change code that printed its own "fatal: " message and exited with a
status code of 128 to use the die_message() function added in a
preceding commit.
This change also demonstrates why the return value of
die_message_routine() needed to be that of "report_fn". We have
callers such as the run-command.c::child_err_spew() which would like
to replace its error routine with the return value of
"get_die_message_routine()".
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'parse-options.c')
-rw-r--r-- | parse-options.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/parse-options.c b/parse-options.c index fc5b43ff0b..8bc0a21f1d 100644 --- a/parse-options.c +++ b/parse-options.c @@ -1075,6 +1075,6 @@ void NORETURN usage_msg_opt(const char *msg, const char * const *usagestr, const struct option *options) { - fprintf(stderr, "fatal: %s\n\n", msg); + die_message("%s\n", msg); /* The extra \n is intentional */ usage_with_options(usagestr, options); } |