diff options
| author | Joe Perches <[email protected]> | 2012-12-17 16:01:56 -0800 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2012-12-17 17:15:19 -0800 |
| commit | 481eb486a88c9b068f0168ac4c21291802720933 (patch) | |
| tree | ded9373e9ace9887f28c8bf3ea1810c600a6b059 | |
| parent | 6cd7f3869c925622bbf420e1107a026d91dbd7f2 (diff) | |
checkpatch: extend line continuation test
Preprocessor directives and asm statements should be allowed to have a
line continuation.
Signed-off-by: Joe Perches <[email protected]>
Tested-by: Jingoo Han <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
| -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 6fa167758f82..3e9fee60642c 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); |