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.
-- 
Stas Malyshev
smalys...@gmail.com

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to