[PATCH 0/2] Re: [PATCH] shortlog: Fix wrapping lines of wraplen

From: Steffen Prohaska <prohaska@zib.de>
Date: Dec 11, 2012, 05:59 UTC
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 <gitster@pobox.com> wrote:

Show 8 quoted lines

> Steffen Prohaska <prohaska@zib.de> 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