On Thu, May 20, 2010 at 08:36:50AM -0400, Jeff King wrote:
> git log --graph --oneline --summary 212f0ba
>
> Summary lines for some reason don't get properly indented or prefixed
> with branch lines.
This is my naive attempt at fixing it just by copying your other
changes. However, it seems to sometimes put several copies of the prefix
in front of summary lines (and sometimes the next commit line!?). So
clearly there is something about the output prefix code that I don't
quite understand.
Maybe this is helpful, and maybe not. :)
diff --git a/diff.c b/diff.c
index 3a1e05a..656f40b 100644
--- a/diff.c
+++ b/diff.c
@@ -3460,27 +3460,41 @@ static void show_rename_copy(FILE *file, const char *renamecopy, struct diff_fil
show_mode_change(file, p, 0);
}
-static void diff_summary(FILE *file, struct diff_filepair *p)
+static void diff_summary(struct diff_options *opt, struct diff_filepair *p)
{
+ FILE *file = opt->file;
+ char *line_prefix = "";
+
+ if (opt->output_prefix) {
+ struct strbuf *buf = opt->output_prefix(file, opt->output_prefix_data);
+ line_prefix = buf->buf;
+ }
+
switch(p->status) {
case DIFF_STATUS_DELETED:
+ fputs(line_prefix, file);
show_file_mode_name(file, "delete", p->one);
break;
case DIFF_STATUS_ADDED:
+ fputs(line_prefix, file);
show_file_mode_name(file, "create", p->two);
break;
case DIFF_STATUS_COPIED:
+ fputs(line_prefix, file);
show_rename_copy(file, "copy", p);
break;
case DIFF_STATUS_RENAMED:
+ fputs(line_prefix, file);
show_rename_copy(file, "rename", p);
break;
default:
if (p->score) {
+ fputs(line_prefix, file);
fputs(" rewrite ", file);
write_name_quoted(p->two->path, file, ' ');
fprintf(file, "(%d%%)\n", similarity_index(p));
}
+ fputs(line_prefix, file);
show_mode_change(file, p, !p->score);
break;
}
@@ -3692,7 +3706,7 @@ void diff_flush(struct diff_options *options)
if (output_format & DIFF_FORMAT_SUMMARY && !is_summary_empty(q)) {
for (i = 0; i < q->nr; i++)
- diff_summary(options->file, q->queue[i]);
+ diff_summary(options, q->queue[i]);
separator++;
}
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html