Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:120025
Message-ID: <a492bd9e-7147-ce9a-83d8-2263a4ea0d00@seld.be>
Date: Thu, 13 Apr 2023 12:39:43 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.9.1
Content-Language: fr
To: internals@lists.php.net
References: <CAFPFaMLghRYP4ioCkfa7WKhTAz+1gWgcZ5mWYAunkb9f3g=fdA@mail.gmail.com>
 <A97A9A6E-DBE5-4BE6-98FE-C5A7381460E4@gmail.com>
 <3b7db03a-1cad-3762-b0ea-d33618be9644@gmail.com>
In-Reply-To: <3b7db03a-1cad-3762-b0ea-d33618be9644@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] [RFC] New core autoloading mechanism with support for
 function autoloading
From: j.boggiano@seld.be (Jordi Boggiano)

On 2023-04-12 22:26, Rowan Tommins wrote:
> I could just about live with that example changing so that the 
> fallback was cached, but I definitely don't think an explicit call 
> like \foo\strlen('x') should become an implicit alias for 
> \strlen('x'), which is apparently the current proposal.
>
> I really like the majority of this proposal, but right now would vote 
> against it based on that.
>
I agree here, I missed that fact while reviewing the RFC as I just 
understood it as an internal cache that \foo\strlen => \strlen, but 
function_exists('foo\strlen') should still return false IMO until it is 
defined, and once it is defined the cache should be busted.

https://3v4l.org/VLW5O looks like correct behavior to me correctly.

Best,
Jordi

-- 
Jordi Boggiano
@seldaek - https://seld.be