Dan Mick wrote:
> well, maybe just surprising.
> 
> I couldn't figure out why my webrev wasn't showing my whitespace changes.  
> It's
> by design: UDIFFCMD and CDIFFCMD both have -b set by default.
> 
> I think that's suboptimal; it means the naive (read: me) are confused into a 
> digging operation, whereas the optimizers (don't show me noise, just show me 
> relevant changes) are handled by default.  I don't always want whitespace to 
> be 
> ignored in the diff, certainly.
> 
> I think it would serve more people better without -b.
> 
> Opinions?

I agree. If I make a whitespace change I want to see it,
and as a code reviewer I want to see it too. Basically,
I want to see every change period.

Cheers,
Jim
_______________________________________________
tools-discuss mailing list
tools-discuss@opensolaris.org

Reply via email to