Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:87179
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.160.177 as permitted sender)
MIME-Version: 1.0
Sender: yohgaki@gmail.com
In-Reply-To: <55A63C90.8000705@gmail.com>
References: <CALYT1Yo-ae+cemXaaXs5Jv5t2DSzdxdYuxuhEGbXyoo_863+bw@mail.gmail.com>
 <55A63C90.8000705@gmail.com>
Date: Wed, 15 Jul 2015 20:20:08 +0900
Message-ID: <CAGa2bXYpZuYssDqLJxhaD0Cx-myNM_vXAOr-G2X4Y9xawbtmVA@mail.gmail.com>
To: Rowan Collins <rowan.collins@gmail.com>
Cc: "internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a114e4a5c8dc583051ae82573
Subject: Re: [PHP-DEV] Core functions throwing exceptions in PHP7
From: yohgaki@ohgaki.net (Yasuo Ohgaki)

--001a114e4a5c8dc583051ae82573
Content-Type: text/plain; charset=UTF-8

Hi Rowan,

On Wed, Jul 15, 2015 at 7:57 PM, Rowan Collins <rowan.collins@gmail.com>
wrote:

> Sammy Kaye Powers wrote on 14/07/2015 22:04:
>
>> Since the core functions in PHP don't throw Exceptions, there is debate on
>> whether or not this change should be implemented. Some say the CSPRNG's
>> should get a special pass since they will be relied on for cryptography.
>> If
>> we can't throw Exceptions, there were suggestions of raising a fatal error
>> if the RNG fails.
>>
>
> Given that the *engine itself* now throws exceptions, and we have bundled
> extensions which throw exceptions, the blanket ban on exceptions "in core
> functions" seems increasingly out-dated. This particular function is in
> ext/standard, which is about as "core" as you can get, but it is a brand
> new function with no BC considerations and an explicit mandate to be as
> secure as possible.
>
> To me, using an exception here makes perfect sense, but I'm not sure how
> to formulate a policy that allows that without opening the floodgates too
> quickly. I'm also a big fan of proper exception hierarchies - I've seen so
> many times that you should never *catch* the base Exception class, so
> *throwing* a base Exception seems very wrong to me.


As I wrote earlier, I prefer exceptions rather than errors.

Engine exception is a little different. The main motivation for engine
exception is
to give a chance to users for graceful program termination. Functions can
achieve
the objective by E_RECOVERABLE_ERROR mostly.

I don't want to see one function raise exception and the other raise error.
It's "should be avoided inconsistency". IMHO.
If we are going to adopt exception for "functions", it would be better to
have
switch that convert all errors to exceptions.

Regards,

--
Yasuo Ohgaki
yohgaki@ohgaki.net

--001a114e4a5c8dc583051ae82573--