Re: [PATCH v8 03/12] Move lower case functions into wrapper.c

On Thu, Mar 27, 2014 at 03:47:01PM -0700, Junio C Hamano wrote:

Actually, I think it ends up being hostile to the users to accept
random cases without a good reason.  If you see two trailer elements
whose where are specified as "after" and "AFTER" in somebody's
configuration file, wouldn't that give a wrong impression that a new
line with the latter somehow has a stronger desire to come later
than the former?

If you consistently take only the fixed strings, you do not have to
worry about many people writing the same things in different ways,
confusing each other.

I do not agree with this line of reasoning at all. After all, do we have
confusion about the case differences between:

  [COLOR]
  diff = true

  [color]
  UI = false

But I also do not overly care. Literally zero people have complained
that "[log]date = RFC822" is not accepted, so it is probably not a big
deal either way.

-Peff