Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:125080
Feedback-ID: ifab94697:Fastmail
Precedence: bulk
MIME-Version: 1.0
Date: Wed, 21 Aug 2024 02:31:27 +0200
To: internals@lists.php.net
Message-ID: <7a650bf9-a905-4b39-9f78-d541209b8f2f@app.fastmail.com>
In-Reply-To: <63654CCD-DF9C-45A4-814E-5696BD780EB3@dafert.at>
References: <2716f729-4008-4f75-8412-861d8960b746@app.fastmail.com>
 <63654CCD-DF9C-45A4-814E-5696BD780EB3@dafert.at>
Subject: Re: [PHP-DEV] function autoloading v4 RFC
Content-Type: multipart/alternative;
 boundary=b2d804026d9046a7a5fcc533b2214c17
From: rob@bottled.codes ("Rob Landers")

--b2d804026d9046a7a5fcc533b2214c17
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

On Mon, Aug 19, 2024, at 20:28, Mel Dafert wrote:
> On August 15, 2024 5:22:51 PM GMT+02:00, Rob Landers <rob@bottled.code=
s> wrote:
> >Hello internals,
> >
> >I've decided to attempt an RFC for function autoloading. After readin=
g hundreds of ancient (and recent) emails relating to the topic along wi=
th several abandoned RFCs from the past, and after much review, I've dec=
ided to put forth a variation of a previous RFC, as it seemed the least =
ambitious and the most likely to work:
> >
> >https://wiki.php.net/rfc/function_autoloading4
> >
> >Please let me know what you think. An implementation should be along =
before opening it for a vote (now that I realize how important that is).
> >
> >=E2=80=94 Rob
>=20
> Hello,
>=20
> I've also noted that on another RFC, but I think it applies here too:=20
> Any new constants that are shaped like an enum and used like an enum s=
hould just be
> an actual enum. I think instead of `SPL_AUTOLOAD_CLASS` and `SPL_AUTOL=
OAD_FUNCTION`,
> there should be an `enum AutoloadType` (to be bikeshed), and the new p=
arameters
> should be typed as such, instead of `int`.
>=20
> Regards,
> Mel
>=20

Hey Mel,

I spent a couple of hours trying to do this today. But due to the weird =
dependency hell that SPL lives in right now, I'm not sure it can be an e=
num (or at least, the linker kept messing up my version of PHP to the po=
int where it couldn't even run=E2=80=94it was bizarre; I've only seen st=
uff like that with incompatible cgo shenanigans. I will have to try agai=
n later when I next rebase my branch).=20

I've reached out to Gina about their RFC and my RFC joining forces. Gina=
's RFC introduces an entirely new API for autoloading and neatly side-st=
eps the whole thing.

=E2=80=94 Rob
--b2d804026d9046a7a5fcc533b2214c17
Content-Type: text/html; charset=utf-8
Content-Transfer-Encoding: quoted-printable

<!DOCTYPE html><html><head><title></title><style type=3D"text/css">p.Mso=
Normal,p.MsoNoSpacing{margin:0}</style></head><body><div>On Mon, Aug 19,=
 2024, at 20:28, Mel Dafert wrote:<br></div><blockquote type=3D"cite" id=
=3D"qt" style=3D""><div>On August 15, 2024 5:22:51 PM GMT+02:00, Rob Lan=
ders &lt;<a href=3D"mailto:rob@bottled.codes">rob@bottled.codes</a>&gt; =
wrote:<br></div><div>&gt;Hello internals,<br></div><div>&gt;<br></div><d=
iv>&gt;I've decided to attempt an RFC for function autoloading. After re=
ading hundreds of ancient (and recent) emails relating to the topic alon=
g with several abandoned RFCs from the past, and after much review, I've=
 decided to put forth a variation of a previous RFC, as it seemed the le=
ast ambitious and the most likely to work:<br></div><div>&gt;<br></div><=
div>&gt;<a href=3D"https://wiki.php.net/rfc/function_autoloading4">https=
://wiki.php.net/rfc/function_autoloading4</a><br></div><div>&gt;<br></di=
v><div>&gt;Please let me know what you think. An implementation should b=
e along before opening it for a vote (now that I realize how important t=
hat is).<br></div><div>&gt;<br></div><div>&gt;=E2=80=94 Rob<br></div><di=
v><br></div><div>Hello,<br></div><div><br></div><div>I've also noted tha=
t on another RFC, but I think it applies here too:&nbsp;<br></div><div>A=
ny new constants that are shaped like an enum and used like an enum shou=
ld just be<br></div><div>an actual enum. I think instead of `SPL_AUTOLOA=
D_CLASS` and `SPL_AUTOLOAD_FUNCTION`,<br></div><div>there should be an `=
enum AutoloadType` (to be bikeshed), and the new parameters<br></div><di=
v>should be typed as such, instead of `int`.<br></div><div><br></div><di=
v>Regards,<br></div><div>Mel<br></div><div><br></div></blockquote><div><=
br></div><div>Hey Mel,<br></div><div><br></div><div>I spent a couple of =
hours trying to do this today. But due to the weird dependency hell that=
 SPL lives in right now, I'm not sure it can be an enum (or at least, th=
e linker kept messing up my version of PHP to the point where it couldn'=
t even run=E2=80=94it was bizarre; I've only seen stuff like that with i=
ncompatible cgo shenanigans. I will have to try again later when I next =
rebase my branch). <br></div><div><br></div><div>I've reached out to Gin=
a about their RFC and my RFC joining forces. Gina's RFC introduces an en=
tirely new API for autoloading and neatly side-steps the whole thing.<br=
></div><div><br></div><div id=3D"sig121229152">=E2=80=94 Rob<br></div></=
body></html>
--b2d804026d9046a7a5fcc533b2214c17--