Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:76123 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 284 invoked from network); 25 Jul 2014 14:36:33 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 25 Jul 2014 14:36:33 -0000 Authentication-Results: pb1.pair.com header.from=tyra3l@gmail.com; sender-id=pass Authentication-Results: pb1.pair.com smtp.mail=tyra3l@gmail.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.192.53 as permitted sender) X-PHP-List-Original-Sender: tyra3l@gmail.com X-Host-Fingerprint: 209.85.192.53 mail-qg0-f53.google.com Received: from [209.85.192.53] ([209.85.192.53:36149] helo=mail-qg0-f53.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id D6/34-08559-07B62D35 for ; Fri, 25 Jul 2014 10:36:32 -0400 Received: by mail-qg0-f53.google.com with SMTP id q107so4982937qgd.26 for ; Fri, 25 Jul 2014 07:36:35 -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=eNopiRfDNVvxbfB8bf+JiQQJtVxMOc3ZXRMhCNoVEoI=; b=KWd76oiF5C7+PjZQ8Inb8/JxLASOo6friT4WgWw78V97Pz2egSfbRmiqcJ1bZ9o6/Q /gvsDb4iivMNCBEkC5T9BjKy94RDC6eTXVNGKec26pOa78TTUzCOvPDCPwxcYY3QLxV7 DLQ1Qx0wi1Qh3fE4TFRhHH5d/HdHR0aNeUFIBB0qDjW2oqpyABYGNPz9YFUfV2c4u8Ut okd1uPoH6ZHSO2NrJzxlwl0AsVo58s186DbqfWFVWpP5ewHa5gZYDsUHR+45H2pJR8GV vXmfpQYorBIUt8cyj4gzIaw4W5GSgejWY+OSdSR9TuwpYNkeY8KesdpAwNHlMM9wufqX lsnA== MIME-Version: 1.0 X-Received: by 10.140.88.41 with SMTP id s38mr16017532qgd.73.1406298995409; Fri, 25 Jul 2014 07:36:35 -0700 (PDT) Received: by 10.140.102.111 with HTTP; Fri, 25 Jul 2014 07:36:35 -0700 (PDT) In-Reply-To: References: <53D01C19.9050606@sugarcrm.com> Date: Fri, 25 Jul 2014 16:36:35 +0200 Message-ID: To: Bob Weinand Cc: Stas Malyshev , Dmitry Stogov , Nikita Popov , Julien Pauli , PHP Internals Content-Type: multipart/alternative; boundary=001a11c13e7e11d8d204ff05809f Subject: Re: [PHP-DEV] Weird constant expression syntax and bug From: tyra3l@gmail.com (Ferenc Kovacs) --001a11c13e7e11d8d204ff05809f Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Wed, Jul 23, 2014 at 10:45 PM, Bob Weinand wrote: > Am 23.7.2014 um 22:33 schrieb Stas Malyshev : > > Hi! > > > >> It was a restriction to not support arrays in constant context. It see= ms > >> like nobody can remember why it was introduced. > > > > My vague recollection is that it had some troubles with keeping > > refcounts consistent, esp. withing bytecode caching context, but it may > > be a false memory :) > > That doesn't really make much sense to me, at least not with current > engine... > As said, I've tested it and didn't find issues. Maybe I just test the > wrong things=E2=80=A6 > > >> However, I think it's too dangerous to break it in last minute before > >> release. > > > > We definitely need to fix the WTF with "no runtime use" for defined > > constant and the segfault before the release. I think for the arrays, i= f > > we can't have it working properly we'd better not have array support > > there for 5.6.0 and fix it in 5.6.1 than have this weird "no runtime > > use" thing. > > Well, we still could fix it now, AFAIK, we will still have another RC now > and when we fix it only for 5.6.1, there also will only be one RC in > between. > That's no gain in testing time etc.. > > Bob > > > -- > > Stanislav Malyshev, Software Architect > > SugarCRM: http://www.sugarcrm.com/ > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php > > yeah, there will be another RC for sure, but we have to have some resulotion for that. I think it would be safer to remove stuff (even if just the array support) than trying to fix in in a hurry, as it would probably just cause shipping the final with some other bug(s) or having a couple of other RCs. --=20 Ferenc Kov=C3=A1cs @Tyr43l - http://tyrael.hu --001a11c13e7e11d8d204ff05809f--