From: Dr. David von Oheimb <dev@ddvo.net>
Date: Fri, 13 Jun 2025 14:44:08 +0000 (+0200)
Subject: check-format-test-negatives.c: add 2nd macro indent test and hint on how known false... 
X-Git-Tag: openssl-3.6.0-alpha1~596
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e925b99f9435826aca3fa41b7661e6e9fa7e10b6;p=thirdparty%2Fopenssl.git

check-format-test-negatives.c: add 2nd macro indent test and hint on how known false positives are marked

Reviewed-by: Neil Horman <nhorman@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/27836)
---

diff --git a/util/check-format-test-negatives.c b/util/check-format-test-negatives.c
index 93cf20228bb..33abf002153 100644
--- a/util/check-format-test-negatives.c
+++ b/util/check-format-test-negatives.c
@@ -10,7 +10,7 @@
 
 /*
  * A collection of test cases where check-format.pl should not report issues.
- * There are some known false positives, though, which are marked below.
+ * There are some known false positives, though, which are marked below using /*@
  */
 
 #include <errno.h> /* should not report whitespace nits within <...> */
@@ -264,7 +264,14 @@ int g(void)
             && expr_line3)
         hanging_stmt;
 }
-#define m \
+
+#define m1                           \
+    if (ctx == NULL)                 \
+        return 0;                    \
+    if (ossl_param_is_empty(params)) \
+        return 1;                    \
+
+#define m2                                                               \
     do { /* should not be confused with function header followed by '{' */ \
     } while (0)