Re: Thread-safe nl_langinfo() and localeconv()

On 14.02.25 15:13, Peter Eisentraut wrote:
> On 09.02.25 15:52, Peter Eisentraut wrote:
>> This patch set is still desirable.  Here is a rebased version of the 
>> v5 patches.  I haven't applied any corrections or review comments.
> 
> Here is the same patch set with some review comments.
> 
> Patch 0001 looks okay to me.  I'm just offering some cosmetic 
> improvements in patch 0004.

I have committed this patch.

The original patch had a typo that prevented the BSD-ish branch (using 
localeconv_l()) from being taken, so it actually used the fallback 
uselocale() branch, which then failed on CI on NetBSD.  (But conversely, 
this gave some testing on CI for the uselocale() branch.)

> Patch 0002 also looks okay, except that the error handling could be 
> unified with existing code, as I had previously pointed out.  Patch 0005 
> fixes that.

I plan to commit this one next, after the above had a bit of time to stew.

> About patch 0003:
> 
> I had previously pointed out that the canonicalization might have been 
> intentional, and that we have canonicalization of ICU locale names.  But 
> we don't have to keep the setlocale()-based locale checking 
> implementation just for that, I think.  (If this was meant to be a real 
> feature offered by libc, there should be a get_locale_name(locale_t) 
> function.)

POSIX 2024 actually has getlocalename_l(), but it doesn't appear to be 
widely implemented.

Anyway, this patch fails tests on CI on NetBSD, so it will need some 
further investigation.