[dm-devel] Re: [PATCH] scripts/checkpatch.pl: Change long line warning to 105 chars

Joe Perches joe at perches.com
Fri Dec 18 17:54:44 UTC 2009


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.




More information about the dm-devel mailing list