On 6/10/23 15:07, Joe Conway wrote:
> On 6/10/23 14:42, Tom Lane wrote:
>> Joe Conway <mail@joeconway.com> writes:
>>> 5/ The attached fixes the issue for me on pg10 and passes check-world.
>>> Comments?
>>
>> The call in PGLC_localeconv seems *very* oddly placed. Why not
>> do that before it does any other locale calls? Otherwise you don't
>> really have reason to believe you're saving the appropriate
>> values to restore later.
>
>
> As far as I can tell it really only affects localeconv(), so I tried to
> place it close to those. But I am fine with moving it up.
This version is against pg16 (rather than pg10), moves up that hunk,
mentions localeconv() in the comment as the reason for the call, and
fixes some whitespace sloppiness. I will plan to apply to all supported
branches.
Better?
--
Joe Conway
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com