Messages in this thread | | | Subject | Re: [PATCH] scripts/checkpatch.pl: Change long line warning to 105 chars | From | Joe Perches <> | Date | Fri, 18 Dec 2009 09:54:44 -0800 |
| |
On Fri, 2009-12-18 at 09:43 -0800, Linus Torvalds wrote: > I like this. Except I think the indent test should count spaces too some > way. Or do we already warn about excessive space that should be tabs?
Andy's the checkpatch guy to verify this but:
There's a separate test for spaces that should be tabs.
If you had a 64 space indent and ran the silly script, it'd complain about the spaces.
If you converted the spaces to tabs and ran the silly script again with --strict, it'd complain about the indent depth.
| |