Code

commit.c::print_summary: do not release the format string too early
authorJunio C Hamano <gitster@pobox.com>
Sun, 17 Jan 2010 08:57:51 +0000 (00:57 -0800)
committerJunio C Hamano <gitster@pobox.com>
Sun, 17 Jan 2010 08:57:51 +0000 (00:57 -0800)
When we are showing a clean merge, log_tree_commit() won't show the header
and we would need the format string to format the commit summary ourselves.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin-commit.c

index a73a532f2ff4e11d070ddb2bdf490f850a6a425f..7f61e87ebdcdcbf79b21631ced97bb4facbecf9f 100644 (file)
@@ -1013,7 +1013,6 @@ static void print_summary(const char *prefix, const unsigned char *sha1)
        rev.verbose_header = 1;
        rev.show_root_diff = 1;
        get_commit_format(format.buf, &rev);
-       strbuf_release(&format);
        rev.always_show_header = 0;
        rev.diffopt.detect_rename = 1;
        rev.diffopt.rename_limit = 100;
@@ -1036,6 +1035,7 @@ static void print_summary(const char *prefix, const unsigned char *sha1)
                printf("%s\n", buf.buf);
                strbuf_release(&buf);
        }
+       strbuf_release(&format);
 }
 
 static int git_commit_config(const char *k, const char *v, void *cb)