Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:76901 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 98116 invoked from network); 27 Aug 2014 07:38:05 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 27 Aug 2014 07:38:05 -0000 Authentication-Results: pb1.pair.com smtp.mail=themattficken@gmail.com; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=themattficken@gmail.com; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.217.182 as permitted sender) X-PHP-List-Original-Sender: themattficken@gmail.com X-Host-Fingerprint: 209.85.217.182 mail-lb0-f182.google.com Received: from [209.85.217.182] ([209.85.217.182:63499] helo=mail-lb0-f182.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 5B/F1-20444-BDA8DF35 for ; Wed, 27 Aug 2014 03:38:03 -0400 Received: by mail-lb0-f182.google.com with SMTP id n15so2656423lbi.13 for ; Wed, 27 Aug 2014 00:38:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=pa8/1ec+ZLR8yZApAai2i1qjsUb1tFD9k+r9GVBBlsk=; b=d7pL3vYCJ38V2JfLT9fXTjW/fgVAzOiQrtmVhLH9moLmrikH6+K4Y1f4DnDli/MPaU 6kYx/5zZ4a7EnBTI3bXkGhRR0QgcpWvzdow4+O4njYlovsJ3KwM4MMC9noipTpzhJJ1o xH0Vkk4ES2YEUGqpSuKgKVJjSQCYFRUFlQGFRJ6/UGEIAuyIRlMVO6h0zO6+Kv7M4fqJ 0NEUqDe9k342uk+4ozFchBkfIq66inGFwsgjsYOmQoNKjR429Ai1WFxrynB+OQqMwLSD lOzW8E3QrciSNDDFnM6+Nq3uS0CuPlNnj5oTNuG7IBHizI2O3YFUmgUv+871NICHvarj QutQ== MIME-Version: 1.0 X-Received: by 10.152.21.98 with SMTP id u2mr19081896lae.80.1409125080636; Wed, 27 Aug 2014 00:38:00 -0700 (PDT) Received: by 10.112.204.164 with HTTP; Wed, 27 Aug 2014 00:38:00 -0700 (PDT) In-Reply-To: References: <53FB6D4C.9070705@php.net> <53FBAD09.1080108@sugarcrm.com> <53FBC2E1.2000404@sugarcrm.com> Date: Wed, 27 Aug 2014 00:38:00 -0700 Message-ID: To: Ferenc Kovacs Cc: Andrea Faulds , Benjamin Eberlei , Stas Malyshev , Trevor Suarez , Nikita Popov , Sebastian Bergmann , PHP internals Content-Type: multipart/alternative; boundary=089e013d15d4e023320501977f25 Subject: Re: [PHP-DEV] Re: [VOTE] Abstract Syntax Tree From: themattficken@gmail.com (Matt Ficken) --089e013d15d4e023320501977f25 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable 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 wrote: > On Tue, Aug 26, 2014 at 1:58 AM, Andrea Faulds wrote: > > > > > On 26 Aug 2014, at 00:53, Benjamin Eberlei wrote: > > > > > Depends, registering for shutdown handlers and catching fatals is a > > pretty > > > common thing. If EngineException does not get caught, it should produ= ce > > an > > > E_FATAL again with the correct error code. > > > > Exceptions already do this, do they not? I don=E2=80=99t 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 clas= s, > these Pok=C3=A9mon constructs ("Gotta Catch 'Em All") will will swallow t= hose > 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 propos= al > and we can discuss it after it's ready. > > -- > Ferenc Kov=C3=A1cs > @Tyr43l - http://tyrael.hu > --089e013d15d4e023320501977f25--