Re: [PATCH] checkpatch: include text files in SPACE_BEFORE_TAB test

[Andy Whitcroft]

On Fri, Dec 05, 2014 at 09:45:37PM -0800, Frank Rowand wrote:
> From: Frank Rowand <frank.rowand@xxxxxxxxxxxxxx>
> 
> git-am whined about a patch that I submitted for the Documentation
> subtree, (https://lkml.org/lkml/2014/11/24/636) but checkpatch does
> not.  Make checkpatch just as whiney.
> 
> This patch moves the SPACE_BEFORE_TAB test before the check that
> excludes text files and also before the test that excludes
> assembly files.  It seems to me that both CODE_INDENT and
> SPACE_BEFORE_TAB are valid tests for assembly files, but I
> added an if around the moved CODE_INDENT test to include only
> source files, not including assembly files.  Thus this patch
> does not change which files are subject to CODE_INDENT.
> 
> The CODE_INDENT test is moved along with the SPACE_BEFORE_TAB
> test because the c76f4cb3d25e commit log says:
> 
>     This SPACE_BEFORE_TAB test is done after CODE_INDENT.
>     
>     If there are spaces used at the beginning of a line that should be
>     converted to tabs, please make sure that the CODE_INDENT test and
>     conversion is done before this SPACE_BEFORE_TAB test and conversion.
> 
> Signed-off-by: Frank Rowand <frank.rowand@xxxxxxxxxxxxxx>
> ---
> 
> 
> Andy, Joe,
> 
> Assembly files are still excluded from the CODE_INDENT test -- do you
> want to make another change to include them?

The format of assembly files is rather variable.  Tabs are not used in the
same way as in the rest of the kernel.  It is not clear you can guarentee
that the rules are appropriate there.  They are also vary rarely edited
in the greater scheme of things.

-apw
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/