shortlog: fix wrapping lines of wraplen
authorSteffen Prohaska <prohaska@zib.de>
Tue, 11 Dec 2012 05:59:21 +0000 (06:59 +0100)
committerJunio C Hamano <gitster@pobox.com>
Tue, 11 Dec 2012 18:01:44 +0000 (10:01 -0800)
commit5b59708268c82262fb13fb8681663dd158b030bd
tree42d7b69fcd74c67a83143085a3053a10c2c8c38b
parent14e1a4e1ff70aff36db3f5d2a8b806efd0134d50
shortlog: fix wrapping lines of wraplen

A recent commit [1] fixed a off-by-one wrapping error.  As a
side-effect, the conditional in add_wrapped_shortlog_msg() to decide
whether to append a newline needs to be removed.  The function
should always append a newline, which was the case before the
off-by-one fix, because strbuf_add_wrapped_text() never returns a
value of wraplen; when it returns wraplen, the string does not end
with a newline, so this caller needs to add one anyway.

[1] 14e1a4e1ff70aff36db3f5d2a8b806efd0134d50 utf8: fix off-by-one
    wrapping of text

Signed-off-by: Steffen Prohaska <prohaska@zib.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/shortlog.c
t/t4201-shortlog.sh