Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> On Thu, Jun 9, 2011 at 10:15 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> If we need to work around brain-dead isupper() tests, maybe the best
>> thing is to implement two versions of the loop:
>>
>> if (encoding is single byte)
>> ... loop as it stands ...
>> else
>> ... loop without the "else if" part
> That seems like a clear improvement. It's a long way from perfect,
> but still worthwhile.
> Would we back-patch that? Just do it in master? Wait for 9.2?
It looks to me like a portability bug fix, so I'd say back-patch.
I'll take care of it, unless you're hot to do so?
regards, tom lane