Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:107335
MIME-Version: 1.0
References: <CAF+90c_Ds+UNZB4hAi1989SRafb=d1JkP2q4pdwkUUXfcHrLCw@mail.gmail.com>
 <CAF+90c99w6Uk+P8rm_p0Pd3iKsBKQqAbBjs2Zzwb5vGW9-snhQ@mail.gmail.com> <FD053AC9-B174-4A90-8F3C-CECD5F2CFC9F@cschneid.com>
In-Reply-To: <FD053AC9-B174-4A90-8F3C-CECD5F2CFC9F@cschneid.com>
Date: Thu, 26 Sep 2019 10:23:25 -0400
Message-ID: <CAPKYkKz=qRwbF200UEr3rBCscCETHV04FihhkVrOn3xgQUp3+w@mail.gmail.com>
To: Christian Schneider <cschneid@cschneid.com>
Cc: Nikita Popov <nikita.ppv@gmail.com>, PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000e4bae805937583e5"
Subject: Re: [PHP-DEV] [VOTE] Reclassifying engine warnings
From: chasepeeler@gmail.com (Chase Peeler)

--000000000000e4bae805937583e5
Content-Type: text/plain; charset="UTF-8"

On Thu, Sep 26, 2019 at 4:31 AM Christian Schneider <cschneid@cschneid.com>
wrote:

> Am 26.09.2019 um 09:41 schrieb Nikita Popov <nikita.ppv@gmail.com>:
> > * Remainder: 54 yes, 3 no. Accepted with 95% majority.
>
> Just for the record:
> The one I'm most concerned about here is the foreach on undefined
> variables throwing an exception.
> While this was already promoted to a warning at an earlier stage it still
> allowed a program to finish with a well-defined result (looping over
> nothing does nothing).
> Now this code will stop with an Exception and possibly won't produce its
> output.
>
> I agree. At the very least, I'd propose that null (and ideally false) NOT
throw an exception. There are enough instances of methods that return one
of those values when there is no data (so, not an error situation) but an
array when there is.


> I noticed this before the vote but as the mailing list was already flooded
> with the 'undefined variables' discussion I feared that there was no place
> to bring this up without getting personally attacked like I was for the
> undefined variables stuff.
>
> Thanks for bringing this up. I think this should be noted in relation to
the other RFC currently out there. Those of us speaking out against this
RFC are the ones being held up as examples of why that RFC is needed, which
this statement actually shows that it was those in favor of the RFC that
were behaving in a way that intimidated others and made them feel reluctant
to contribute.


> Anyway, the vote is done, I said my piece and will shut up about it now,
> - Chris
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>

-- 
Chase Peeler
chasepeeler@gmail.com

--000000000000e4bae805937583e5--