checkSpace: be more careful about bad function headers.

Previously we would forbid macro indentations like this:

FOO({
  int x;
  })

But clang-format sometimes generates those.

diff --git a/scripts/maint/checkSpace.pl b/scripts/maint/checkSpace.pl
index bf1c69e00aa77bd378086a0270e44494e74571d8..27615f910cc0e00af8661694450d011e48f2b8bc 100755 (executable)
--- a/scripts/maint/checkSpace.pl
+++ b/scripts/maint/checkSpace.pl
@@ -208,7 +208,7 @@ for my $fn (@ARGV) {
                 ($fn !~ /\.h$/ && /^[a-zA-Z0-9_]/ &&
                  ! /^(?:const |static )*(?:typedef|struct|union)[^\(]*$/ &&
                  ! /= *\{$/ && ! /;$/) && ! /^[a-zA-Z0-9_]+\s*:/) {
-                if (/.\{$/){
+                if (/[^,\s]\s*\{$/){
                     msg "fn() {:$fn:$.\n";
                     $in_func_head = 0;
                 } elsif (/^\S[^\(]* +\**[a-zA-Z0-9_]+\(/) {