diff options
author | Jeff King <peff@peff.net> | 2016-09-29 04:37:51 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2016-09-29 16:43:47 -0700 |
commit | 16477935245a522d99d0dd7e346638c02542f1d0 (patch) | |
tree | 469a3be766092e050b4f96dad4bc4342ac5993eb /t/t6020-merge-df.sh | |
parent | Git 2.8.4 (diff) | |
download | tgif-16477935245a522d99d0dd7e346638c02542f1d0.tar.xz |
graph: fix extra spaces in graph_padding_line
The graph_padding_line() function outputs a series of "|"
columns, and then pads with spaces to graph->width by
calling graph_pad_horizontally(). However, we tell the
latter that we wrote graph->num_columns characters, which is
not true; we also needed spaces between the columns. Let's
keep a count of how many characters we've written, which is
what all the other callers of graph_pad_horizontally() do.
Without this, any output that is written at the end of a
padding line will be bumped out by at least an extra
graph->num_columns spaces. Presumably nobody ever noticed
the bug because there's no code path that actually writes to
the end of a padding line.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t6020-merge-df.sh')
0 files changed, 0 insertions, 0 deletions