Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:73992 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 786 invoked from network); 7 May 2014 04:31:53 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 7 May 2014 04:31:53 -0000 Authentication-Results: pb1.pair.com header.from=tjerk.meesters@gmail.com; sender-id=pass Authentication-Results: pb1.pair.com smtp.mail=tjerk.meesters@gmail.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.128.170 as permitted sender) X-PHP-List-Original-Sender: tjerk.meesters@gmail.com X-Host-Fingerprint: 209.85.128.170 mail-ve0-f170.google.com Received: from [209.85.128.170] ([209.85.128.170:59060] helo=mail-ve0-f170.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id A7/10-32043-737B9635 for ; Wed, 07 May 2014 00:31:51 -0400 Received: by mail-ve0-f170.google.com with SMTP id db11so564044veb.15 for ; Tue, 06 May 2014 21:31:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:cc:content-type; bh=0DjDwzMEz6LackSixepzy0Ac0WqI7WqSQUFc4jwnw5c=; b=dG0t+LWlGS7L8etGtAZyBf9DAu2KsAilgpKmFF8uspxfQEftrfCubXZ1cRHI9Fawgu 7R89bdaj05EMVH2X5WpeQGns16qgRwJos4EMUjjg/vFPnwDqDGPfBchEWt7uzLRTA3un RL0sISE/RakiwXNw1jT8lk7q8WoHwcdmHCt5TvgZDfsfkrtGPzebkTh9XLMM36v27Emc FUscHs8FRPqo5+MRNPrE8r5Zuucgn9JlSyUqcULTPXqOhhud6SfDZLBRw2Xj7P5mU64f HhsxGadg3liBof6YwiJTc+xGEGQaRt9kb7eJYTMFqNaWg5yMpaz60YR4NpEkA2Z1Ybht jl1A== MIME-Version: 1.0 X-Received: by 10.52.37.130 with SMTP id y2mr3727625vdj.38.1399437108834; Tue, 06 May 2014 21:31:48 -0700 (PDT) Sender: tjerk.meesters@gmail.com Received: by 10.58.133.84 with HTTP; Tue, 6 May 2014 21:31:48 -0700 (PDT) In-Reply-To: References: Date: Wed, 7 May 2014 12:31:48 +0800 X-Google-Sender-Auth: bG82N17SstA1xSVcMDFsqrVSAbs Message-ID: To: Dmitry Stogov Cc: Julien Pauli , PHP Internals Content-Type: multipart/alternative; boundary=bcaec51d20e0c1f6be04f8c7d7b0 Subject: Re: 5dee3c11 break From: datibbaw@php.net (Tjerk Anne Meesters) --bcaec51d20e0c1f6be04f8c7d7b0 Content-Type: text/plain; charset=UTF-8 Hi, On Tue, May 6, 2014 at 7:50 PM, Dmitry Stogov wrote: > Ah, I didn't get what did you mean by (check_empty == 2), but they > probably should be changed into (check_empty != 1) > > It fixes the problem. > Could you please verify and commit into all relevant branches. > > Thanks. Dmitry. > > --- a/ext/spl/spl_array.c > +++ b/ext/spl/spl_array.c > @@ -603,7 +603,7 @@ static int spl_array_has_dimension_ex(int > check_inherited, zval *object, zval *o > > if (rv && zend_is_true(rv TSRMLS_CC)) { > zval_ptr_dtor(&rv); > - if (check_empty == 2) { > + if (check_empty != 1) { > return 1; > } else if (intern->fptr_offset_get) { > value = spl_array_read_dimension_ex(1, > object, offset, BP_VAR_R TSRMLS_CC); > @@ -622,7 +622,7 @@ static int spl_array_has_dimension_ex(int > check_inherited, zval *object, zval *o > switch(Z_TYPE_P(offset)) { > case IS_STRING: > if (zend_symtable_find(ht, > Z_STRVAL_P(offset), Z_STRLEN_P(offset)+1, (void **) &tmp) != FAILURE) { > - if (check_empty == 2) { > + if (check_empty != 1) { > return 1; > } > } else { > @@ -639,7 +639,7 @@ static int spl_array_has_dimension_ex(int > check_inherited, zval *object, zval *o > index = Z_LVAL_P(offset); > } > if (zend_hash_index_find(ht, index, (void > **)&tmp) != FAILURE) { > - if (check_empty == 2) { > + if (check_empty != 1) { > return 1; > } > } else { > This patch broke an earlier test case, so I've refactored it into the following compromise (until PHP.next): https://gist.github.com/datibbaw/55f0dc8e95f45b91e514 Does that look okay to you? > > > > On Tue, May 6, 2014 at 3:29 PM, Tjerk Anne Meesters wrote: > >> Hi, >> >> >> On Tue, May 6, 2014 at 7:25 PM, Dmitry Stogov wrote: >> >>> I didn't review you patch careful, but why do you need to call >>> offsetGet() when offsetExists() must be enough? >>> is it for empty()? >>> >> >> Mostly for empty(), yes. It fixes the expected empty() behaviour for >> `$a['foo'] == 0` when using ArrayObject. This is further explained in this >> bug report: https://bugs.php.net/bug.php?id=66834 >> >> >>> Then you probably should implement it a way similar to >>> zend_std_has_dimension() in Zend/zend_object_handlers.c. >>> call offsetExists() and then offsetGet() if necessary. >>> >> >> That's already the case :) >> >> http://lxr.php.net/xref/PHP_TRUNK/Zend/zend_object_handlers.c#719 >> >> >>> >>> Thanks. Dmitry. >>> >>> >>> On Tue, May 6, 2014 at 2:31 PM, Tjerk Anne Meesters wrote: >>> >>>> Hi Dmitry, >>>> >>>> On Tue, May 6, 2014 at 4:11 PM, Dmitry Stogov wrote: >>>> >>>>> Hi Tjerk, >>>>> >>>>> your commit broke the code that worked fine before (still works in 5.5 >>>>> but broken in 5.6 and above). >>>>> It leads into infinity recursion until stack overflow. >>>>> >>>>> It must be fixed or reverted. >>>>> >>>> >>>> This has been mentioned by Jakub before and a fix to ZF2 has already >>>> been merged: >>>> >>>> https://github.com/zendframework/zf2/pull/6096 >>>> >>>> The previous code and my patch basically cannot coexist; it used to >>>> work in 5.6 before, but only by the "virtue" of an unfortunate >>>> implementation. >>>> >>>> I believe this is not the only 5.6 issue that ZF2 is dealing with, but >>>> if you feel that this breaks too many things for a 5.x release I suppose we >>>> can revert it in PHP-5.6 and keep it for PHP-6? >>>> >>>> Let me know. >>>> >>>> >>>>> Thanks. Dmitry. >>>>> >>>>> >>>> class Parameters extends ArrayObject { >>>>> public function __construct(array $values = null) { >>>>> if (null === $values) { >>>>> $values = array(); >>>>> } >>>>> parent::__construct($values, ArrayObject::ARRAY_AS_PROPS); >>>>> } >>>>> public function offsetGet($name) { >>>>> if (isset($this[$name])) { >>>>> return parent::offsetGet($name); >>>>> } >>>>> return null; >>>>> } >>>>> } >>>>> $x = new Parameters(); >>>>> var_dump($x['foo']); >>>>> $x['foo'] = 'bar'; >>>>> var_dump($x['foo']); >>>>> ?> >>>>> >>>> >>>> >>>> >>>> -- >>>> -- >>>> Tjerk >>>> >>> >>> >> >> >> -- >> -- >> Tjerk >> > > -- -- Tjerk --bcaec51d20e0c1f6be04f8c7d7b0--