Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:130309
Feedback-ID: i8414410d:Fastmail
Precedence: list
MIME-Version: 1.0
Date: Fri, 13 Mar 2026 13:11:31 -0500
To: "php internals" <internals@lists.php.net>
Message-ID: <98259e0e-22e6-48ca-b520-b91ae8072e11@app.fastmail.com>
In-Reply-To: <58700a4c-7d1b-4e4b-b6f9-adc5843b0206@bastelstu.be>
References: <5bec7fdd-e2d0-47ab-92d0-ead8eade21e6@bastelstu.be>
 <AM8P250MB01705C4FC45781D33C5A110EE27EA@AM8P250MB0170.EURP250.PROD.OUTLOOK.COM>
 <CF0E415D-EA81-404D-AE3A-2DF21AACDA3C@php.net>
 <0fdf66b9-2027-444b-bee5-01bed296e3c9@app.fastmail.com>
 <58700a4c-7d1b-4e4b-b6f9-adc5843b0206@bastelstu.be>
Subject: Re: [PHP-DEV] [RFC] enum SortDirection
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
From: larry@garfieldtech.com ("Larry Garfield")

On Tue, Mar 3, 2026, at 2:50 PM, Tim D=C3=BCsterhus wrote:
> Hi
>
> On 3/3/26 01:36, Larry Garfield wrote:
>> I concur.  I'd like to see the functions updated now in the RFC, so w=
e get a sense of how many functions will be impacted.  Not because I'm c=
oncerned about a widened type breaking anything, but more so readers can=
 see the value this brings.
>
> I see the main value in helping userland not to reinvent the wheel eve=
ry=20
> time and to set some further precedent in including types for this kin=
d=20
> of ubiquitous concept in the standard library.
>
> I don't want to commit to supporting the enum for every possible=20
> function in the standard library for the reasons I mentioned in my rep=
ly=20
> to Bob and also because some of the functions already have a very mess=
y=20
> API signature where backfitting the enum is not exactly trivial.

I'm not suggesting we update every single function in one RFC shot.  But=
 the RFC should include updating at least one or two so that the Enum is=
 actually used.

Otherwise, there's no real advantage to putting it in core rather than F=
IG.  The reason to put it in core is so it can be used in core.  So let'=
s use it in core, even if not the full scope of eventual usage.

--Larry Garfield