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....@gmail.com> wrote: > On Fri, Oct 7, 2016 at 9:31 PM, Derick Rethans <der...@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 >
