diff options
| author | D. Ben Knoble <ben.knoble+github@gmail.com> | 2025-11-02 11:17:48 -0500 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2025-11-04 09:25:52 -0800 |
| commit | 383e5e1c4bfa604bcd479100258b4ff354dbaabb (patch) | |
| tree | 2116e9d7f8d2d3a6ba5f0b077bb187e078e53245 | |
| parent | 4dbb7f4f820ca392699bac884311e2c9204cddcb (diff) | |
| download | git-383e5e1c4bfa604bcd479100258b4ff354dbaabb.tar.gz | |
parseopt: restore const qualifier to parsed filename
This was unintentionally dropped in ccfcaf399f (parseopt: values of
pathname type can be prefixed with :(optional), 2025-09-28). Notably,
continue dropping the const qualifier when free'ing value; see
4049b9cfc0 (fix const issues with some functions, 2007-10-16) or
83838d5c1b (cast variable in call to free() in builtin/diff.c and
submodule.c, 2011-11-06) for more details on why.
Suggested-by: Phillip Wood <phillip.wood@dunelm.org.uk>
Signed-off-by: D. Ben Knoble <ben.knoble+github@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | parse-options.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/parse-options.c b/parse-options.c index b105e4b6cf..27c1e75d53 100644 --- a/parse-options.c +++ b/parse-options.c @@ -213,7 +213,7 @@ static enum parse_opt_result do_get_value(struct parse_opt_ctx_t *p, if (unset) value = NULL; else if (opt->flags & PARSE_OPT_OPTARG && !p->opt) - value = (char *)opt->defval; + value = (const char *)opt->defval; else { int err = get_arg(p, opt, flags, &value); if (err) |