Re: color.diff.whitespace unused on removed lines

2016-10-04 Thread Junio C Hamano
Jeff King writes: > On Tue, Oct 04, 2016 at 05:35:23PM +0200, Sandro Santilli wrote: > >> > We later did b8767f7 (diff.c: --ws-error-highlight= option, >> > 2015-05-26) to let you see them on other lines, though. I think that >> > would do what you want. >> >> Thanks, it does do what I want. >>

Re: color.diff.whitespace unused on removed lines

2016-10-04 Thread Jeff King
On Tue, Oct 04, 2016 at 05:35:23PM +0200, Sandro Santilli wrote: > > We later did b8767f7 (diff.c: --ws-error-highlight= option, > > 2015-05-26) to let you see them on other lines, though. I think that > > would do what you want. > > Thanks, it does do what I want. > Any chance to specify it in t

Re: color.diff.whitespace unused on removed lines

2016-10-04 Thread Sandro Santilli
On Tue, Oct 04, 2016 at 11:29:55AM -0400, Jeff King wrote: > On Tue, Oct 04, 2016 at 10:14:29AM +0200, Sandro Santilli wrote: > > > The color.diff.whitespace configuration is not used on > > removed lines, but only on added lines. > > Right. The original purpose was to warn you when you were intr

Re: color.diff.whitespace unused on removed lines

2016-10-04 Thread Jeff King
On Tue, Oct 04, 2016 at 10:14:29AM +0200, Sandro Santilli wrote: > The color.diff.whitespace configuration is not used on > removed lines, but only on added lines. Right. The original purpose was to warn you when you were introducing whitespace breakages. Getting rid of other people's whitespace

color.diff.whitespace unused on removed lines

2016-10-04 Thread Sandro Santilli
The color.diff.whitespace configuration is not used on removed lines, but only on added lines. As I'm removing trailing whitespaces all my diffs fail to give me any information about the real action being taken, due to this lack of support. As a workaround, I've found the -R switch for "git show"