diff options
| author | Alejandro Colomar <alx@kernel.org> | 2024-11-16 16:51:31 +0100 |
|---|---|---|
| committer | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2024-11-25 16:45:59 -0300 |
| commit | 53fcdf5f743aa9b02972eec658e66f96d6a63386 (patch) | |
| tree | e1bd3ed90d89027abe4b8ba6f0dbffd833f08a9b /argp/argp-parse.c | |
| parent | 83d4b42ded712bbbc22ceeefe886b8315190da5b (diff) | |
| download | glibc-53fcdf5f743aa9b02972eec658e66f96d6a63386.tar.xz glibc-53fcdf5f743aa9b02972eec658e66f96d6a63386.zip | |
Silence most -Wzero-as-null-pointer-constant diagnostics
Replace 0 by NULL and {0} by {}.
Omit a few cases that aren't so trivial to fix.
Link: <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=117059>
Link: <https://software.codidact.com/posts/292718/292759#answer-292759>
Signed-off-by: Alejandro Colomar <alx@kernel.org>
Diffstat (limited to 'argp/argp-parse.c')
| -rw-r--r-- | argp/argp-parse.c | 44 |
1 files changed, 22 insertions, 22 deletions
diff --git a/argp/argp-parse.c b/argp/argp-parse.c index 9b7b91f65d..7a75fa3506 100644 --- a/argp/argp-parse.c +++ b/argp/argp-parse.c @@ -83,13 +83,13 @@ static volatile int _argp_hang; static const struct argp_option argp_default_options[] = { - {"help", '?', 0, 0, N_("Give this help list"), -1}, - {"usage", OPT_USAGE, 0, 0, N_("Give a short usage message")}, + {"help", '?', NULL, 0, N_("Give this help list"), -1}, + {"usage", OPT_USAGE, NULL, 0, N_("Give a short usage message")}, {"program-name",OPT_PROGNAME, N_("NAME"), OPTION_HIDDEN, N_("Set the program name")}, {"HANG", OPT_HANG, N_("SECS"), OPTION_ARG_OPTIONAL | OPTION_HIDDEN, N_("Hang for SECS seconds (default 3600)")}, - {0, 0} + {NULL, 0} }; static error_t @@ -149,8 +149,8 @@ static const struct argp argp_default_argp = static const struct argp_option argp_version_options[] = { - {"version", 'V', 0, 0, N_("Print program version"), -1}, - {0, 0} + {"version", 'V', NULL, 0, N_("Print program version"), -1}, + {NULL, 0} }; static error_t @@ -341,7 +341,7 @@ convert_options (const struct argp *argp, ? optional_argument : required_argument) : no_argument); - cvt->long_end->flag = 0; + cvt->long_end->flag = NULL; /* we add a disambiguating code to all the user's values (which is removed before we actually call the function to parse the value); this means that @@ -364,9 +364,9 @@ convert_options (const struct argp *argp, group->args_processed = 0; group->parent = parent; group->parent_index = parent_index; - group->input = 0; - group->hook = 0; - group->child_inputs = 0; + group->input = NULL; + group->hook = NULL; + group->child_inputs = NULL; if (children) /* Assign GROUP's CHILD_INPUTS field some space from @@ -382,7 +382,7 @@ convert_options (const struct argp *argp, parent = group++; } else - parent = 0; + parent = NULL; if (children) { @@ -417,7 +417,7 @@ parser_convert (struct parser *parser, const struct argp *argp, int flags) parser->argp = argp; if (argp) - parser->egroup = convert_options (argp, 0, 0, parser->groups, &cvt); + parser->egroup = convert_options (argp, NULL, 0, parser->groups, &cvt); else parser->egroup = parser->groups; /* No parsers at all! */ } @@ -530,7 +530,7 @@ parser_init (struct parser *parser, const struct argp *argp, makes very simple wrapper argps more convenient). */ group->child_inputs[0] = group->input; - err = group_parse (group, &parser->state, ARGP_KEY_INIT, 0); + err = group_parse (group, &parser->state, ARGP_KEY_INIT, NULL); } if (err == EBADKEY) err = 0; /* Some parser didn't understand. */ @@ -582,11 +582,11 @@ parser_finalize (struct parser *parser, group < parser->egroup && (!err || err==EBADKEY); group++) if (group->args_processed == 0) - err = group_parse (group, &parser->state, ARGP_KEY_NO_ARGS, 0); + err = group_parse (group, &parser->state, ARGP_KEY_NO_ARGS, NULL); for (group = parser->egroup - 1; group >= parser->groups && (!err || err==EBADKEY); group--) - err = group_parse (group, &parser->state, ARGP_KEY_END, 0); + err = group_parse (group, &parser->state, ARGP_KEY_END, NULL); if (err == EBADKEY) err = 0; /* Some parser didn't understand. */ @@ -625,7 +625,7 @@ parser_finalize (struct parser *parser, /* Since we didn't exit, give each parser an error indication. */ for (group = parser->groups; group < parser->egroup; group++) - group_parse (group, &parser->state, ARGP_KEY_ERROR, 0); + group_parse (group, &parser->state, ARGP_KEY_ERROR, NULL); } else /* Notify parsers of success, and propagate back values from parsers. */ @@ -636,14 +636,14 @@ parser_finalize (struct parser *parser, for (group = parser->egroup - 1 ; group >= parser->groups && (!err || err == EBADKEY) ; group--) - err = group_parse (group, &parser->state, ARGP_KEY_SUCCESS, 0); + err = group_parse (group, &parser->state, ARGP_KEY_SUCCESS, NULL); if (err == EBADKEY) err = 0; /* Some parser didn't understand. */ } /* Call parsers once more, to do any final cleanup. Errors are ignored. */ for (group = parser->egroup - 1; group >= parser->groups; group--) - group_parse (group, &parser->state, ARGP_KEY_FINI, 0); + group_parse (group, &parser->state, ARGP_KEY_FINI, NULL); if (err == EBADKEY) err = EINVAL; @@ -682,7 +682,7 @@ parser_parse_arg (struct parser *parser, char *val) { parser->state.next--; /* For ARGP_KEY_ARGS, put back the arg. */ key = ARGP_KEY_ARGS; - err = group_parse (group, &parser->state, key, 0); + err = group_parse (group, &parser->state, key, NULL); } } @@ -792,11 +792,11 @@ parser_parse_next (struct parser *parser, int *arg_ebadkey) parser->opt_data.optopt = KEY_END; if (parser->state.flags & ARGP_LONG_ONLY) opt = _getopt_long_only_r (parser->state.argc, parser->state.argv, - parser->short_opts, parser->long_opts, 0, + parser->short_opts, parser->long_opts, NULL, &parser->opt_data); else opt = _getopt_long_r (parser->state.argc, parser->state.argv, - parser->short_opts, parser->long_opts, 0, + parser->short_opts, parser->long_opts, NULL, &parser->opt_data); /* And see what getopt did. */ parser->state.next = parser->opt_data.optind; @@ -893,7 +893,7 @@ __argp_parse (const struct argp *argp, int argc, char **argv, unsigned flags, child[child_index++].argp = &argp_default_argp; if (argp_program_version || argp_program_version_hook) child[child_index++].argp = &argp_version_argp; - child[child_index].argp = 0; + child[child_index].argp = NULL; argp = &top_argp; } @@ -930,7 +930,7 @@ __argp_input (const struct argp *argp, const struct argp_state *state) return group->input; } - return 0; + return NULL; } #ifdef weak_alias weak_alias (__argp_input, _argp_input) |
