On 26.09.2008, at 22:06, Hannes Magnusson wrote:

On Fri, Sep 26, 2008 at 21:38, Lukas Kahwe Smith <[EMAIL PROTECTED]> wrote:

and I are a bit worried, that this code did not see that much testing since it was checked in to HEAD quite a while ago. And seeing that the backport is

That sentence worries me a bit. Are you advocating developing new
features in 5.x branches rather then HEAD?

Not at all. I am not sure how you even read that into what I said.

That code has been in HEAD for months, I so no reason not to merge it.
We are in alpha phase for a reason just like this.


All I said was that the fact that it has been in HEAD is no proof that there are no BC issues or even new bugs in it. Again Johannes asked for this to be merged long long long ago and nobody reacted. Now we are at alpha2, hoping to release the last alpha soon. We have plenty of other big stuff that really wants to make it out of the door.

BTW: we have a similar situation for mcrypt. We have "cleanups" (though Derick does not define them as such) in HEAD, which have not been merged to 5.3. I dont like this at all, since either cleanups make sense or they dont. However the larger the cleanups, the messier the old code, the trickier it is to get BC just right and the less I think it makes sense to add such cleanups now.

regards,
Lukas Kahwe Smith
[EMAIL PROTECTED]




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

Reply via email to