Re: [PATCH]CHECKPATCH:fix misleading output on spaces/tabs error

[Andy Whitcroft]

On Wed, Mar 19, 2008 at 11:06:02AM +0800, Daolong Wang wrote:
> I got confused by this line:
> 
>     "ERROR: use tabs not spaces"
> 
> It literally means "use tabs but not spaces is WRONG", which
>  is WRONG.
> 
>     "ERROR: do not use spaces when tabs expected"
> 
> sounds more appropriate.

Except that it says "use tabs not spaces when you should use tabs" and
doesn't actually tell us what the error was, nor how to correct it.  So
its not really any better than the original (which is also useless in
this regard).

How about:

	"code indent should use tabs where possible"

> Signed-off-by: Wang Daolong <ahlongxp@xxxxxxxxx>
> ---
> --- scripts/checkpatch.pl.org	2008-03-18 23:08:41.000000000 +0800
> +++ scripts/checkpatch.pl	2008-03-18 23:19:34.000000000 +0800
> @@ -1064,7 +1064,7 @@ sub process {
>  		if ($rawline =~ /^\+\s* \t\s*\S/ ||
>  		    $rawline =~ /^\+\s*        \s*/) {
>  			my $herevet = "$here\n" . cat_vet($rawline) . "\n";
> -			ERROR("use tabs not spaces\n" . $herevet);
> +			ERROR("do not use spaces when tabs expected\n" . $herevet);
>  		}
> 
>  # check for RCS/CVS revision markers

-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/