]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
checksrc: improve the SPACESEMICOLON error message
authorDaniel Stenberg <daniel@haxx.se>
Mon, 18 Oct 2021 08:46:04 +0000 (10:46 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Mon, 18 Oct 2021 21:52:30 +0000 (23:52 +0200)
and adjust the MULTISPACE one to use plural

Closes #7866

lib/checksrc.pl

index bb3495f792bfc2d0f874d7f0436ea7ad6b6cb96b..8f98a99ab57a2bda9e7e7210f652b0f93508c559 100755 (executable)
@@ -547,7 +547,6 @@ sub scanfile {
             }
             elsif($even && $postparen &&
                ($postparen !~ /^ *$/) && ($postparen !~ /^ *[,{&|\\]+/)) {
-                print STDERR "5: '$postparen'\n";
                 checkwarn("ONELINECONDITION",
                           $line, length($l)-length($postparen), $file, $l,
                           "conditional block on the same line");
@@ -648,7 +647,7 @@ sub scanfile {
         # check for space before the semicolon last in a line
         if($l =~ /^(.*[^ ].*) ;$/) {
             checkwarn("SPACESEMICOLON",
-                      $line, length($1), $file, $ol, "space before last semicolon");
+                      $line, length($1), $file, $ol, "no space before semicolon");
         }
 
         # scan for use of banned functions
@@ -706,7 +705,6 @@ sub scanfile {
         # more steps, if not a cpp line
         if(!$prevp && ($prevl =~ /^( *)((if|while|for)\(.*\{|else)\z/)) {
             my $first = length($1);
-
             # this line has some character besides spaces
             if($l =~ /^( *)[^ ]/) {
                 my $second = length($1);
@@ -799,9 +797,7 @@ sub scanfile {
            $nostr =~ /^(.*(\S)) + [{?]/i) {
             checkwarn("MULTISPACE",
                       $line, length($1)+1, $file, $ol,
-                      "multiple space");
-            print STDERR "L: $l\n";
-            print STDERR "nostr: $nostr\n";
+                      "multiple spaces");
         }
       preproc:
         $line++;