Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:124338
Date: Wed, 10 Jul 2024 10:31:54 +0100 (BST)
To: Larry Garfield <larry@garfieldtech.com>
cc: php internals <internals@lists.php.net>
Subject: Re: [PHP-DEV] [RFC] Property Hook improvements
In-Reply-To: <fbf45a9a-0385-4c42-a6a0-4f3ee1c8728d@app.fastmail.com>
Message-ID: <520ee513-d6fe-a658-5845-00296a9e08a2@php.net>
References: <fbf45a9a-0385-4c42-a6a0-4f3ee1c8728d@app.fastmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
From: derick@php.net (Derick Rethans)

On Mon, 1 Jul 2024, Larry Garfield wrote:

> Hi folks.  As Ilija's been polishing off hooks to get the PR merged, 
> we've run into two small revisions that should make life better for 
> all involved.  One is a performance improvement that requires a very 
> slight error handling behavior change, and the other is enabling 
> readonly in selected (but probably all of the relevant) circumstances.
> 
> I'd say we expect these to be uncontroversial, but this is PHP. :-)  
> So I will instead just note that it's a short RFC and open the 
> discussion accordingly.
> 
> https://wiki.php.net/rfc/hook_improvements

1. Remove the proactive guard against recursive backing value access

	"which would eventually trigger a stack overflow"

Would it though? PHP 8.4 has an actual protection against this now, 
instead of getting the white-screen-of-death.

Not that it matters much, but it's probably good to be precise in the 
language.

cheers,
Derick