Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:117117
MIME-Version: 1.0
References: <620eda0f.1c69fb81.d2cb1.0846SMTPIN_ADDED_MISSING@mx.google.com>
 <5efecaef-a024-3c61-e12e-ffc342956718@gmail.com> <CAOWwgpkcnaVctXu8LgYpp_CLzx0QySbV87ibRqy0aD6qqHmsxA@mail.gmail.com>
 <6214c40e.1c69fb81.73261.05d3SMTPIN_ADDED_MISSING@mx.google.com>
In-Reply-To: <6214c40e.1c69fb81.73261.05d3SMTPIN_ADDED_MISSING@mx.google.com>
Date: Tue, 22 Feb 2022 14:53:05 +0100
Message-ID: <CAOWwgpkoGvonQDwOS331JNWLT=zJ8pNN+CEoEce4rtebMn2+0w@mail.gmail.com>
To: Mark Randall <marandall@php.net>
Cc: PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000c829c505d89bab42"
Subject: Re: [PHP-DEV] [RFC] Undefined Variable Error Promotion
From: nicolas.grekas+php@gmail.com (Nicolas Grekas)

--000000000000c829c505d89bab42
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Le mar. 22 f=C3=A9vr. 2022 =C3=A0 12:07, Mark Randall <marandall@php.net> a=
 =C3=A9crit :

> On 22/02/2022 09:15, Nicolas Grekas wrote:
> > I very much call for an implementation to be provided before starting a=
ny
> > vote on the topic btw.
> The RFC states on its first line that accessing an undefined variable
> emits an E_WARNING. If it does not currently emit an E_WARNING then it
> is out of scope of this RFC.
>

This last sentence could be worth copy/pasting into the RFC :)
Thanks for the recent edits, they help clarify to me.


> It is not sensible to provide a full implementation for something that
> will require extensive engine and test changes for something several
> years into the future, during which many other changes will be made.
>

> A sensible time for this would be after the branch is cut for 8.4.
>

I understand if the target is to do everything in 9.0.
But this makes me think: we should trigger a deprecation just before all
corresponding warnings!
This would signal the change to 8.x users and help them spot where a change
is needed.
This should happen before the warning to that error handlers that turn the
warning into an exception also get a note about the soon-to-be change.

WDYT?

Nicolas

--000000000000c829c505d89bab42--