From: Junio C Hamano <gitster@pobox.com>
Date: Sat, 14 Jun 2008 07:22:38 GMT
Subject: Re: [PATCH 3/4] avoid trailing whitespace in zero-change diffstat lines
Message-ID: <7vtzfwh45d.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <20080614065614.GC9006@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

> In some cases, we produce a diffstat line even though no lines have
> changed (e.g., because of an exact rename). In this case, there is no
> +/- "graph" after the number of changed lines. However, we output the
> space separator unconditionally, meaning that these lines contained a
> trailing space character.
>
> This isn't a huge problem, but in cleaning up the output we are able to
> eliminate some trailing whitespace from a test vector.

This is why I love your patches.  Not merely fixing superficial issues but
doing so with _real thinking_ ;-)