Re: Merge PR 621
| From: | Stas Malyshev | Date: | Thu, 20 Mar 2014 01:04:08 +0000 |
| Subject: | Re: Merge PR 621 | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-73311@lists.php.net to get a copy of this message | ||
Hi!
> https://github.com/php/php-src/pull/621
>
> It makes isset() and empty() call offsetGet() on objects that extend
> ArrayObject if offsetExists() returns TRUE.
Looks at this patch, isn't that what check_empty parameter was supposed
to be doing? Why doesn't it work and it needs to be moved to opcodes?
--
Stanislav Malyshev, Software Architect
SugarCRM: http://www.sugarcrm.com/
(408)454-6900 ext. 227