From: Kevin Ballard <kevin@sb.org>
Date: Wed, 20 Oct 2010 06:38:41 GMT
Subject: Re: [PATCH] diff: handle lines containing only whitespace better
Message-ID: <780B144B-03E0-4ED5-8E92-D4EB3CBBBF71@sb.org>
In-Reply-To: <7vzku9wfrp.fsf@alter.siamese.dyndns.org>

On Oct 19, 2010, at 11:16 PM, Junio C Hamano wrote:

> Hmm, tests?

I checked, there seem to be no existing tests for the whitespace highlighting output. All the tests just use `git diff --check` to see if it was caught. And given that the problem only occurs when it's emitting the colored highlighting, I wasn't sure how to go about adding tests for this as I'd need to create an expect file that contains all the same ansi color codes, and I thought that might be a bit fragile or hard to do correctly.

Incidentally, I just realized the description of the patch is slightly wrong. The problem only occurs when the line contains at least one tab. Should I resend the patch with an updated description? I can also attempt to write tests if you can give me some guidance on how to deal with the need for ansi color codes.

-Kevin Ballard