Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:90123
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: softfail (pb1.pair.com: domain golemon.com does not designate 209.85.215.47 as permitted sender)
MIME-Version: 1.0
Sender: php@golemon.com
In-Reply-To: <CAH-PCH5gcWZSae7D1YcB501himD1QOjy7FUGm_4ebjTT-DPofQ@mail.gmail.com>
References: <CADZXHp6UA4iH6196vN_U+rLSu1Tw+EazQod2LVgwJ7kkUCTd7Q@mail.gmail.com>
	<CAESVnVq0RK-q0UQaqOpEg2=rEwW5znp7d_EoeHLoOsigvM77Sg@mail.gmail.com>
	<4F.0A.12097.24FFB865@pb1.pair.com>
	<CAESVnVr2uMOjnTUhcL7b03-Nh4H+z1c5NttF=ykP5DFMmdHHgg@mail.gmail.com>
	<CAH-PCH5gcWZSae7D1YcB501himD1QOjy7FUGm_4ebjTT-DPofQ@mail.gmail.com>
Date: Tue, 5 Jan 2016 10:22:49 -0800
Message-ID: <CAESVnVqQKqtn6tEK9CaykCtRkrXq_YXuM429f1cZvPK_5gsLeg@mail.gmail.com>
To: Ferenc Kovacs <tyra3l@gmail.com>
Cc: Andrea Faulds <ajf@ajf.me>, PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] Deprecation of the Error Control Operator (@ symbol)
From: pollita@php.net (Sara Golemon)

On Tue, Jan 5, 2016 at 10:09 AM, Ferenc Kovacs <tyra3l@gmail.com> wrote:
>> > I don't think that would work out too well. Remember that many projects
>> > have
>> > error handles which convert all errors to exceptions: if you disable @
>> > in
>> > those projects, wouldn't their code break?
>> >
>> Nope.
>> Supressed errors already reach custom error handlers:
>> https://3v4l.org/TG8aA
>
> sure and most projects check the error_reporting() level against the $errno
> like in the manual:
>     if (!(error_reporting() & $errno)) {
>         // This error code is not included in error_reporting
>         return;
>     }
>
> @ changes the error_reporting() level for that particular call, so those
> custom error handler won't throw exceptions for the suppressed errors but
> when you remove/nop @ their code would throwing stuff left and right.
>
Today I learned...  Okay, def a problem (for a specific set of
circumstances).  And one which makes me more amenable to Ze'ev
declare() suggestion (on a per-request bases, not per-file) as anyone
modifying a project's codebase for testing can also modify it to
suppress those exceptions as needed and/or just not enable the custom
error handler.

-Sara