Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:85546
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.212.179 as permitted sender)
MIME-Version: 1.0
Date: Mon, 30 Mar 2015 12:19:17 +0200
Message-ID: <CAF+90c9wiu1qOoHgZs_Q6DLACkCyOMUd38wAwjOLsRAqGiXfBQ@mail.gmail.com>
To: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11c2b34889407605127ed069
Subject: Exception hierarchy: open issues
From: nikita.ppv@gmail.com (Nikita Popov)

--001a11c2b34889407605127ed069
Content-Type: text/plain; charset=UTF-8

Hi internals!

There are a number of open issues with regard to the exception hierarchy
for recently introduced exception, for which I would like to open a new
thread:

 * Naming and classiness of BaseException. There's an RFC to change this to
a Throwable interface: https://wiki.php.net/rfc/throwable - If we want to
move forward with that, it needs a patch, as turning it into an interface
likely has non-trivial consequences that need to be addressed in the RFC.

 * Expectations introduced an AssertionException. Currently it extends
Exception. Should it extend BaseException instead, as you usually should
not catch assertion exceptions? This was discussed in the RFC thread, but
the decision was postponed until engine exceptions land, which has happened
by now.

* The scalar type hints implementation added a TypeException extends
EngineException. We didn't discuss this on-list, so I'd like to bring this
up in case anyone disagrees. Imho TypeException may not be best name for
it, as it's also thrown for non-type related error conditions, like
mismatched argument count.

Thanks,
Nikita

--001a11c2b34889407605127ed069--