[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] checkpatch: tweak the files in which TABs a
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] checkpatch: tweak the files in which TABs are checked |
Date: |
Wed, 10 Aug 2016 08:46:07 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> Include Python and shell scripts, and make an exception for Perl
> scripts we imported from Linux or elsewhere.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> scripts/checkpatch.pl | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index b7cb4ab..7ccf6a8 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -1351,8 +1351,10 @@ sub process {
> WARN("adding a line without newline at end of file\n" .
> $herecurr);
> }
>
> -# check we are in a valid source file C or perl if not then ignore this hunk
> - next if ($realfile !~ /\.(h|c|cpp|pl)$/);
> +# check we are in a valid source file; if not then tabs are allowed.
> +# make an exception from some scripts imported from other projects.
> + next if ($realfile !~ /\.(h|c|cpp|pl|py|sh)$/);
> + next if ($realfile =~
> /(checkpatch|get_maintainer|texi2pod)\.pl$/);
>
> # in QEMU, no tabs are allowed
> if ($rawline =~ /^\+.*\t/) {
I'm afraid "if not then tabs are allowed" is confusing. We're obviously
skipping more than just the tabs check: RCS/CVS revision markers, and a
whole bunch of C style checks. Makes sense, we don't want to do these
checks for imported files. "if not then tabs are allowed" starts to
make some sense only once you've stared at the next "next if ..." line
for a while. Let's avoid that. Minimal change:
# check we are in a valid source file; if not then ignore this hunk
# make an exception from some scripts imported from other projects.
Radim's "[PATCH] checkpatch: ignore automatically imported Linux
headers" adds a similar exception for other imported files in a
different place:
@@ -1312,6 +1312,9 @@ sub process {
# ignore non-hunk lines and lines being removed
next if (!$hunk_line || $line =~ /^-/);
+# ignore files that are being periodically imported from Linux
+ next if ($realfile =~
/^(linux-headers|include\/standard-headers)\//);
+
#trailing whitespace
if ($line =~ /^\+.*\015/) {
my $herevet = "$here\n" . cat_vet($rawline) . "\n";
Should both exceptions be in the same place?
[Qemu-devel] [PATCH 3/3] checkpatch: default to success if only warnings, Paolo Bonzini, 2016/08/09