]> rtime.felk.cvut.cz Git - linux-imx.git/commitdiff
checkpatch: don't warn on blank lines before/after braces as often
authorJoe Perches <joe@perches.com>
Wed, 3 Jul 2013 22:05:29 +0000 (15:05 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 3 Jul 2013 23:07:44 +0000 (16:07 -0700)
Check to make sure the blank lines aren't comment lines like:

  bool foo(bool bar)
  {
/* Don't warn on a leading comment */
return !bar;
/* Don't warn on a trailing comment either */
  }

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
scripts/checkpatch.pl

index 4ad40523b41992c0c00e5d59dfd476042c07e454..c43be815cc71e5c8cb6b579aebecb7eb3bba40be 100755 (executable)
@@ -3258,11 +3258,11 @@ sub process {
                }
 
 # check for unnecessary blank lines around braces
-               if (($line =~ /^.\s*}\s*$/ && $prevline =~ /^.\s*$/)) {
+               if (($line =~ /^.\s*}\s*$/ && $prevrawline =~ /^.\s*$/)) {
                        CHK("BRACES",
                            "Blank lines aren't necessary before a close brace '}'\n" . $hereprev);
                }
-               if (($line =~ /^.\s*$/ && $prevline =~ /^..*{\s*$/)) {
+               if (($rawline =~ /^.\s*$/ && $prevline =~ /^..*{\s*$/)) {
                        CHK("BRACES",
                            "Blank lines aren't necessary after an open brace '{'\n" . $hereprev);
                }