Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:84463
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain heroku.com designates 209.85.220.47 as permitted sender)
Content-Type: text/plain; charset=us-ascii
Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2070.6\))
In-Reply-To: <CAF+90c-ZyULAJNWU4h8Qqonkszh_9B1dN-8L0c=h42YO+nFu3w@mail.gmail.com>
Date: Mon, 9 Mar 2015 22:04:15 +1100
Cc: PHP internals <internals@lists.php.net>,
 Dmitry Stogov <dmitry@zend.com>
Content-Transfer-Encoding: 7bit
Message-ID: <438F62BF-E1D3-484E-8242-C616F6007811@heroku.com>
References: <CAF+90c8nqCBGWpsqdfy+EKekOm3Cw5_zzE_nKeyLK5pzROTueg@mail.gmail.com> <CAF+90c-ZyULAJNWU4h8Qqonkszh_9B1dN-8L0c=h42YO+nFu3w@mail.gmail.com>
To: Nikita Popov <nikita.ppv@gmail.com>
Subject: Re: [PHP-DEV] [VOTE] Exceptions in the engine
From: dz@heroku.com (David Zuelke)

Why not wait with the merge until a consensus emerges regarding Throwable?


> On 09.03.2015, at 05:26, Nikita Popov <nikita.ppv@gmail.com> wrote:
> 
> On Mon, Feb 23, 2015 at 7:15 PM, Nikita Popov <nikita.ppv@gmail.com> wrote:
> 
>> Hi internals!
>> 
>> Voting on the engine exceptions RFC, which proposes to convert existing
>> fatal and recoverable fatal errors into exceptions, has opened:
>> 
>>    https://wiki.php.net/rfc/engine_exceptions_for_php7#vote
>> 
>> The primary vote requires a 2/3 majority, as this is a language change.
>> 
>> A second vote will decide whether to use a BaseException based inheritance
>> hierarchy. This vote uses a simple majority.
>> 
>> Voting is open until 2015-03-08.
>> 
> 
> This RFC has been accepted with 60 votes in favor and 2 against. The
> BaseException hierarchy has been accepted with 39 in favor and 19 against.
> 
> During voting some concerns about the naming of the BaseException class
> have been raised. This detail can be sorted out in a follow-up discussion -
> changing this (assuming only the name is changed) is simple and only a
> couple of tests need to be adjusted.
> 
> Dmitry, do you want to merge your current patch?
> 
> Nikita