From 180926636e47ecfe28d03cec493af75899994f0f Mon Sep 17 00:00:00 2001 From: Linus Torvalds Date: Thu, 23 Jun 2005 13:56:55 -0700 Subject: [PATCH] Clean up git-diff-tree 'header' generation --- diff-tree.c | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/diff-tree.c b/diff-tree.c index 7446e09af..d467e6cf8 100644 --- a/diff-tree.c +++ b/diff-tree.c @@ -280,11 +280,7 @@ static int call_diff_flush(void) return 0; } if (header) { - const char *fmt = "%s"; - if (diff_output_format == DIFF_FORMAT_MACHINE) - fmt = "%s%c"; - - printf(fmt, header, 0); + printf("%s%c", header, diff_output_format == DIFF_FORMAT_MACHINE ? 0 : '\n'); header = NULL; } diff_flush(diff_output_format); @@ -318,18 +314,16 @@ static int diff_root_tree(const unsigned char *new, const char *base) return retval; } -static char *generate_header(const char *commit, const char *parent, const char *msg, unsigned long len) +static const char *generate_header(const char *commit, const char *parent, const char *msg, unsigned long len) { static char this_header[16384]; int offset; - offset = sprintf(this_header, "%s%s (from %s)\n", header_prefix, commit, parent); - if (verbose_header) { - offset += pretty_print_commit(commit_format, msg, len, this_header + offset, sizeof(this_header) - offset); - this_header[offset++] = '\n'; - this_header[offset++] = 0; - } + if (!verbose_header) + return commit; + offset = sprintf(this_header, "%s%s (from %s)\n", header_prefix, commit, parent); + offset += pretty_print_commit(commit_format, msg, len, this_header + offset, sizeof(this_header) - offset); return this_header; } -- 2.26.2