On Dec 9, 2012, at 10:36 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Steffen Prohaska <prohaska@xxxxxx> 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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html