Re: [PATCH] whitespace: fix initial-indent checking

El 16/12/2007, a las 10:08, Jakub Narebski escribió:

J. Bruce Fields wrote:

quoted

This allows catching initial indents like '\t        ' (a tab  
followed
by 8 spaces), while previously indent-with-non-tab caught only  
indents
that consisted entirely of spaces.

I prefer to use tabs for indent, but _spaces_ for align. While  
previous,
less strict version of check catches indent using spaces, this one  
also
catches _align_ using spaces.

I'd say that Jakub's is a fairly common use case (it's used in many  
places in the Git codebase too, I think) so it would be a bad thing to  
change the behaviour of "indent-with-non-tab".

If you also want to check for "align-with-non-tab" then it really  
should be a separate, optional class of whitespace error.

Cheers,
Wincent