On Tue, May 6, 2014 at 4:01 AM, Laruence <[email protected]> wrote:
> On Tue, May 6, 2014 at 6:25 AM, Pierre Joye <[email protected]> wrote:
> > hi,
> >
> > Some comments about the future ideas:
> >
> > . replace the memory manager
> >
> > If you are working on it now, please do it in public so other can
> > contribute and ensure that it will work on all supported platforms and
> > SAPIs. Debugging a huge patch is much harder and time consuming than
> > constantly improving something.
> >
> > - refactoring zpp
> >
> > this idea fits very well with what we need for the 64 bit branch as
> > well. Some coop are necessary here as well.
> >
> > - jsond
> >
> > Unless there are some very bad BC breaks, I would really like to see
> > it merged in master very soon. This is the best way to get it stable
> > and tested.
> Hey:
> I was working on it yesterday, but, it turns out, the parser of it
> assume array and object properties are all Hashtables..
>
> unfortunately, in ng, array is zend_array now... so, it will need
> a little more works to do...
>
> anyway: I will keep working on it, but not now... :)
>
> thanks
>
Yeah it looks that it will require some changes in the parser. I'll
definitely have a look... ;)
It could make sense to replace json with jsond in the phpng branch? There
are no bad BC breaks... :) It would also help to stabilize it as this
branch probably will be quite heavily tested... I could also work more on
this branch and help to fix some other issues...
Regards
Jakub