Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:119978
Message-ID: <A97A9A6E-DBE5-4BE6-98FE-C5A7381460E4@gmail.com>
Content-Type: multipart/alternative;
	boundary="Apple-Mail=_84780BA5-C67F-40E0-82E2-67219196C4A5"
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3731.500.231\))
Date: Wed, 12 Apr 2023 18:31:39 +0200
In-Reply-To: <CAFPFaMLghRYP4ioCkfa7WKhTAz+1gWgcZ5mWYAunkb9f3g=fdA@mail.gmail.com>
Cc: PHP internals <internals@lists.php.net>
To: "G. P. B." <george.banyard@gmail.com>
References: <CAFPFaMLghRYP4ioCkfa7WKhTAz+1gWgcZ5mWYAunkb9f3g=fdA@mail.gmail.com>
Subject: Re: [PHP-DEV] [RFC] New core autoloading mechanism with support for
 function autoloading
From: claude.pache@gmail.com (Claude Pache)

--Apple-Mail=_84780BA5-C67F-40E0-82E2-67219196C4A5
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain;
	charset=utf-8



> Le 10 avr. 2023 =C3=A0 14:17, G. P. B. <george.banyard@gmail.com> a =
=C3=A9crit :
>=20
> Hello Internals,
>=20
> Dan and I would like to propose a new core autoloading mechanism that =
fixes
> some minor design issues with the current class autoloading mechanism =
and
> introduce a brand-new function autoloading mechanism:
> https://wiki.php.net/rfc/core-autoloading
>=20

Hi,

1. The proposed modification of `function_exists()` will break existing =
code:

<?php
namespace foo;
if (!function_exists(__NAMESPACE__.'\\bar')) {
    function bar() {
        // ....
    }
}
?>

as soon as a `bar()` function is defined in global scope, with no =
obvious hint why it suddenly broke. You should either (from my =
increasing order of preference):

* acknowledge it in the =E2=80=9CBackward Incompatible Changes=E2=80=9D =
section;

*  default the $autoload extra parameter to `false`;

* keep the current semantics, that using a namespaced or qualified =
function name, (including a function name given as string, which is =
implicitly fully qualified), doesn=E2=80=99t fall back to the global =
scope, =E2=80=94 even when a previous use of the same function name did =
trigger the fall back, see: https://3v4l.org/mnVWO (Independently of any =
mechanism introduced to avoid to repeatedly trigger the function =
autoloader.)


2. If you add function autoloading, you ought to support constant =
autoloading as well, of course.

=E2=80=94Claude=

--Apple-Mail=_84780BA5-C67F-40E0-82E2-67219196C4A5--