diff options
author | Junio C Hamano <gitster@pobox.com> | 2016-10-12 00:19:03 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2016-10-12 00:19:03 +0300 |
commit | 18fd96f1d706357704c8d51f12b9412cd424ac49 (patch) | |
tree | eae41a4b0201011c458a4504ae4906d6c30c5cda /graph.c | |
parent | 1f253d88face02b68a86fae36e6722c9ebc73856 (diff) | |
parent | 16477935245a522d99d0dd7e346638c02542f1d0 (diff) |
Merge branch 'jk/graph-padding-fix' into maint
The "graph" API used in "git log --graph" miscounted the number of
output columns consumed so far when drawing a padding line, which
has been fixed; this did not affect any existing code as nobody
tried to write anything after the padding on such a line, though.
* jk/graph-padding-fix:
graph: fix extra spaces in graph_padding_line
Diffstat (limited to 'graph.c')
-rw-r--r-- | graph.c | 16 |
1 files changed, 12 insertions, 4 deletions
@@ -1145,6 +1145,7 @@ int graph_next_line(struct git_graph *graph, struct strbuf *sb) static void graph_padding_line(struct git_graph *graph, struct strbuf *sb) { int i; + int chars_written = 0; if (graph->state != GRAPH_COMMIT) { graph_next_line(graph, sb); @@ -1160,14 +1161,21 @@ static void graph_padding_line(struct git_graph *graph, struct strbuf *sb) */ for (i = 0; i < graph->num_columns; i++) { struct column *col = &graph->columns[i]; + strbuf_write_column(sb, col, '|'); - if (col->commit == graph->commit && graph->num_parents > 2) - strbuf_addchars(sb, ' ', (graph->num_parents - 2) * 2); - else + chars_written++; + + if (col->commit == graph->commit && graph->num_parents > 2) { + int len = (graph->num_parents - 2) * 2; + strbuf_addchars(sb, ' ', len); + chars_written += len; + } else { strbuf_addch(sb, ' '); + chars_written++; + } } - graph_pad_horizontally(graph, sb, graph->num_columns); + graph_pad_horizontally(graph, sb, chars_written); /* * Update graph->prev_state since we have output a padding line |