Em Tue, 17 Jul 2012 23:49:55 +0200, Stas Malyshev <smalys...@sugarcrm.com>
escreveu:
Commit: b47d6b32ba8d6dc1b9d1fc91a83eef29a28363c5
Author: Reeze Xia <reeze....@gmail.com> Tue, 17 Jul 2012
23:01:20 +0800
Committer: Gustavo André dos Santos Lopes <cataphr...@php.net>
Tue, 17 Jul 2012 23:08:11 +0200
Parents: d0e58bf6eb54676a711d98d3b72a13e828e2deef
Branches: PHP-5.4 master
Link:
http://git.php.net/?p=php-src.git;a=commitdiff;h=b47d6b32ba8d6dc1b9d1fc91a83eef29a28363c5
Log:
Fix test fails: ext/standard/tests/general_functions/bug27678.phpt
After commit 3e62aae1, number_format() returns string with length,
but _php_math_number_format_ex_len() didn't set string length
on nan and inf. This cause segfault when destruct the return value.
Thanks for fixing it! However that means I'd have to revert the original
fix 3e62aae1b456440328af4153524e22679b84f68a from 5.4.5, since it's not
correct but I don't want to make a new RC out of it which would be
necessary if I added this new patch.
I think that's overly cautious given the simplicity of the problem and the
fix, but it's your call.
But even as a matter of principle, I don't know why you'd need a new RC in
one case, but not in the other. You'd have a release different from the
last RC in both cases. It's true that the revert would only restore old
code, but, in general, you can't be sure the old code is still correct in
the context of the other changes since the last release (i.e. something
else could rely on the change. In principle).
--
Gustavo Lopes
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
