Re: [PHP-CVS] com php-src: Add optional second arg to unserialize(): ext/standard/basic_functions.c ext/standard/tests/serialize/serialization_error_001.phpt ext/standard/tests/serialize/unserialize_c

2013-07-04 Thread Stas Malyshev
Hi! Please add a note to UPGRADING as well. Thanks! I understand UPGRADING still not updated? Could you please update it? Cheers, On Fri, May 17, 2013 at 12:18 AM, Sara Golemon poll...@php.net wrote: Commit:cfd104582220d578ab1b78a5991065d038e1f931 Author:Sara Golemon

Re: [PHP-CVS] com php-src: Add optional second arg to unserialize(): ext/standard/basic_functions.c ext/standard/tests/serialize/serialization_error_001.phpt ext/standard/tests/serialize/unserialize_c

2013-05-17 Thread Nikita Popov
On Fri, May 17, 2013 at 12:18 AM, Sara Golemon poll...@php.net wrote: Commit:cfd104582220d578ab1b78a5991065d038e1f931 Author:Sara Golemon poll...@php.net Thu, 16 May 2013 14:37:36 -0700 Parents: bc656cde0453aa6de50812ba9edfc6f74c61ce37 Branches: master Link:

Re: [PHP-CVS] com php-src: Add optional second arg to unserialize(): ext/standard/basic_functions.c ext/standard/tests/serialize/serialization_error_001.phpt ext/standard/tests/serialize/unserialize_c

2013-05-17 Thread Stas Malyshev
Hi! Imho this should have been briefly discussed on internals beforehand. At least to me this seems rather dubious (why is it needed? why are you manually parsing a part of the DLL serialization?) and also clashes with Stas' proposed changes to unserialize. I don't remember any discussion

Re: [PHP-CVS] com php-src: Add optional second arg to unserialize(): ext/standard/basic_functions.c ext/standard/tests/serialize/serialization_error_001.phpt ext/standard/tests/serialize/unserialize_c

2013-05-16 Thread Pierre Joye
hi Sara! Please add a note to UPGRADING as well. Thanks! -- Cheers, On Fri, May 17, 2013 at 12:18 AM, Sara Golemon poll...@php.net wrote: Commit:cfd104582220d578ab1b78a5991065d038e1f931 Author:Sara Golemon poll...@php.net Thu, 16 May 2013 14:37:36 -0700 Parents: