Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:89947
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.54 as permitted sender)
MIME-Version: 1.0
Reply-To: bishop@php.net
Sender: bishop.bettini@gmail.com
In-Reply-To: <568599AA.4090209@gmail.com>
References: <CADZXHp6UA4iH6196vN_U+rLSu1Tw+EazQod2LVgwJ7kkUCTd7Q@mail.gmail.com>
 <568599AA.4090209@gmail.com>
Date: Thu, 31 Dec 2015 16:48:13 -0500
Message-ID: <CAEYWF=7pdYc5okZim9_2Xin=djydvRYmCknA0-z9EViFQ1c-0A@mail.gmail.com>
To: Stanislav Malyshev <smalyshev@gmail.com>, Junade Ali <mjsa@junade.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11432ede561f750528389e9b
Subject: Re: [PHP-DEV] Deprecation of the Error Control Operator (@ symbol)
From: bishop@php.net (Bishop Bettini)

--001a11432ede561f750528389e9b
Content-Type: text/plain; charset=UTF-8

On Thu, Dec 31, 2015 at 4:10 PM, Stanislav Malyshev <smalyshev@gmail.com>
wrote:

>
> The inherent problem is that some error reporting is unneeded in some
> contexts, while welcome in others. That's the problem that needs
> solving, the syntax is secondary to it.
>

That hits the nail on the head.

When triggered errors are unneeded, @ works beautifully.

When they're needed, use a helper like from my third-party library
<https://github.com/haldayne/fox/blob/master/src/CaptureErrors.php> to
capture them:

use Haldayne\Fox\CaptureErrors;
$copy = new CaptureErrors(function ($a, $b) { return copy($a, $b); });
$okay = $copy('foo.txt', 'bar.txt');
if (! $okay) {
    throw new
RuntimeException($copy->getCapturedErrors()->pop()->get('message'));
}

I am -1 on removing @ for 7.x series.  But, I would be in favor of all
changes that remove unnecessary error messages or add functionality to
better work with error messages. In my mind, those are requisite steps
before removing @.

Thanks,
bishop

--001a11432ede561f750528389e9b--