Re: Regression between RC1 and RC2?
| From: | Davey Shafik | Date: | Fri, 07 Oct 2016 20:04:52 +0000 |
| Subject: | Re: Regression between RC1 and RC2? | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-96292@lists.php.net to get a copy of this message | ||
Yes, we should not mask the exception. The behavior in 7.0/7.1.0RC1 is much
better IMO.
(As seen here: https://3v4l.org/EJpD4#v700)
- Davey
On Fri, Oct 7, 2016 at 12:52 PM, Nikita Popov <nikita.ppv@gmail.com> wrote:
> On Fri, Oct 7, 2016 at 9:31 PM, Derick Rethans <derick@php.net> wrote:
>
> > Hi,
> >
> > I was looking at Xdebug for PHP 7.1, and I ran into the following
> > inconsistency:
> >
> > https://3v4l.org/tHteN
> >
> > I first thought that Xdebug was messing up, but it seems like it's
> > different behaviour in PHP itself. As I clearly return an array from
> > __debugInfo, I don't think the new result is the correct one.
> >
> > cheers,
> > Derick
> >
>
> This is due to https://github.com/php/php-src/commit/
> 2d8ab51576695630a7471ff829cc5ea10becdc0f, which landed in PHP-7.0 as well.
> The problem is that __debugInfo currently is not able to handle exceptions
> gracefully. I think we should revert this change for now as it hides the
> fact that the underlying cause of the error is an exception.
>
> Nikita
>