Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:84465
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain basereality.com from 209.85.213.47 cause and error)
MIME-Version: 1.0
In-Reply-To: <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>
	<438F62BF-E1D3-484E-8242-C616F6007811@heroku.com>
Date: Mon, 9 Mar 2015 11:40:51 +0000
Message-ID: <CA+kxMuSGu62onLh-GZBORgMSh+pR3aFX-OGSEaS1nR9pg01c9g@mail.gmail.com>
To: David Zuelke <dz@heroku.com>
Cc: Nikita Popov <nikita.ppv@gmail.com>, PHP internals <internals@lists.php.net>, 
	Dmitry Stogov <dmitry@zend.com>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] [VOTE] Exceptions in the engine
From: danack@basereality.com (Dan Ackroyd)

On 9 March 2015 at 11:04, David Zuelke <dz@heroku.com> wrote:
> Why not wait with the merge until a consensus emerges regarding Throwable?

The patch for engine exceptions is large - the longer that it is left
unmerged, the more difficult it will be to do and people's time is
valuable. It also delays finding bugs or other issues in the engine
exception patch by at least two weeks, which is about 15% of the time
until we're meant to be having an RC.

Any change in the exception hierarchy or use of interfaces should be a
small patch. Holding off on merging the engine exception patch doesn't
make it significantly easier to do.

So even though I hope we can clean up the exception hierarchy, merging
the engine exceptions is the right choice imo.

cheers
Dan