Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:69775
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain bof.de designates 80.242.145.70 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <5266BECD.8000204@php.net>
References: <3D.BC.23638.84CA1625@pb1.pair.com>
	<alpine.DEB.2.10.1310211054300.5542@whisky.home.derickrethans.nl>
	<5264FE65.8050808@php.net>
	<alpine.DEB.2.10.1310221001370.5542@whisky.home.derickrethans.nl>
	<CADajX4BqUetQPPOefEetYFxVb0s4FNhAEvtXhgmCLCUouS19yg@mail.gmail.com>
	<5266B6A8.4010607@pthreads.org>
	<CADajX4CDvrHc8PoiOyfCM8601AcZjEM6Cw_X0+O=hJSPT65d_A@mail.gmail.com>
	<5266BECD.8000204@php.net>
Date: Tue, 22 Oct 2013 20:32:41 +0200
Message-ID: <CAJ26g5Rs-+byvCe63fdv5GhDhEf3Lmc+ze9AbZe5kyUnr9JCeA@mail.gmail.com>
To: Joe Watkins <krakjoe@php.net>
Cc: Adam Harvey <aharvey@php.net>, internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=047d7b6786fe42debe04e958a07c
Subject: Re: [PHP-DEV] RFC: Expectations
From: bof@bof.de (Patrick Schaaf)

--047d7b6786fe42debe04e958a07c
Content-Type: text/plain; charset=ISO-8859-1

Am 22.10.2013 20:07 schrieb "Joe Watkins" <krakjoe@php.net>:
>
> You can catch exceptions, and log them.
> You can do that without impacting everything around you, you can do that,
or whatever else you like.

You can do that handling when you have only a few toplevel scripts, like
when you have a setup with a small number of toplevel controllers and
almost no CLI stuff around. But due to the scoped nature of try/catch you
cannot reasonably / painlessly do that when you have 270 toplevel web and
CLI entry points that then each need such a try/catch block.

Also, there is the problem with catch (Exception) blocks, which you might
easily dismiss as bad form, but which I'm sure are widespread in the field
- I certainly know they are in our codebase...

On the other hand, setup of an assertion-failed callback can easily go into
an auto_prepend file, or into any other standard include (autoloader or
something) you might have - exactly because it is something done on the
global level instead of the scoped try/catch requirement. And IF you like
the exception thing you can make that callback throw whatever you like -
but you do not force that model on everybody.

Furthermore such an assertion-failed callback has exactly the same change
of looking at backtraces, so touting that as a singular feature of the
exception approach is not valid.

Finally, with the exception approach it is simply not true that it will
completely go away in production - because these try/catch blocks will be
present for any code that wants to handle the issue, and you cannot make
those go away.

I'm all for an assert alternative that works with expressions instead of
eval, and that completely goes away in the opcode (cache) when disabled in
production.

best regards
  Patrick

--047d7b6786fe42debe04e958a07c--