dmitry Wed Jun 13 13:38:26 2007 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/simplexml simplexml.c
/php-src/ext/simplexml/tests027.phpt bug35785.phpt bug41582.phpt
Log:
Fixed wrong fix for bug #41582
http://cvs.php.net/viewvc.cgi/php-sr
dmitry Wed Jun 13 13:39:00 2007 UTC
Modified files:
/php-src/ext/simplexml simplexml.c
/php-src/ext/simplexml/tests027.phpt bug35785.phpt bug41582.phpt
Log:
Fixed wrong fic for bug #41582
http://cvs.php.net/viewvc.cgi/php-src/ext/simplexml/s
iliaa Wed Jun 13 17:07:59 2007 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/json json.c JSON_parser.c
/php-src/ext/json/tests pass001.1.phpt
/php-srcNEWS
Log:
Fixed bug #41673 (json_encode breaks large numbers in arrays).
http://cvs.ph
iliaa Wed Jun 13 17:10:06 2007 UTC
Modified files:
/php-src/ext/json json.c JSON_parser.c
Log:
MFB: Fixed bug #41673 (json_encode breaks large numbers in arrays).
http://cvs.php.net/viewvc.cgi/php-src/ext/json/json.c?r1=1.26&r2=1.27&diff_format=u
Index: php-
+ double d = zend_strtod(buf->c, NULL);
+ if (d > LONG_MAX) {
Shouldn't it be d > LONG_MAX || d < -LONG_MAX?
-int(4)
+%s(4)
If it's 32-bit it's always float, no chance to get int(4)
from PHP, so why %s?
--
Stanislav Malyshev, Zend Pr
The test is universal, on 64 bit machines you will get int because
PHP int is actually a long. The -LONG_MAX is a good point though.
On 13-Jun-07, at 1:11 PM, Stanislav Malyshev wrote:
+ double d = zend_strtod(buf->c, NULL);
+ if (d > LONG_MAX) {
Shouldn't it be d > LONG_MAX |
iliaa Wed Jun 13 17:56:41 2007 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/json JSON_parser.c
Log:
Handle very small longs via double
http://cvs.php.net/viewvc.cgi/php-src/ext/json/JSON_parser.c?r1=1.1.2.11&r2=1.1.2.12&diff_format=u
Index: php-src/e
iliaa Wed Jun 13 17:57:10 2007 UTC
Modified files:
/php-src/ext/json JSON_parser.c
Log:
MFB: Handle very small longs via double
http://cvs.php.net/viewvc.cgi/php-src/ext/json/JSON_parser.c?r1=1.14&r2=1.15&diff_format=u
Index: php-src/ext/json/JSON_parser.
The test is universal
That's why it has this:
if (PHP_INT_SIZE != 4) die("skip this test is for 32bit platform only");
at the beginning, I guess? ;)
--
Stanislav Malyshev, Zend Products Engineer
[EMAIL PROTECTED] http://www.zend.com/
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscr
There is no reason for it to be such, I will probably merge with the
separate 64bit test.
On 13-Jun-07, at 1:57 PM, Stanislav Malyshev wrote:
The test is universal
That's why it has this:
if (PHP_INT_SIZE != 4) die("skip this test is for 32bit platform
only");
at the beginning, I gue
kraghubaWed Jun 13 22:38:49 2007 UTC
Added files: (Branch: PHP_5_2)
/php-src/ext/standard/tests/filefwrite_variation-win32.phpt
fwrite_basic-win32.phpt
filegroup_basic.phpt
11 matches
Mail list logo
