Re: [PATCH 1/6] graph: output padding for merge subsequent parents

2013-02-08 Thread John Keeping
[Moved from the thread where this was initially posted to reply to the series.] Matthieu Moy wrote: > This works, but if we know we're not going to enter the while loop, it > seams even easier to do this: > > --- a/graph.c > +++ b/graph.c > @@ -1227,7 +1227,17 @@ void graph_show_commit(struct gi

[PATCH 1/6] graph: output padding for merge subsequent parents

2013-02-07 Thread John Keeping
When showing merges in git-log, the same commit is shown once for each parent. Combined with "--graph" this results in graph_show_commit() being called once for each parent without graph_update() being called. Currently graph_show_commit() does not print anything on subsequent invocations for the