On Thu, May 30, 2013 at 5:29 AM, Anthony Ramine <n.ox...@gmail.com> wrote:
> Yes indeed. Will amend. Should I add your name in Reviewed-by as well?

No. I merely spotted a minor typographical error.

> --
> Anthony Ramine
>
> Le 30 mai 2013 à 11:07, Eric Sunshine a écrit :
>
>> On Thu, May 30, 2013 at 4:45 AM, Anthony Ramine <n.ox...@gmail.com> wrote:
>>> Case folding is not done correctly when matching against the [:upper:]
>>> character class and uppercased character ranges (e.g. A-Z).
>>> Specifically, an uppercase letter fails to match against any of them
>>> when case folding is requested because plain characters in the pattern
>>> and the whole string and preemptively lowercased to handle the base case
>>
>> Did you mean s/and preemptively/are preemptively/ ?
>>
>>> fast.
>>>
>>> That optimization is kept and ISLOWER() is used in the [:upper:] case
>>> when case folding is requested, while matching against a character range
>>> is retried with toupper() if the character was lowercase, as the bounds
>>> of the range itself cannot be modified (in a case-insensitive context,
>>> [A-_] is not equivalent to [a-_]).
>>>
>>> Signed-off-by: Anthony Ramine <n.ox...@gmail.com>
>
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to