Messages in this thread Patch in this message | | | Subject | [PATCH V2] checkpatch: extend line continuation test | From | Joe Perches <> | Date | Fri, 16 Nov 2012 11:31:11 -0800 |
| |
Preprocessor directives and asm statements should be allowed to have a line continuation.
Signed-off-by: Joe Perches <joe@perches.com> --- V2: Make the check for preprocessor directives work properly # is not optional.
scripts/checkpatch.pl | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index d2d5ba1..bfb06ee 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -3006,10 +3006,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);
| |