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


   

Thread (23 messages)

• Tjerk MeestersThu, 20 Mar 2014 00:03:20 +0000
  • Stas MalyshevThu, 20 Mar 2014 01:04:08 +0000
    • Tjerk MeestersThu, 20 Mar 2014 01:13:24 +0000
      • Stas MalyshevThu, 20 Mar 2014 01:16:58 +0000
        • Tjerk MeestersThu, 20 Mar 2014 02:36:48 +0000
          • Tjerk MeestersThu, 20 Mar 2014 02:41:29 +0000
        • Etienne KneussThu, 20 Mar 2014 16:26:45 +0000
          • Etienne KneussThu, 20 Mar 2014 16:30:44 +0000
          • Stas MalyshevThu, 20 Mar 2014 20:05:47 +0000
            • Etienne KneussThu, 20 Mar 2014 21:21:41 +0000
              • Stas MalyshevThu, 20 Mar 2014 22:02:41 +0000
                • Tjerk MeestersThu, 20 Mar 2014 23:23:11 +0000
                  • Stas MalyshevFri, 21 Mar 2014 01:11:11 +0000
                    • Tjerk MeestersFri, 21 Mar 2014 02:48:48 +0000
                      • Tjerk MeestersTue, 25 Mar 2014 00:24:40 +0000
                        • Levi MorrisonTue, 25 Mar 2014 01:48:17 +0000
                        • Stas MalyshevTue, 25 Mar 2014 03:34:01 +0000
                          • Tjerk MeestersTue, 25 Mar 2014 03:45:23 +0000
                            • Stas MalyshevTue, 25 Mar 2014 03:54:31 +0000
                              • Tjerk MeestersTue, 25 Mar 2014 10:38:45 +0000
                                • Levi MorrisonTue, 25 Mar 2014 13:51:45 +0000
                                  • Tjerk MeestersWed, 26 Mar 2014 04:14:55 +0000
                                    • Tjerk MeestersWed, 26 Mar 2014 04:56:06 +0000