Moriyoshi,

Well, the only thing we could do at this point is to re-tag the release with the change delaying the release by a week. You know where I stand on the matter of the code default value, but if there is a consensus on internal that this change needs to happen, then this can definitely be done. However, looking at the past discussion on internals between Andrei and yourself I don't see anyone objecting to the current implementation, so I see no issue with proceeding as is. Let's reach a conclusion by end of day (EST time) so release can either be made or delayed.


On 25-Feb-09, at 11:29 PM, Moriyoshi Koizumi wrote:

In addition, I didn't insist that the whole thing should be reverted.
I just argue about the default value.

Moriyoshi

On Thu, Feb 26, 2009 at 1:27 PM, Moriyoshi Koizumi <m...@mozo.jp> wrote:
The issue is not whether it is valid or not. Andrei's patch broke the
backwards compatibility that affects real-world applications and thus
cannot be regarded as an improvement.

Moriyoshi

On Thu, Feb 26, 2009 at 11:51 AM, Ilia Alshanetsky <i...@prohost.org> wrote:
Moriyoshi,

My opinion is that the current implementation is valid, which is why
Andrei's patch was accepted in the first place.


On 25-Feb-09, at 9:34 PM, Moriyoshi Koizumi wrote:

Ilia,

We didn't resolve the array_unique() issue yet. Please take this into
account.

Moriyoshi

2009/2/25 Ilia Alshanetsky <il...@php.net>:

iliaa           Wed Feb 25 14:56:51 2009 UTC

 Modified files:              (Branch: PHP_5_2)
  /php-src    configure.in NEWS
  /php-src/main       php_version.h
 Log:

 5.2.9

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php


Ilia Alshanetsky







Ilia Alshanetsky





--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to