diff options
| author | René Scharfe <l.s.r@web.de> | 2016-08-05 20:37:11 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2016-08-05 22:09:25 (GMT) |
| commit | bc57b9c0cc5a123365a922fa1831177e3fd607ed (patch) | |
| tree | 520d65d7a99278bbec6e1b2e25e5e045d2ac28b0 /builtin | |
| parent | 02962d36845b89145cd69f8bc65e015d78ae3434 (diff) | |
| download | git-bc57b9c0cc5a123365a922fa1831177e3fd607ed.zip git-bc57b9c0cc5a123365a922fa1831177e3fd607ed.tar.gz git-bc57b9c0cc5a123365a922fa1831177e3fd607ed.tar.bz2 | |
use strbuf_addstr() instead of strbuf_addf() with "%s"
Call strbuf_addstr() for adding a simple string to a strbuf instead of
using the heavier strbuf_addf(). This is shorter and documents the
intent more clearly.
Signed-off-by: Rene Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
| -rw-r--r-- | builtin/fmt-merge-msg.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/fmt-merge-msg.c b/builtin/fmt-merge-msg.c index e5658c3..ac84e99 100644 --- a/builtin/fmt-merge-msg.c +++ b/builtin/fmt-merge-msg.c @@ -272,7 +272,7 @@ static int cmp_string_list_util_as_integral(const void *a_, const void *b_) static void add_people_count(struct strbuf *out, struct string_list *people) { if (people->nr == 1) - strbuf_addf(out, "%s", people->items[0].string); + strbuf_addstr(out, people->items[0].string); else if (people->nr == 2) strbuf_addf(out, "%s (%d) and %s (%d)", people->items[0].string, |