From: Junio C Hamano Date: Fri, 05 Feb 2010 06:02:03 GMT Subject: Re: Showing whitespace on minus lines of diff ouput Message-ID: <7vr5p0kyp0.fsf@alter.siamese.dyndns.org> In-Reply-To: <7veil0o02z.fsf@alter.siamese.dyndns.org> Junio C Hamano writes: > Jay Soffian writes: > >> The colored diff output is quite helpful to show introductions of whitespace. >> >> However, if whitespace has been removed, it's impossible to see in the >> diff output since whitespace is colorized on '+' lines, but not '-' >> lines. >> >> I'm looking at diff.c, but wow. Can someone more familiar with this >> file point me toward how I'd make a patch to colorize whitespace on >> '-' lines as well? > > Totally uninterested. "diff -R" would be enough, wouldn't it? Actually the reason I don't like the idea to highlight corrected errors was not just because "'diff -R' is enough". The coloring of whitespace errors are like compiler warnings, at least to me, and giving similar highlighting to corrected ones defeats the purpose by making the output more distracting..