提交 90a7c905 编写于 作者: D Dr. David von Oheimb

fix false positive of check-format.pl regarding '#if' on preceding line; extend negative tests

Reviewed-by: NRichard Levitte <levitte@openssl.org>
Reviewed-by: NDavid von Oheimb <david.von.oheimb@siemens.com>
(Merged from https://github.com/openssl/openssl/pull/11285)
上级 aed723f1
...@@ -113,7 +113,46 @@ int f(void) /* ...@@ -113,7 +113,46 @@ int f(void) /*
5; 5;
else else
6; 6;
if (1) {
if (2) {
case MAC_TYPE_MAC:
{
EVP_MAC_CTX *new_mac_ctx;
if (ctx->pkey == NULL)
return 0;
}
break;
default:
/* This should be dead code */
return 0;
}
}
if (expr_line1
== expr_line2
&& expr_line3) {
c1;
} else {
c;
d;
}
if (expr_line1
== expr_line2
&& expr_line3)
hanging_stmt;
} }
const OPTIONS passwd_options[] = {
{"aixmd5", OPT_AIXMD5, '-', "AIX MD5-based password algorithm"},
#if !defined(OPENSSL_NO_DES) && !defined(OPENSSL_NO_DEPRECATED_3_0)
{"crypt", OPT_CRYPT, '-', "Standard Unix password algorithm (default)"},
#endif
OPT_R_OPTIONS,
{NULL}
};
typedef * d(int) typedef * d(int)
x; x;
typedef (int) typedef (int)
......
...@@ -1001,7 +1001,7 @@ while (<>) { # loop over all lines of all input files ...@@ -1001,7 +1001,7 @@ while (<>) { # loop over all lines of all input files
# check for opening brace after if/while/for/switch/do not on same line # check for opening brace after if/while/for/switch/do not on same line
# note that "no '{' on same line after '} else'" is handled further below # note that "no '{' on same line after '} else'" is handled further below
if (/^[\s@]*{/ && # leading '{' if (/^[\s@]*{/ && # leading '{'
$line_before > 0 && $line_before > 0 && !($contents_before_ =~ m/^\s*#/) && # not preprocessor directive '#if
(my ($head, $mid, $tail) = ($contents_before_ =~ m/(^|^.*\W)(if|while|for|switch|do)(\W.*$|$)/))) { (my ($head, $mid, $tail) = ($contents_before_ =~ m/(^|^.*\W)(if|while|for|switch|do)(\W.*$|$)/))) {
my $brace_after = $tail =~ /^[\s@]*{/; # any whitespace or comments then '{' my $brace_after = $tail =~ /^[\s@]*{/; # any whitespace or comments then '{'
report("'{' not on same line as preceding '$mid'") if !$brace_after; report("'{' not on same line as preceding '$mid'") if !$brace_after;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册