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 22:25:15 +0100 |
| commit | e081a7c3b7290c50c817598e5f2686d068540762 (patch) | |
| tree | e1b23b79d8067b0d4fe4629c5bc8d8d787958aa2 /http-backend.c | |
| parent | usage.c: add a die_message() routine (diff) | |
| download | git-e081a7c3b7290c50c817598e5f2686d068540762.tar.xz git-e081a7c3b7290c50c817598e5f2686d068540762.zip | |
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 'http-backend.c')
| -rw-r--r-- | http-backend.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/http-backend.c b/http-backend.c index 3d6e2ff17f..982cb62c7c 100644 --- a/http-backend.c +++ b/http-backend.c @@ -659,8 +659,9 @@ static NORETURN void die_webcgi(const char *err, va_list params) { if (dead <= 1) { struct strbuf hdr = STRBUF_INIT; + report_fn die_message_fn = get_die_message_routine(); - vreportf("fatal: ", err, params); + die_message_fn(err, params); http_status(&hdr, 500, "Internal Server Error"); hdr_nocache(&hdr); |