diff options
author | Junio C Hamano <gitster@pobox.com> | 2017-02-23 15:04:40 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-02-23 22:03:03 -0800 |
commit | 1274a155af96fcbcf2e60ae9fc9ee8dbdbbe987a (patch) | |
tree | 3ad3cc57445fa68f74fd2c9d694fc6bf8b527190 /t/t5100/patch0014--scissors | |
parent | config: move a few helper functions up (diff) | |
download | tgif-1274a155af96fcbcf2e60ae9fc9ee8dbdbbe987a.tar.xz |
config: use git_config_parse_key() in git_config_parse_parameter()
The parsing of one-shot assignments of configuration variables that
come from the command line historically was quite loose and allowed
anything to pass. It also downcased everything in the variable name,
even a three-level <section>.<subsection>.<variable> name in which
the <subsection> part must be treated in a case sensitive manner.
Existing git_config_parse_key() helper is used to parse the variable
name that comes from the command line, i.e. "git config VAR VAL",
and handles these details correctly. Replace the strbuf_tolower()
call in git_config_parse_parameter() with a call to it to correct
both issues. git_config_parse_key() does a bit more things that are
not necessary for the purpose of this codepath (e.g. it allocates a
separate buffer to return the canonicalized variable name because it
takes a "const char *" input), but we are not in a performance-critical
codepath here.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t5100/patch0014--scissors')
0 files changed, 0 insertions, 0 deletions