(isunordered): Rename local variables to ensure correct code.
[kopensolaris-gnu/glibc.git] / argp / argp-help.c
index 0d94441..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.  */