Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:86275
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.212.178 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <5557AF50.3060203@gmail.com>
References: <5557AF50.3060203@gmail.com>
Date: Sat, 16 May 2015 23:56:39 +0200
Message-ID: <CAF+90c9e3TjFiUYsdBUMCZvqckASx=7TSY04eWy+11qGS_WsRw@mail.gmail.com>
To: Stanislav Malyshev <smalyshev@gmail.com>
Cc: PHP Internals <internals@lists.php.net>, Sebastian Bergmann <sebastian@php.net>
Content-Type: multipart/alternative; boundary=047d7bf0ef18160b2705163a09c6
Subject: Re: [PHP-DEV] Throwable patch(es) and finalizing the story
From: nikita.ppv@gmail.com (Nikita Popov)

--047d7bf0ef18160b2705163a09c6
Content-Type: text/plain; charset=UTF-8

On Sat, May 16, 2015 at 10:57 PM, Stanislav Malyshev <smalyshev@gmail.com>
wrote:

> Hi!
>
> I am assuming we have broad consensus about the move from BaseException
> to Throwable. I've looked at the patch at
> https://github.com/php/php-src/pull/1282 and it seems pretty close to
> what we want to me. The only difference is that
> https://wiki.php.net/rfc/throwable says Throwable is an interface and
> this patch makes it abstract class - which I guess makes some sense as
> both Exception and engine errors need the same properties - but I'm not
> sure if it doesn't mess something up. Sebastian, could you comment on
> this? Of course, other comments are welcome too.
>
> There's a question left about the naming - particularly, renaming
> EngineException to Error. I'm not sure I am a fan of this. I think it's
> better to leave it as EngineException, but I'm open to arguments here.
>
> One thing seems to be clear though - a) there's a broad support for
> introducing Throwable and b) we need to do this before we release PHP 7
> alpha. So I think we need to start converging on something really soon.
> I'd like to take https://github.com/php/php-src/pull/1282 as a baseline,
> and once we close open questions about interface/abstract class and the
> name of the Error class, merge it - targeting sometime within the
> timeframe of the next week. If someone thinks it's impossible or should
> not be done, please tell.
>

Sorry for going a bit off-topic here, but why does this need to be resolved
for the first alpha? If I'm not mistaken, we do feature freeze on first
beta, not first alpha, right? I'd like to see an alpha release ASAP without
waiting for the last few features to trickle in.

Btw, the author of PR #1282 also has an implementation that makes Throwable
an interface (with semantics similar to Traversable, so cannot be directly
implemented).

Nikita

--047d7bf0ef18160b2705163a09c6--