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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

J. Bruce Fields wrote:

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

-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux