Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:107363
MIME-Version: 1.0
References: <CAF+90c_Ds+UNZB4hAi1989SRafb=d1JkP2q4pdwkUUXfcHrLCw@mail.gmail.com>
 <CAF+90c99w6Uk+P8rm_p0Pd3iKsBKQqAbBjs2Zzwb5vGW9-snhQ@mail.gmail.com> <CAF+90c9zAggYVeU76kgJydmNr3nzN2wKgOjTz3QF7yohJLQCOw@mail.gmail.com>
In-Reply-To: <CAF+90c9zAggYVeU76kgJydmNr3nzN2wKgOjTz3QF7yohJLQCOw@mail.gmail.com>
Date: Wed, 2 Oct 2019 10:45:17 -0500
Message-ID: <CAESVnVojZe9ePpDxpaPQoN5g4TR1s3WKYzPg8KPpd8MgXUZENQ@mail.gmail.com>
To: Nikita Popov <nikita.ppv@gmail.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000bc13650593ef5ba7"
Subject: Re: [PHP-DEV] Re: [VOTE] Reclassifying engine warnings
From: pollita@php.net (Sara Golemon)

--000000000000bc13650593ef5ba7
Content-Type: text/plain; charset="UTF-8"

On Wed, Oct 2, 2019 at 8:24 AM Nikita Popov <nikita.ppv@gmail.com> wrote:

> * The "Undefined array index" case. This one passed the vote with an exact
> 2/3 majority, so I'm a bit uncomfortable making changes here. This is also
> the only case where convincing usability concerns have been brought
> forward, primarily around the $array[$key]++ example. I'm not yet decided
> on what to do here and whether we should pursue additional library or
> language features first.
>
>
We raised margins to 2/3 to avoid having to feel bad about narrow
majorities.  I say this as someone who was quite tempted to vote "Keep
Notice" which on retrospect would have changed the entire outcome based on
that one vote.

If this had been a simple majority vote and it had pass by like....
52%-48%, then obviously that's not a mandate worth making potentially
catastrophic changes over, but this is 67% to 33%, it feels like you're
okay here.

-Sara

--000000000000bc13650593ef5ba7--