On Fri, Jul 27, 2018 at 2:39 PM Junio C Hamano <gits...@pobox.com> wrote: > > Brandon Williams <bmw...@google.com> writes: > > > Either way you're probably going to need to be careful about how you do > > string comparison against the different parts. > > Good suggestion.
The suggestion is a rabit hole and was a waste of time. However I did some more manual testing and inspected the code with trace_printf debugging, and it turns out the strings compared are brought into the correct form already. > >> + # v.a.r and v.A.r are not the same variable, as the middle > >> + # level of a three-level configuration variable name is > >> + # case sensitive. > > In other words, perhaps add > > # "V.a.r" and "v.a.R" are the same variable, though > > and corresponding test here? I removed that section and went for a shorter, more concise expression. patch to follow.
