DMARC-Filter:	OpenDMARC Filter v1.4.2 delorie.com 52P95PKk2598300
Authentication-Results:	delorie.com; dmarc=pass (p=none dis=none) header.from=cygwin.com
Authentication-Results:	delorie.com; spf=pass smtp.mailfrom=cygwin.com
DKIM-Filter:	OpenDKIM Filter v2.11.0 delorie.com 52P95PKk2598300
Authentication-Results:	delorie.com;
	dkim=pass (1024-bit key, unprotected) header.d=cygwin.com header.i=@cygwin.com header.a=rsa-sha256 header.s=default header.b=tWyxfzo3
X-Recipient:	archive-cygwin AT delorie DOT com
DKIM-Filter:	OpenDKIM Filter v2.11.0 sourceware.org A7D6F385771D
DKIM-Signature:	v=1; a=rsa-sha256; c=relaxed/relaxed; d=cygwin.com;
	s=default; t=1742893523;
	bh=qddf2Re7GqUS1oDHCjihlXsgHs3RK5YfPbtcK+iobAo=;
	h=Date:To:Subject:References:In-Reply-To:List-Id:List-Unsubscribe:
	List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc:
	From;
	b=tWyxfzo36MuqBBpa4VUIhF16zZAttlsI6LhxK2zfxqHMW6JmCkrBJYJ81Grv4iAFs
	p3pwzJEPT/QuwBu1ZX+wwXl8T7aSpUZzhgnHAxa0WFxcAx2v9pXLDhnLOLJrAmAXrV
	kUc7Vn+9VD0prILTQSLqKWiEFF5Kc2ayyJ0CZBN8=
X-Original-To:	cygwin AT cygwin DOT com
Delivered-To:	cygwin AT cygwin DOT com
DKIM-Filter:	OpenDKIM Filter v2.11.0 sourceware.org A19043857820
Date:	Tue, 25 Mar 2025 10:04:20 +0100
To:	cygwin AT cygwin DOT com
Subject:	Re: two bugs in getlocalename_l
Message-ID:	<Z-JxlLL2eWjAUMxQ@calimero.vinschen.de>
Mail-Followup-To:	cygwin AT cygwin DOT com, Bruno Haible <bruno AT clisp DOT org>
References:	<4134860 DOT iTQEcLzFEP AT nimes> <Z-GSfgjYaICmrYQY AT calimero DOT vinschen DOT de>
MIME-Version:	1.0
In-Reply-To:	<Z-GSfgjYaICmrYQY@calimero.vinschen.de>
X-BeenThere:	cygwin AT cygwin DOT com
X-Mailman-Version:	2.1.30
List-Id:	General Cygwin discussions and problem reports <cygwin.cygwin.com>
List-Unsubscribe:	<https://cygwin.com/mailman/options/cygwin>,
	<mailto:cygwin-request AT cygwin DOT com?subject=unsubscribe>
List-Archive:	<https://cygwin.com/pipermail/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-request AT cygwin DOT com?subject=help>
List-Subscribe:	<https://cygwin.com/mailman/listinfo/cygwin>,
	<mailto:cygwin-request AT cygwin DOT com?subject=subscribe>
From:	Corinna Vinschen via Cygwin <cygwin AT cygwin DOT com>
Reply-To:	cygwin AT cygwin DOT com
Cc:	Corinna Vinschen <corinna-cygwin AT cygwin DOT com>,
	Bruno Haible <bruno AT clisp DOT org>
Errors-To:	cygwin-bounces~archive-cygwin=delorie DOT com AT cygwin DOT com
Sender:	"Cygwin" <cygwin-bounces~archive-cygwin=delorie DOT com AT cygwin DOT com>

Hi Bruno,

On Mar 24 18:12, Corinna Vinschen via Cygwin wrote:
> On Mar 24 14:12, Bruno Haible via Cygwin wrote:
> > Hi,
> > 
> > It's good to see the addition of getlocalename_l from POSIX [1] in Cygwin 3.6.0.
> > 
> > [1] https://pubs.opengroup.org/onlinepubs/9799919799/functions/getlocalename_l.html
> > 
> > However, there are two bugs:
> > 
> > 
> >   - "If
> >      the value is LC_ALL or is not a supported locale category value (see
> >      <<setlocale>>), <<getlocalename_l>> shall fail."
> >     No, that's now how it is specified.
> >     <https://www.austingroupbugs.net/view.php?id=1220> specified it this
> >     way, but <https://www.austingroupbugs.net/view.php?id=1741> added the
> >     LC_ALL support.
> 
> Drat.  I implemented this prior to id=1741.  This is unfortunate.
> 
> Without LC_ALL, only a single per-thread buffer of size ENCODING_LEN+1
> was required to allow LC_GLOBAL_LOCALE.
> 
> With LC_ALL and with the extra requirement that only a call to
> getlocalename_l(LC_GLOBAL_LOCALE) overwrites the per-thread
> LC_GLOBAL_LOCALE string buffer, we now need 2 per-thread buffers 
> of size _LC_LAST * (ENCODING_LEN+1).
> 
> >   - "<<getlocalename_l>> is POSIX-1.2008 since Base Specification Issue 8"
> >     No, it was introduced in POSIX-1.2024.
> 
> Yeah, that was temporary because the nameof the new standard wasn't
> known at the time I added this.

I fixed the comment and the invalid return value when specifying
LC_GLOBAL_LOCALE, as well as adding LC_ALL handling per the POSIX
specs.

The changes will be in test release cygwin-3.7.0-0.14.ga0fe984953dd
and in 3.6.1.


Thanks,
Corinna

-- 
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple

- Raw text -