Re: Re: [VOTE] Abstract Syntax Tree

From: Date: Wed, 27 Aug 2014 07:38:00 +0000
Subject: Re: Re: [VOTE] Abstract Syntax Tree
References: 1 2 3 4 5 6 7 8 9 10  Groups: php.internals 
Request: Send a blank email to internals+get-76901@lists.php.net to get a copy of this message
I get no new test failures on master after AST merged. Before and after the AST merge, I still get ~200 tests failing on Windows. My test runs turned up a few crashes, which I am reproducing to get BTs to share. I'm not sure if the crashes are AST related. On Tue, Aug 26, 2014 at 2:50 AM, Ferenc Kovacs <tyra3l@gmail.com> wrote: > On Tue, Aug 26, 2014 at 1:58 AM, Andrea Faulds <ajf@ajf.me> wrote: > > > > > On 26 Aug 2014, at 00:53, Benjamin Eberlei <kontakt@beberlei.de> wrote: > > > > > Depends, registering for shutdown handlers and catching fatals is a > > pretty > > > common thing. If EngineException does not get caught, it should produce > > an > > > E_FATAL again with the correct error code. > > > > Exceptions already do this, do they not? I don’t see why those exceptions > > would be any different. > > > > people have code out there like this: > try { > //something stupid > } catch (Exception $e) { > //do nothing or maybe log it > } > > if we turn fatal errors into Exceptions extending the base Exception class, > these Pokémon constructs ("Gotta Catch 'Em All") will will swallow those > errors and contrinue the execution, maybe causing more dangerous > results(data corruption, etc.) than a simple fatal error could. > But these issues was discussed before, so I'm fairly sure that Nikita is > aware of those concerns, and let's just wait for him to update his proposal > and we can discuss it after it's ready. > > -- > Ferenc Kovács > @Tyr43l - http://tyrael.hu >

« previous php.internals (#76901) next »