Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:124758
Date: Mon, 5 Aug 2024 12:23:50 +0100 (BST)
To: Ilija Tovilo <tovilo.ilija@gmail.com>
cc: PHP internals <internals@lists.php.net>
Subject: Re: [PHP-DEV] [Concept] Flip relative function lookup order (global,
 then local)
In-Reply-To: <CAPyj-LCoBY2bPHLmsyONYDbY9aRSraLz5QNskeTiWfpMD0pTSg@mail.gmail.com>
Message-ID: <73c0d611-16f6-ff3e-95a2-3c78c66d413d@php.net>
References: <CAPyj-LCoBY2bPHLmsyONYDbY9aRSraLz5QNskeTiWfpMD0pTSg@mail.gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
From: derick@php.net (Derick Rethans)

On Fri, 2 Aug 2024, Ilija Tovilo wrote:

> As for providing a migration path: One approach might be to introduce 
> an INI setting that performs the function lookup in both local and 
> global scope at run-time, and informs the user about the behavioral 
> change in the future.

That INI setting would control the *warning*, and not the 
*functionlity*, right?

> Lastly, I've already raised this idea in the PHP Foundations internal 
> chat but did not receive much positive feedback, mostly due to fear of 
> the potential BC impact. I'm not particularly convinced this is an 
> issue, given the impact analysis. Given the surprisingly large 
> performance benefits, I was inclined to raise it here anyway.

I am surprised that it is that much of a performance benefit as well, 
but I am also concerned about the BC impact. But if that isn't too much, 
then I guess we need to consider this, but only for a major version. Not 
something I believe we can change in a 8.x version.

cheers,
Derick