Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:116072
MIME-Version: 1.0
References: <b61f3e9e-c0f3-a490-6750-b9b480cae539@wikimedia.org>
 <CAF+90c8-=CpNh+Gczw35wdXKGzu4PdQvqZCSqWYxwKhx1yzymQ@mail.gmail.com>
 <CAGBsUrfYEh9L8-eSscbXbQw4goc8SZg_WWWUpHWPcNoAzz9Qaw@mail.gmail.com> <e9ffbf41-c83f-b7f6-a281-b50b067cc916@wikimedia.org>
In-Reply-To: <e9ffbf41-c83f-b7f6-a281-b50b067cc916@wikimedia.org>
Date: Fri, 17 Sep 2021 15:23:56 +0200
Message-ID: <CAF+90c8ys40wfhvfhdFDskbXvy+Lcrs6v8+pm=zcZ85nkRf4UA@mail.gmail.com>
To: Tim Starling <tstarling@wikimedia.org>
Cc: Kamil Tekiela <tekiela246@gmail.com>, 
	"internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000dd09a205cc30d80d"
Subject: Re: [PHP-DEV] Make strtolower/strtoupper just do ASCII
From: nikita.ppv@gmail.com (Nikita Popov)

--000000000000dd09a205cc30d80d
Content-Type: text/plain; charset="UTF-8"

On Fri, Sep 17, 2021 at 12:07 PM Tim Starling <tstarling@wikimedia.org>
wrote:

> On 17/9/21 7:15 pm, Kamil Tekiela wrote:
> > +1 from me. I wasn't even aware that these functions are
> > locale-dependent until recently. I see an added benefit that we could
> > add them to the optimizer once they are no longer locale-dependent.
> > What would happen to users who really need the locale-dependent
> > functions? Do we offer some workarounds?
>
> We could add a global mode, although that would prevent constant
> propagation, if that's what you mean by adding them to the optimizer.
> Or we could add variant functions like locale_strtolower() and
> locale_strtoupper(). But I think I would want to hear from someone who
> uses locale-dependence so I can understand what their needs are. I
> guess the RFC will sort that out.
>

I would expect that in nearly all cases the replacement would be one of
these:
1. You were using an UTF-8 locale (which you likely are), then just keep
using strtolower(). Without having checked all the details here, I think
strtolower() under UTF-8 locales already effectively behaves like ASCII
lowercase, because it skips continuation bytes.
2. If you were using some other charset, then using mb_strtolower() with
that charset should work. So if you were using de_DE.ISO8859-1, then using
mb_strtolower() with "ISO8859-1" encoding would be the replacement.

As a matter of general policy, it is unlikely that we will accept an option
(whether that be an ini option or something else) to control this behavior.
We can make the change or not make it, but not both ;)

Regards,
Nikita

--000000000000dd09a205cc30d80d--