Subject: Re: [PATCH/RFC 2/2] git rebase -i: Warn removed or dupplicated commits

From: Junio C Hamano <[email-masked]>
Date: Wed, 27 May 2015 12:18:10 -0700

Remi Galan Alfonso <[email-masked]>
writes:

> Thank you for reviewing the code. 
>
> Eric Sunshine<[email-masked]> writes:
>> > +       # To uppercase
>> > +       checkLevel=$(echo "$checkLevel" | tr '[:lower:]' '[:upper:]')
>> 
>> Is there precedence elsewhere for recognizing uppercase and lowercase
>> variants of config values?
>
> It seems to be commonly used when parsing options in the C files
> through strcasecmp.  For exemple, in config.c:818 :
> if (!strcmp(var, "core.safecrlf")) {
> 	if (value && !strcasecmp(value, "warn")) {
> 		[...]
> However we didn't see any precedence in shell files. Do you think we
> should remove it?

I think there is a difference between (silently) accepting just to
be lenient and documenting and advocating mixed case uses.

Personally, I'd rather not to see gratuitous flexibility to allow
the same thing spelled in 47 different ways for no good reason.