Fix: lttng: -Wshadow error in cmd_snapshot
authorFrancis Deslauriers <francis.deslauriers@efficios.com>
Wed, 14 Apr 2021 14:12:04 +0000 (10:12 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Mon, 19 Apr 2021 15:15:29 +0000 (11:15 -0400)
The optarg variable name shadows a variable in an external dependency.
clang returns the following error on my machine:
  commands/snapshot.c:627:16: error: declaration shadows a variable in the global scope [-Werror,-Wshadow]
                          const char *optarg = poptGetOptArg(pc);
                                      ^
  /usr/include/x86_64-linux-gnu/bits/getopt_core.h:36:14: note: previous declaration is here
  extern char *optarg;

Signed-off-by: Francis Deslauriers <francis.deslauriers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: I85dd7b0761ebc2c023d84ba869c0551f91f38a17

src/bin/lttng/commands/snapshot.c

index 8f44b9f1c189aeb893e3f8baf3883177fb880629..513257a3c43f8bf8785f60b5ec52fddbf1714d36 100644 (file)
@@ -624,10 +624,11 @@ int cmd_snapshot(int argc, const char **argv)
                case OPT_MAX_SIZE:
                {
                        uint64_t val;
-                       const char *optarg = poptGetOptArg(pc);
+                       const char *max_size_arg = poptGetOptArg(pc);
 
-                       if (utils_parse_size_suffix((char *) optarg, &val) < 0) {
-                               ERR("Unable to handle max-size value %s", optarg);
+                       if (utils_parse_size_suffix((char *) max_size_arg, &val) < 0) {
+                               ERR("Unable to handle max-size value %s",
+                                               max_size_arg);
                                cmd_ret = CMD_ERROR;
                                goto end;
                        }
This page took 0.027116 seconds and 4 git commands to generate.