On Thu 20-01-11 19:27:16, Jeff King wrote: > The intent of the change looks fine, but your sentence doesn't quite > parse to me (to be fair, the problem is in the one you are replacing, > but adding the third clause makes it even more confusing). How about: > > The variables names are case-insensitive, allow only alphanumeric > characters and '-', and must start with an alphabetic character. I like that very much. The original sentence sounded a little bit artificial to me, and after my amendments it felt like a collapsed list of items. This is far more natural, thanks for help. > > --get-regexp:: > > Like --get-all, but interprets the name as a regular expression. > > - Also outputs the key names. > > + Regular expression matching is case sensitive in all parts of the key, > > + therefore make sure your pattern matches lower case letters in section > > + and variable names. Also outputs the key names. > > That is only true because of the breakage in your first patch. Without > your patch, both of these work: > > git config --get-regexp 'Foo.*' > git config --get-regexp 'foo.*' > > That being said, the downcasing is extremely naive for regexps, and you > should try to match the canonical name. The current downcasing behavior > should probably stay for historical reasons, but is not well thought-out > (it may even be accidental). Perhaps we should therefore explain it in > those terms: > > Regular expression matching is case-sensitive and done against a > canonicalized version of the key in which section and variable names > are lowercased, but subsection names are not. For historical reasons, > some simple regular expressions are lower-cased before matching > (everything before the first dot and after the last dot), which makes > things like "Core.*' work. > > I dunno. Maybe we should just declare "Core.*' to be broken, and anybody > who was relying on it is wrong. After thinking about it more, I realized that the goal is to have a "mixed case-sensitivity" regexp matching. The case sensitivity if of course determined by the part of the key which is being processed, and not the pattern itself. That probably means changes in the regexp mathing sengine. Declaring patterns like 'Core.*' invalid would be a step back in my opinion. Therefore I've added the regexp lowercasing code back and documented the limitation. I believe it's fair to the users. Updated patches will follow in a while. Libor -- Libor Pechacek SUSE L3 Team, Prague -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html