Re: [PHP-CVS] com php-src: Add optional depth parameter to json_encode #62369: ext/json/json.c ext/json/php_json.h ext/json/tests/bug62369.phpt

2012-07-26 Thread Stas Malyshev
Hi! Commit:45d596ea1e32792c7b7b7f28be220dea861b6708 Author:Florian Anderiasch f...@php.net Tue, 24 Jul 2012 13:15:16 +0200 Parents: dd9d64b21e4bbc8a106a6156dc6ffefbcc33ec02 Branches: master Link:

Re: [PHP-CVS] com php-src: Add optional depth parameter to json_encode #62369: ext/json/json.c ext/json/php_json.h ext/json/tests/bug62369.phpt

2012-07-26 Thread Pierre Joye
hi! On Thu, Jul 26, 2012 at 9:16 AM, Stas Malyshev smalys...@sugarcrm.com wrote: Hi! Commit:45d596ea1e32792c7b7b7f28be220dea861b6708 Author:Florian Anderiasch f...@php.net Tue, 24 Jul 2012 13:15:16 +0200 Parents: dd9d64b21e4bbc8a106a6156dc6ffefbcc33ec02 Branches: master

Re: [PHP-CVS] com php-src: Add optional depth parameter to json_encode #62369: ext/json/json.c ext/json/php_json.h ext/json/tests/bug62369.phpt

2012-07-26 Thread Florian Anderiasch
On 26.07.2012 14:14, Pierre Joye wrote: Was it ever discussed? afair we did not want to add that, back then when we discussed the security issue related to huge input data. Hi, I hadn't seen any discussion apart from Laruence's change to FR so I went ahead and implemented it. The default