fix(cli): avoid shadowing global optarg variable
Rename parameter 'optarg' to 'arg' in handle_one_option() to fix -Wshadow warning.
This commit is contained in:
parent
44cd05a499
commit
d0444f9431
1 changed files with 3 additions and 3 deletions
|
|
@ -10,7 +10,7 @@ static void init_config(t_ping_config *config);
|
||||||
static void build_long_options(struct option *long_opts, size_t nb_opts,
|
static void build_long_options(struct option *long_opts, size_t nb_opts,
|
||||||
const t_option_descriptor *src);
|
const t_option_descriptor *src);
|
||||||
static const t_option_descriptor *find_option_handler(int opt);
|
static const t_option_descriptor *find_option_handler(int opt);
|
||||||
static int handle_one_option(int opt, char *optarg, t_ping_config *config);
|
static int handle_one_option(int opt, char *arg, t_ping_config *config);
|
||||||
/* ------------------- */
|
/* ------------------- */
|
||||||
|
|
||||||
int
|
int
|
||||||
|
|
@ -33,7 +33,7 @@ cli_parse_arguments(int argc, char **argv, t_ping_config *config)
|
||||||
}
|
}
|
||||||
|
|
||||||
static int
|
static int
|
||||||
handle_one_option(int opt, char *optarg, t_ping_config *config)
|
handle_one_option(int opt, char *arg, t_ping_config *config)
|
||||||
{
|
{
|
||||||
const t_option_descriptor *desc = find_option_handler(opt);
|
const t_option_descriptor *desc = find_option_handler(opt);
|
||||||
int res;
|
int res;
|
||||||
|
|
@ -42,7 +42,7 @@ handle_one_option(int opt, char *optarg, t_ping_config *config)
|
||||||
return CLI_ERROR;
|
return CLI_ERROR;
|
||||||
if (NULL != desc)
|
if (NULL != desc)
|
||||||
{
|
{
|
||||||
res = desc->handler(optarg, config);
|
res = desc->handler(arg, config);
|
||||||
if (0 != res)
|
if (0 != res)
|
||||||
return res;
|
return res;
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue