summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohannes Sixt <j6t@kdbg.org>2010-03-06 16:40:39 +0100
committerJunio C Hamano <gitster@pobox.com>2010-03-07 00:37:35 -0800
commitebaa79f462c48e0ed0341d9c8f9c97de557afcfd (patch)
treef3657097f459f1446ec6dce275aeac8101261f50
parent1d8cd418b49f08146d6d47cf8881965f922686cb (diff)
downloadgit-ebaa79f462c48e0ed0341d9c8f9c97de557afcfd.tar.gz
Make report() from usage.c public as vreportf() and use it.
There exist already a number of static functions named 'report', therefore, the function name was changed. Signed-off-by: Johannes Sixt <j6t@kdbg.org> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--fast-import.c8
-rw-r--r--git-compat-util.h1
-rw-r--r--http-backend.c5
-rw-r--r--usage.c10
4 files changed, 10 insertions, 14 deletions
diff --git a/fast-import.c b/fast-import.c
index 309f2c58a2..f2ef20cbf0 100644
--- a/fast-import.c
+++ b/fast-import.c
@@ -483,14 +483,12 @@ static void dump_marks(void);
static NORETURN void die_nicely(const char *err, va_list params)
{
static int zombie;
- char message[2 * PATH_MAX];
- vsnprintf(message, sizeof(message), err, params);
- fputs("fatal: ", stderr);
- fputs(message, stderr);
- fputc('\n', stderr);
+ vreportf("fatal: ", err, params);
if (!zombie) {
+ char message[2 * PATH_MAX];
+
zombie = 1;
write_crash_report(message);
end_packfile();
diff --git a/git-compat-util.h b/git-compat-util.h
index a3c4537366..3cabcdd8c4 100644
--- a/git-compat-util.h
+++ b/git-compat-util.h
@@ -192,6 +192,7 @@ extern char *gitbasename(char *);
#include "compat/bswap.h"
/* General helper functions */
+extern void vreportf(const char *prefix, const char *err, va_list params);
extern NORETURN void usage(const char *err);
extern NORETURN void usagef(const char *err, ...) __attribute__((format (printf, 1, 2)));
extern NORETURN void die(const char *err, ...) __attribute__((format (printf, 1, 2)));
diff --git a/http-backend.c b/http-backend.c
index 345c12b790..8c7b7d09ea 100644
--- a/http-backend.c
+++ b/http-backend.c
@@ -538,14 +538,11 @@ static void service_rpc(char *service_name)
static NORETURN void die_webcgi(const char *err, va_list params)
{
- char buffer[1000];
-
http_status(500, "Internal Server Error");
hdr_nocache();
end_headers();
- vsnprintf(buffer, sizeof(buffer), err, params);
- fprintf(stderr, "fatal: %s\n", buffer);
+ vreportf("fatal: ", err, params);
exit(0);
}
diff --git a/usage.c b/usage.c
index 79856a2b9f..ec4cf53b6b 100644
--- a/usage.c
+++ b/usage.c
@@ -5,7 +5,7 @@
*/
#include "git-compat-util.h"
-static void report(const char *prefix, const char *err, va_list params)
+void vreportf(const char *prefix, const char *err, va_list params)
{
char msg[4096];
vsnprintf(msg, sizeof(msg), err, params);
@@ -14,24 +14,24 @@ static void report(const char *prefix, const char *err, va_list params)
static NORETURN void usage_builtin(const char *err, va_list params)
{
- report("usage: ", err, params);
+ vreportf("usage: ", err, params);
exit(129);
}
static NORETURN void die_builtin(const char *err, va_list params)
{
- report("fatal: ", err, params);
+ vreportf("fatal: ", err, params);
exit(128);
}
static void error_builtin(const char *err, va_list params)
{
- report("error: ", err, params);
+ vreportf("error: ", err, params);
}
static void warn_builtin(const char *warn, va_list params)
{
- report("warning: ", warn, params);
+ vreportf("warning: ", warn, params);
}
/* If we are in a dlopen()ed .so write to a global variable would segfault