(isunordered): Rename local variables to ensure correct code.
[kopensolaris-gnu/glibc.git] / argp / argp-help.c
index 2120ab1..8faacd5 100644 (file)
@@ -778,7 +778,7 @@ hol_append (struct hol *hol, struct hol *more)
        more->num_entries = 0;  /* Mark MORE's fields as invalid.  */
       }
     else
-      /* append the entries in MORE to those in HOL, taking care to only add
+      /* Append the entries in MORE to those in HOL, taking care to only add
         non-shadowed SHORT_OPTIONS values.  */
       {
        unsigned left;
@@ -791,12 +791,12 @@ hol_append (struct hol *hol, struct hol *more)
        char *short_options =
          malloc (hol_so_len + strlen (more->short_options) + 1);
 
-       memcpy (entries, hol->entries,
-               hol->num_entries * sizeof (struct hol_entry));
-       memcpy (entries + hol->num_entries, more->entries,
-               more->num_entries * sizeof (struct hol_entry));
+       __mempcpy (__mempcpy (entries, hol->entries,
+                             hol->num_entries * sizeof (struct hol_entry)),
+                  more->entries,
+                  more->num_entries * sizeof (struct hol_entry));
 
-       memcpy (short_options, hol->short_options, hol_so_len);
+       __mempcpy (short_options, hol->short_options, hol_so_len);
 
        /* Fix up the short options pointers from HOL.  */
        for (e = entries, left = hol->num_entries; left > 0; e++, left--)
@@ -922,7 +922,7 @@ filter_doc (const char *doc, int key, const struct argp *argp,
     }
   else
     /* No filter.  */
-    return (char *)doc;
+    return doc;
 }
 
 /* Prints STR as a header line, with the margin lines set appropiately, and
@@ -1319,7 +1319,8 @@ argp_args_usage (const struct argp *argp, const struct argp_state *state,
 
   if (fdoc)
     {
-      nl = strchr (fdoc, '\n');
+      const char *cp = fdoc;
+      nl = strchr (cp, '\n');
       if (nl)
        /* This is a `multi-level' args doc; advance to the correct position
           as determined by our state in LEVELS, and update LEVELS.  */
@@ -1327,17 +1328,17 @@ argp_args_usage (const struct argp *argp, const struct argp_state *state,
          int i;
          multiple = 1;
          for (i = 0; i < *our_level; i++)
-           fdoc = nl + 1, nl = strchr (fdoc, '\n');
+           cp = nl + 1, nl = strchr (cp, '\n');
          (*levels)++;
        }
       if (! nl)
-       nl = fdoc + strlen (fdoc);
+       nl = cp + strlen (cp);
 
       /* Manually do line wrapping so that it (probably) won't get wrapped at
         any embedded spaces.  */
-      space (stream, 1 + nl - fdoc);
+      space (stream, 1 + nl - cp);
 
-      __argp_fmtstream_write (stream, fdoc, nl - fdoc);
+      __argp_fmtstream_write (stream, cp, nl - cp);
     }
   if (fdoc && fdoc != tdoc)
     free ((char *)fdoc);       /* Free user's modified doc string.  */
@@ -1499,7 +1500,7 @@ _help (const struct argp *argp, const struct argp_state *state, FILE *stream,
       do
        {
          int old_lm;
-         int old_wm = __argp_fmtstream_set_wmargin (fs, USAGE_INDENT);
+         int old_wm = __argp_fmtstream_set_wmargin (fs, uparams.usage_indent);
          char *levels = pattern_levels;
 
          __argp_fmtstream_printf (fs, "%s %s",
@@ -1508,7 +1509,7 @@ _help (const struct argp *argp, const struct argp_state *state, FILE *stream,
 
          /* We set the lmargin as well as the wmargin, because hol_usage
             manually wraps options with newline to avoid annoying breaks.  */
-         old_lm = __argp_fmtstream_set_lmargin (fs, USAGE_INDENT);
+         old_lm = __argp_fmtstream_set_lmargin (fs, uparams.usage_indent);
 
          if (flags & ARGP_HELP_SHORT_USAGE)
            /* Just show where the options go.  */