diff options
Diffstat (limited to 'util')
-rw-r--r-- | util/check-format-test-negatives.c | 9 | ||||
-rwxr-xr-x | util/check-format.pl | 10 |
2 files changed, 11 insertions, 8 deletions
diff --git a/util/check-format-test-negatives.c b/util/check-format-test-negatives.c index 89b8b19c2e..c73cc04695 100644 --- a/util/check-format-test-negatives.c +++ b/util/check-format-test-negatives.c @@ -273,11 +273,12 @@ static varref cmp_vars[] = { /* comment. comment? comment! */ /* comment */ \ } -/* 'struct' in function header */ -static int f(struct pem_pass_data *pass_data) +union un var; /* struct/union/enum in variable type */ +struct provider_store_st *f() /* struct/union/enum in function return type */ +{ +} +static void f(struct pem_pass_data *data) /* struct/union/enum in arg list */ { - if (pass_data == NULL) - return 0; } static void *fun(void) diff --git a/util/check-format.pl b/util/check-format.pl index 734022c540..882b38de41 100755 --- a/util/check-format.pl +++ b/util/check-format.pl @@ -929,10 +929,12 @@ while (<>) { # loop over all lines of all input files } # set $in_typedecl and potentially $hanging_offset for type declaration - if (!$in_expr && @nested_indents == 0 && # not in expression - m/(^|^.*\W)(typedef|struct|union|enum)(\W.*|$)$/ && - parens_balance($1) == 0) { # not in newly started expression - # not needed: $keyword_opening_brace = $2 if $3 =~ m/\{/; + if (!$in_expr && @nested_indents == 0 # not in expression + && m/(^|^.*\W)(typedef|struct|union|enum)(\W.*|$)$/ + && parens_balance($1) == 0 # not in newly started expression or function arg list + && ($2 eq "typedef" || !($3 =~ m/\s*\w++\s*(.)/ && $1 ne "{")) # 'struct'/'union'/'enum' <name> not followed by '{' + # not needed: && $keyword_opening_brace = $2 if $3 =~ m/\{/; + ) { $in_typedecl++; $hanging_offset += INDENT_LEVEL if m/\*.*\(/; # '*' followed by '(' - seems consistent with Emacs C mode } |