Andrew Dunstan <andrew@dunslane.net> writes:
> Tom Lane wrote:
>> You have to be on a first byte before you can meaningfully apply
>> NextChar, and you have to use NextChar or else you don't count
>> characters correctly (eg "__" must match 2 chars not 2 bytes).
> Yes, I agree completely. However it looks to me like IsFirstByte will in
> fact always be true when we get to call NextChar for matching "_" for UTF8.
If that's true, the patch is failing to achieve its goal of treating %
bytewise ...
regards, tom lane