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
> Add optional second arg to unserialize()
> Returns the number of bytes consumed by reference for
> streaming unserialization.
> Actual unserialization behavior is not modified at all.
> The need for this came up while trying to parse SplDoublyLinkedList's
> serialization format which uses a non-standard stream of serialized values.
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.