On 19.10.25 02:29, Jeff Davis wrote:
> On Fri, 2025-10-17 at 18:15 +0200, Peter Eisentraut wrote:
>>
>> This is indeed a bit mysterious. AFAICT, the behavior you describe
>> is
>> conditional on if (prs->usewide), so it apparently depends also on
>> the
>> encoding? I'm not sure if the new code covers this.
>
> I believe the new code does cover this case:
>
> Previously, the code was effectively:
> if (prs->usewide && prs->pgwstr != NULL && c > 0x7f)
> retirm nonascii
>
> and the new code is:
> if (prs->charmaxlen > 1 && locale->ctype_is_c && wc > 0x7f)
> return nonascii;
>
> unless I missed something, those are equivalent.
Yes, this looks ok.