From: Steffen Prohaska Date: Tue, 11 Dec 2012 05:59:20 GMT Subject: [PATCH 0/2] Re: [PATCH] shortlog: Fix wrapping lines of wraplen Message-ID: <1355205562-23459-1-git-send-email-prohaska@zib.de> In-Reply-To: <7v8v97efdv.fsf@alter.siamese.dyndns.org> On Dec 9, 2012, at 10:36 AM, Junio C Hamano wrote: > Steffen Prohaska writes: > > > A recent commit [1] fixed a off-by-one wrapping error. As > > a side-effect, add_wrapped_shortlog_msg() needs to be changed to always > > append a newline. > > Could you clarify "As a side effect" a bit more? Do you mean > something like this? See updated patches below. Steffen Steffen Prohaska (2): shortlog: Fix wrapping lines of wraplen (was broken since recent off-by-one fix) strbuf_add_wrapped*(): Remove unused return value builtin/shortlog.c | 5 ++--- t/t4201-shortlog.sh | 24 ++++++++++++++++++++++++ utf8.c | 13 ++++++------- utf8.h | 4 ++-- 4 files changed, 34 insertions(+), 12 deletions(-) -- 1.8.1.rc1.2.gfb98a3a