diff options
| author | Joe Perches <joe@perches.com> | 2012-12-17 16:01:56 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-12-17 17:15:19 -0800 |
| commit | 481eb486a88c9b068f0168ac4c21291802720933 (patch) | |
| tree | ded9373e9ace9887f28c8bf3ea1810c600a6b059 /scripts/checkpatch.pl | |
| parent | 6cd7f3869c925622bbf420e1107a026d91dbd7f2 (diff) | |
| download | op-kernel-dev-481eb486a88c9b068f0168ac4c21291802720933.zip op-kernel-dev-481eb486a88c9b068f0168ac4c21291802720933.tar.gz | |
checkpatch: extend line continuation test
Preprocessor directives and asm statements should be allowed to have a
line continuation.
Signed-off-by: Joe Perches <joe@perches.com>
Tested-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts/checkpatch.pl')
| -rwxr-xr-x | scripts/checkpatch.pl | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 6fa1677..3e9fee6 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -3009,10 +3009,12 @@ sub process { } } -# check for line continuations outside of #defines +# check for line continuations outside of #defines, preprocessor #, and asm } else { if ($prevline !~ /^..*\\$/ && + $line !~ /^\+\s*\#.*\\$/ && # preprocessor + $line !~ /^\+.*\b(__asm__|asm)\b.*\\$/ && # asm $line =~ /^\+.*\\$/) { WARN("LINE_CONTINUATIONS", "Avoid unnecessary line continuations\n" . $herecurr); |
