Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:120017
Date: Thu, 13 Apr 2023 09:11:25 +0100
To: internals@lists.php.net
User-Agent: K-9 Mail for Android
In-Reply-To: <CA+kxMuSn+VFQd7E_TDcZyNs_4GAUfw4t4pkQAvVSX1Y9jUJOug@mail.gmail.com>
References: <CAFPFaMLghRYP4ioCkfa7WKhTAz+1gWgcZ5mWYAunkb9f3g=fdA@mail.gmail.com> <A97A9A6E-DBE5-4BE6-98FE-C5A7381460E4@gmail.com> <CA+kxMuSn+VFQd7E_TDcZyNs_4GAUfw4t4pkQAvVSX1Y9jUJOug@mail.gmail.com>
Message-ID: <C320B1FA-3890-410D-9913-74581607F034@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain;
 charset=utf-8
Content-Transfer-Encoding: quoted-printable
Subject: =?US-ASCII?Q?Re=3A_=5BPHP-DEV=5D_=5BRFC=5D_New_core_autoloading_mec?= =?US-ASCII?Q?hanism_with_support_for_function_autoloading?=
From: rowan.collins@gmail.com (Rowan Tommins)

On 12 April 2023 23:50:25 BST, Dan Ackroyd <Danack@basereality=2Ecom> wrote=
:
>On Wed, 12 Apr 2023 at 17:32, Claude Pache <claude=2Epache@gmail=2Ecom> w=
rote:
>>
>> The proposed modification of `function_exists()` will break existing co=
de:
>
>Please can you submit a failing test to
>https://github=2Ecom/Girgias/php-src/tree/zend_autoloader that shows a
>BC break=2E


I see that George has added this test based on a previous example in this =
thread: https://github=2Ecom/php/php-src/blob/7aaae2efa080084035911065c346d=
5f8932b0d63/Zend/tests/autoloading/function/local_function_pinned-to_global=
=2Ephpt

It currently enshrines the proposed change in behaviour, so if you want a =
test that passes on existing versions and fails on the current branch, just=
 delete the last two lines in the EXPECT section=2E

Regards,

--=20
Rowan Tommins
[IMSoP]