Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:85700
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain lerdorf.com designates 209.85.192.170 as permitted sender)
Message-ID: <551DBE5C.8040402@lerdorf.com>
Date: Thu, 02 Apr 2015 15:10:36 -0700
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0
MIME-Version: 1.0
To: Dan Ackroyd <danack@basereality.com>, 
 Ryan Pallas <derokorian@gmail.com>
CC: Adam Harvey <aharvey@php.net>, Keyur Govande <keyurgovande@gmail.com>, 
 "internals@lists.php.net" <internals@lists.php.net>
References: <CA+kxMuSvxRq-AvjQSBUc-FiYdTrD_1X2SuQtjWRjTwAbcwWRoA@mail.gmail.com>	<551CEDF6.7090408@lerdorf.com>	<CAJhmKHkyJN8E7RGOo19bYVQ5JjPaJcW8iJ+ad4VPzGiGC54+uA@mail.gmail.com>	<CA+kxMuR3NjCGf+Xz1w+NDR0hPAYS=LCDkJMG_p_ZNK4rZmJ7=Q@mail.gmail.com>	<CADajX4DqVQP0rGbLzu1YVrcamyYWj0JqoiDryWPuenQCQtj6wg@mail.gmail.com>	<CAObuZduomooUsjsTrvH63HrFp4z7GsZQtBvptC3XzakPD=9-jg@mail.gmail.com> <CA+kxMuSCLo2VnGnHngW7WEhSjGuEH=+DiLb=ngYVDYHhNQ2Zfg@mail.gmail.com>
In-Reply-To: <CA+kxMuSCLo2VnGnHngW7WEhSjGuEH=+DiLb=ngYVDYHhNQ2Zfg@mail.gmail.com>
Content-Type: multipart/signed; micalg=pgp-sha1;
 protocol="application/pgp-signature";
 boundary="doAaWI781ScF2mLpoRd2XldUmmexcUHej"
Subject: Re: [PHP-DEV] Deprecate setlocale?
From: rasmus@lerdorf.com (Rasmus Lerdorf)

--doAaWI781ScF2mLpoRd2XldUmmexcUHej
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

On 04/02/2015 02:13 PM, Dan Ackroyd wrote:
> Adam Harvey wrote:
>> What about just adding another function: setlocale_global(), or simila=
r?
>=20
> Do you mean setlocale would become the thread safe one, and
> setlocale_global() would be the current behaviour? If so, that would
> be a BC break for the small number of people who are deliberately
> using setlocale globally. We could do that at a major version I guess.
>=20
> I think the ini setting is still better - it allows people to use code
> that was written assuming that setlocale was safe to use, rather than
> forcing them to change it.
>=20
> It might be best to introduce both setlocale_ts() and
> setlocale_global(), so that users could call either explicitly, and
> have the ini setting control which one setlocale() points to. If we
> did that we could also then plan to eventually deprecate both the ini
> setting and the plain setlocale without _ts/_global.
>=20
> Ryan Pallas wrote:
>> I like the idea of an INI actually, but I would make it default to the=
 current behaviour and user education so those who need TS know to change=
 it.
>=20
> The problem with that is that for people who aren't aware of the
> setting, they wouldn't know that they need to change the setting.
> Instead they just want their code to work properly 'out of the box'.
> For the vast majority of people calling the thread safe version of
> setlocale would be what they problem want it to do, and so PHP should
> default to the right thing for the majority of people.
>=20
> Of course there is a separate argument about changing the behaviour at
> major/minor version, and if this was introduced then having to
> `thread_safe_setlocale=3Doff` for 7.1 and `thread_safe_setlocale=3Don` =
for
> 8 could be correct.

A step back here, please. Which concrete problem are you actually trying
to solve? HHVM needed this fix because of its single-process threaded
architecture on UNIX. PHP is not typically threaded on UNIX, so it
doesn't seem like an issue affecting very many people. It has been on
our radar for ages and has a big prominent red warning box about it in
the documentation. Threaded use of PHP on UNIX tends to be rather
specialized cases by people who understand that things like setlocale()
and putenv() are process-wide.

That doesn't mean there is anything wrong with implementing thread-local
setlocale for our threadsafe builds, and we should definitely fix any
in-process setlocale interference issues, but the thread safety part of
it just doesn't seem like something very many people are going to care
about. Why do you?

-Rasmus



--doAaWI781ScF2mLpoRd2XldUmmexcUHej
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iEYEARECAAYFAlUdvlwACgkQlxayKTuqOuDh8wCggZO/Ih/0WTFyrX5K2721IL1C
9MsAn0cRIBHr7KtvKdpG/++Rr4gWIG1g
=NCZC
-----END PGP SIGNATURE-----

--doAaWI781ScF2mLpoRd2XldUmmexcUHej--