[PHP-CVS] cvs: php-src /ext/standard array.c /ext/standard/tests/array bug34934.phpt

2005-10-24 Thread Dmitry Stogov
dmitry Mon Oct 24 03:01:44 2005 EDT Removed files: /php-src/ext/standard/tests/array bug34934.phpt Modified files: /php-src/ext/standard array.c Log: Reverted fix for bug #34934. The bug marked as bogus.

RE: [PHP-CVS] cvs: php-src /ext/standard array.c /ext/standard/tests/array bug34934.phpt

2005-10-24 Thread Dmitry Stogov
/standard array.c /ext/standard/tests/array bug34934.phpt Hello Dmitry, this is wrong! We agreed upon not allowing ArrayAccess in array* functions. If you change this then change the macros in array.c from Z_ARRVAL to HASH_OF and ensure that all functions also work with ArrayAccess

[PHP-CVS] cvs: php-src /ext/standard array.c /ext/standard/tests/array bug34934.phpt

2005-10-21 Thread Dmitry Stogov
dmitry Fri Oct 21 11:20:31 2005 EDT Modified files: /php-src/ext/standard array.c /php-src/ext/standard/tests/array bug34934.phpt Log: Fixed bug #34934 (offsetExists is not called from array_key_exists)

Re: [PHP-CVS] cvs: php-src /ext/standard array.c /ext/standard/tests/array bug34934.phpt

2005-10-21 Thread Marcus Boerger
Hello Dmitry, this is wrong! We agreed upon not allowing ArrayAccess in array* functions. If you change this then change the macros in array.c from Z_ARRVAL to HASH_OF and ensure that all functions also work with ArrayAccess. Since you cannot do that you might probably explain why the bug