tony2001 Wed May 16 12:53:36 2007 UTC
Added files:
/php-src/ext/json/tests bug41403.phpt
Modified files:
/php-src/ext/json JSON_parser.c
Log:
fix #41403 (json_decode cannot decode floats if localeconv decimal_point is
not '.')
http://cvs.php.net/viewvc.cgi/php-src/ext/json/JSON_parser.c?r1=1.9&r2=1.10&diff_format=u
Index: php-src/ext/json/JSON_parser.c
diff -u php-src/ext/json/JSON_parser.c:1.9 php-src/ext/json/JSON_parser.c:1.10
--- php-src/ext/json/JSON_parser.c:1.9 Thu Apr 19 22:49:11 2007
+++ php-src/ext/json/JSON_parser.c Wed May 16 12:53:36 2007
@@ -288,7 +288,7 @@
}
else if (type == IS_DOUBLE)
{
- ZVAL_DOUBLE(*z, atof(buf->c));
+ ZVAL_DOUBLE(*z, zend_strtod(buf->c, NULL));
}
else if (type == IS_STRING)
{
http://cvs.php.net/viewvc.cgi/php-src/ext/json/tests/bug41403.phpt?view=markup&rev=1.1
Index: php-src/ext/json/tests/bug41403.phpt
+++ php-src/ext/json/tests/bug41403.phpt
--TEST--
Bug #41403 (json_decode cannot decode floats if localeconv decimal_point is not
'.')
--SKIPIF--
<?php
if (setlocale(LC_NUMERIC, "de_DE") === false) {
die("skip no de_DE locale");
}
?>
--INI--
precision=14
--FILE--
<?php
setlocale(LC_NUMERIC, 'de_DE');
var_dump(json_decode('[2.1]'));
var_dump(json_decode('[0.15]'));
var_dump(json_decode('[123.13452345]'));
var_dump(json_decode('[123,13452345]'));
echo "Done\n";
?>
--EXPECTF--
array(1) {
[0]=>
float(2,1)
}
array(1) {
[0]=>
float(0,15)
}
array(1) {
[0]=>
float(123,13452345)
}
array(2) {
[0]=>
int(123)
[1]=>
int(13452345)
}
Done
--UEXPECTF--
Strict Standards: setlocale(): deprecated in Unicode mode, please use ICU
locale functions in %s on line %d
array(1) {
[0]=>
float(2,1)
}
array(1) {
[0]=>
float(0,15)
}
array(1) {
[0]=>
float(123,13452345)
}
array(2) {
[0]=>
int(123)
[1]=>
int(13452345)
}
Done
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
