aboutsummaryrefslogtreecommitdiff
path: root/scripts/checkpatch.pl
diff options
context:
space:
mode:
authorPaolo Bonzini <pbonzini@redhat.com>2016-08-09 17:47:42 +0200
committerPaolo Bonzini <pbonzini@redhat.com>2016-08-10 11:09:54 +0200
commit906fb135e4b875465c424cb9b2b47d90265f7897 (patch)
tree5ceacf4ca0b5d8333f7bc34e5d97cc53ec0ddbee /scripts/checkpatch.pl
parent3fdd0ee393e26178a4892e101e60b011bbfaa9ea (diff)
downloadqemu-906fb135e4b875465c424cb9b2b47d90265f7897.zip
qemu-906fb135e4b875465c424cb9b2b47d90265f7897.tar.gz
qemu-906fb135e4b875465c424cb9b2b47d90265f7897.tar.bz2
checkpatch: tweak the files in which TABs are checked
Include Python and shell scripts, and make an exception for Perl scripts we imported from Linux or elsewhere. Acked-by: Cornelia Huck <cornelia.huck@de.ibm.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'scripts/checkpatch.pl')
-rwxr-xr-xscripts/checkpatch.pl9
1 files changed, 5 insertions, 4 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 8d1813e..082c4ce 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -1334,7 +1334,7 @@ sub process {
}
# check we are in a valid source file if not then ignore this hunk
- next if ($realfile !~ /\.(h|c|cpp|s|S|pl|sh)$/);
+ next if ($realfile !~ /\.(h|c|cpp|s|S|pl|py|sh)$/);
#80 column limit
if ($line =~ /^\+/ &&
@@ -1354,10 +1354,11 @@ 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)$/);
+# tabs are only allowed in assembly source code, and in
+# some scripts we imported from other projects.
+ next if ($realfile =~ /\.(s|S)$/);
+ next if ($realfile =~ /(checkpatch|get_maintainer|texi2pod)\.pl$/);
-# in QEMU, no tabs are allowed
if ($rawline =~ /^\+.*\t/) {
my $herevet = "$here\n" . cat_vet($rawline) . "\n";
ERROR("code indent should never use tabs\n" . $herevet);