]> git.ipfire.org Git - thirdparty/coreutils.git/commitdiff
build: numfmt.c: avoid a shadowing warning
authorJim Meyering <meyering@fb.com>
Mon, 29 Jun 2015 17:25:07 +0000 (10:25 -0700)
committerJim Meyering <meyering@fb.com>
Mon, 29 Jun 2015 18:11:49 +0000 (11:11 -0700)
* src/numfmt.c (parse_field_arg): Rename parameter s/optarg/arg/,
to avoid shadowing getopt's global variable.
Otherwise, building on OS X, with --enable-gcc-warnings, I saw this:

  In file included from src/numfmt.c:19:0:
  src/numfmt.c: In function 'parse_field_arg':
  ./lib/config.h:3109:25: error: declaration of 'rpl_optarg' shadows\
    a global declaration [-Werror=shadow]

src/numfmt.c

index 09cdd422f194435f2af92f8fce62ec4bb7b053b1..e58972bce62565c94c6195790fc27d972f52e352 100644 (file)
@@ -1354,7 +1354,7 @@ free_field (const void *elt)
    TODO: Refactor the more performant cut implementation
    for use by both utilities.  */
 static void
-parse_field_arg (char *optarg)
+parse_field_arg (char *arg)
 {
 
   char *start, *end;
@@ -1362,9 +1362,9 @@ parse_field_arg (char *optarg)
   size_t field_val;
   size_t range_val = 0;
 
-  start = end = optarg;
+  start = end = arg;
 
-  if (STREQ (optarg, "-"))
+  if (STREQ (arg, "-"))
     {
       all_fields = true;
 
@@ -1438,7 +1438,7 @@ parse_field_arg (char *optarg)
   if (range_val)
     {
       /* range N-
-         range_val was not reset indicating optarg
+         range_val was not reset indicating ARG
          ended with a trailing '-' */
       all_fields_after = range_val;
     }