All of the existing tests appear to pass on my system without any changes. As far as I can tell the only difference with the patch is that the rounding behavior of 32bit *nux, Windows machines and 64 bit *nixes will now be consistent. But, we may need one more RC to confirm all is well...

On 29-Oct-08, at 4:52 PM, Hannes Magnusson wrote:

On Wed, Oct 29, 2008 at 21:17, Ilia Alshanetsky <[EMAIL PROTECTED]> wrote:
iliaa           Wed Oct 29 20:17:33 2008 UTC

Modified files:              (Branch: PHP_5_2)
  /php-src    NEWS
  /php-src/ext/standard       math.c
Log:

MFB: Fixed bug #42294 (Unified solution for round() based on C99 round).


Is this really something you want to change minutes before latest RC
of (probably) the latest 5.2 release without any tests what so ever?
It looks to me that the likelihoods of change in behaviour are quite big..

And what about the bug reporters big fat note:
..."DOCUMENT this behaviour so that anybody who wants to use the round
function knows exactly what expects him."?
That screams to me like he anticipates changes in behaviour which
really really should be documented.. and you neither tagged this
commit with [DOC] nor closed the report as "To be documented"?


Kudos for MFB to HEAD though :)

-Hannes

Ilia Alshanetsky





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

Reply via email to