I want to talk about the BC impact of what has been discussed.
Currently the meaning of this code is to catch all possible
exceptions, because all exceptions *must* extend `\Exception`:
} catch (Exception $e) {
By making some other root exception you just broke all the code that
is *appropriately* using the catch-all variant.
So either way it is a BC break. So let's consider other impact to help decide:
If we introduce a new root exception then the people "punished" are
the ones currently using Exception correctly. Whereas if engine
exceptions just extend Exception then the people "punished" are the
ones using it inappropriately.
Please, revert all of these changes with Error and some new root
exception and *do the right thing* based on logical arguments. It
makes no sense to punish the people using code correctly to avoid
punishing people misusing a feature, especially when it is a BC break
either way.
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
