It's pointless to use the
do { ... } while (0)
idiom and add a trailing semicolon inside the macro's expansion.
2023-09-06 Bruno Haible <[email protected]>
argp: Fix clang -Wextra-semi-stmt warning.
* lib/argp-help.c (SKIPWS): Remove trailing semicolon.
diff --git a/lib/argp-help.c b/lib/argp-help.c
index 15d94993c1..7141edc74e 100644
--- a/lib/argp-help.c
+++ b/lib/argp-help.c
@@ -165,7 +165,7 @@ fill_in_uparams (const struct argp_state *state)
const char *var = getenv ("ARGP_HELP_FMT");
struct uparams new_params = uparams;
-#define SKIPWS(p) do { while (isspace ((unsigned char) *p)) p++; } while (0);
+#define SKIPWS(p) do { while (isspace ((unsigned char) *p)) p++; } while (0)
if (var)
{
