diff options
author | René Scharfe <l.s.r@web.de> | 2017-10-01 16:45:45 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-10-02 13:14:07 +0900 |
commit | 38bdf62b738bb93f7e1a6af8058dc31f27c91d4e (patch) | |
tree | e9a0e62f052d12d0d5ac0360e6e4cec15c873a3a | |
parent | 72d4a9a721d519982cbc97c36f3fcf33b4103ed4 (diff) | |
download | git-rs/cleanup-strbuf-users.tar.gz |
graph: use strbuf_addchars() to add spacesrs/cleanup-strbuf-users
strbuf_addf() can be used to add a specific number of space characters
by using the format "%*s" with an empty string and specifying the
desired width. Use strbuf_addchars() instead as it's shorter, makes the
intent clearer and is a bit more efficient.
Signed-off-by: Rene Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r-- | graph.c | 10 |
1 files changed, 3 insertions, 7 deletions
@@ -696,12 +696,8 @@ static void graph_pad_horizontally(struct git_graph *graph, struct strbuf *sb, * This way, fields printed to the right of the graph will remain * aligned for the entire commit. */ - int extra; - if (chars_written >= graph->width) - return; - - extra = graph->width - chars_written; - strbuf_addf(sb, "%*s", (int) extra, ""); + if (chars_written < graph->width) + strbuf_addchars(sb, ' ', graph->width - chars_written); } static void graph_output_padding_line(struct git_graph *graph, @@ -787,7 +783,7 @@ static void graph_output_pre_commit_line(struct git_graph *graph, if (col->commit == graph->commit) { seen_this = 1; strbuf_write_column(sb, col, '|'); - strbuf_addf(sb, "%*s", graph->expansion_row, ""); + strbuf_addchars(sb, ' ', graph->expansion_row); chars_written += 1 + graph->expansion_row; } else if (seen_this && (graph->expansion_row == 0)) { /* |