Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:86043
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.192.174 as permitted sender)
Message-ID: <55416A09.50907@gmail.com>
Date: Wed, 29 Apr 2015 16:32:25 -0700
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:31.0) Gecko/20100101 Thunderbird/31.6.0
MIME-Version: 1.0
To: Patrick ALLAERT <patrickallaert@php.net>, 
 Kalle Sommer Nielsen <kalle@php.net>
CC: Olivier Garcia <oliviergarcia@php.net>, 
 PHP Internals <internals@lists.php.net>
References: <CAMZ7uYexNm6Fy42GYCUW9MPnFWEun=JFOPPSa+-mxOPL7hpaGg@mail.gmail.com> <CAJW__o3WnrbZESJexF-tKgdw_br6MGYEtWKkweU_n1na5XOP=A@mail.gmail.com> <CACwt+JfB8vhZCtwCd4vH32Oan7e++75u3_E9TdYfgCoAHiViBg@mail.gmail.com> <CAJW__o3tmPMycgjhSEW79Bf-HKYiU6Q8d8U0Rpy0C94c2y7Yvw@mail.gmail.com> <CACwt+Jd_j8BssM1XRwkS2r8XYv_rgQb4bVERCDwT0bZp9r6TkA@mail.gmail.com>
In-Reply-To: <CACwt+Jd_j8BssM1XRwkS2r8XYv_rgQb4bVERCDwT0bZp9r6TkA@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] [RFC][VOTE] Improved Error Callback Mechanism
From: smalyshev@gmail.com (Stanislav Malyshev)

Hi!

> The issue targeted here is "two extensions overriding php_error_cb can't
> live side by side" and that issue exists since PHP 4.0. Should a bug report
> be opened?

I'm not sure why can't they live side by side. Of course, if the
override is made in incompatible manner, they can't, but the same is
true with proposed API - i.e., one extension can boot all other
extension's hooks with zend_clear_error_hook, potentially breaking all
other extension's functionality.

> We believe that having an extension that additionally hooks in the error
> handler mechanism (aside Xdebug) would be of a great help to migrate from
> PHP 5.x to 7.0.

While I agree with the first part, I'm not sure what makes it vital for
7.0 - as it doesn't introduce BC breaks, is this really important to
have it in major version? Also not sure how it would help migration - if
anything, new API means *more* work when migrating, not less.

-- 
Stas Malyshev
smalyshev@gmail.com