argp: Replace some 'continue;' statements with if/else.

* lib/argp-help.c (validate_uparams): Use if/else instead of
'continue;'.

diff --git a/ChangeLog b/ChangeLog
index 42163d2c83682cb196ce03c0081a03f19696977e..ab93efe0523eb704006f8423929f1b1c610bb1cb 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2025-12-07  Bruno Haible  <bruno@clisp.org>
+
+       argp: Replace some 'continue;' statements with if/else.
+       * lib/argp-help.c (validate_uparams): Use if/else instead of
+       'continue;'.
+
 2025-12-07  G. Branden Robinson  <g.branden.robinson@gmail.com>
 
        doc: Fix a typo.

diff --git a/lib/argp-help.c b/lib/argp-help.c
index 5bf4d3aacf34d903d7c9cd187254a77c83adadb1..d0ac0a5fa59ffd7176f681f756be0dc6ffa25149 100644 (file)
--- a/lib/argp-help.c
+++ b/lib/argp-help.c
@@ -140,19 +140,18 @@ validate_uparams (const struct argp_state *state, struct uparams *upptr)
 
   for (up = uparam_names; up < uparam_names + nuparam_names; up++)
     {
-      if (up->is_bool
-          || up->uparams_offs == offsetof (struct uparams, rmargin))
-        continue;
-      if (*(int *)((char *)upptr + up->uparams_offs) >= upptr->rmargin)
-        {
-          __argp_failure (state, 0, 0,
-                          dgettext (state == NULL ? NULL
-                                    : state->root_argp->argp_domain,
-                                    "\
+      if (!(up->is_bool
+            || up->uparams_offs == offsetof (struct uparams, rmargin)))
+        if (*(int *)((char *)upptr + up->uparams_offs) >= upptr->rmargin)
+          {
+            __argp_failure (state, 0, 0,
+                            dgettext (state == NULL ? NULL
+                                      : state->root_argp->argp_domain,
+                                      "\
 ARGP_HELP_FMT: %s value is less than or equal to %s"),
-                          "rmargin", up->name);
-          return;
-        }
+                            "rmargin", up->name);
+            return;
+          }
     }
   uparams = *upptr;
   uparams.valid = 1;