Re: [PHP-DEV] [RFC] jsond

2015-01-12 Thread Pierre Joye
On Mon, Jan 12, 2015 at 6:34 PM, Xinchen Hui wrote: > Hey: > > On Tue, Jan 13, 2015 at 2:06 AM, Jakub Zelenka wrote: >> On Mon, Jan 12, 2015 at 1:40 PM, Xinchen Hui wrote: >>> >>> Unfortunately, I don't see performance improvement in wordpress. even >>> slowdown a bit. >>> >>> for IRs , it incre

Re: [PHP-DEV] [RFC] jsond

2015-01-12 Thread Xinchen Hui
Hey: On Tue, Jan 13, 2015 at 2:06 AM, Jakub Zelenka wrote: > On Mon, Jan 12, 2015 at 1:40 PM, Xinchen Hui wrote: >> >> Unfortunately, I don't see performance improvement in wordpress. even >> slowdown a bit. >> >> for IRs , it increased IR by 0.05% while run wordpress homepage for 100 >> times :

Re: [PHP-DEV] [RFC] jsond

2015-01-12 Thread Jakub Zelenka
On Mon, Jan 12, 2015 at 1:40 PM, Xinchen Hui wrote: > Unfortunately, I don't see performance improvement in wordpress. even > slowdown a bit. > > for IRs , it increased IR by 0.05% while run wordpress homepage for 100 > times :< > > do you think there is some room to be improvement? > > thanks >

Re: [PHP-DEV] [RFC] jsond

2015-01-12 Thread Xinchen Hui
Hey: On Mon, Jan 12, 2015 at 6:24 PM, Jakub Zelenka wrote: > Hey > > On Mon, Jan 12, 2015 at 3:01 AM, Xinchen Hui wrote: >> >> >> does master of josnd already works with PHP7? >> > > There is a patch against PHP master that replaces current json ext. I > referenced a branch of my PHP fork in RFC

Re: [PHP-DEV] [RFC] jsond

2015-01-12 Thread Jakub Zelenka
Hey On Mon, Jan 12, 2015 at 3:01 AM, Xinchen Hui wrote: > > does master of josnd already works with PHP7? > > There is a patch against PHP master that replaces current json ext. I referenced a branch of my PHP fork in RFC. I have just created a PR as well to make it a bit easier if you or other

Re: [PHP-DEV] [RFC] jsond

2015-01-11 Thread Remi Collet
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Le 11/01/2015 18:27, Jakub Zelenka a écrit : > Hi, > > This rfc about replacing json with jsond is under discussion. > > https://wiki.php.net/rfc/jsond > +1 -BEGIN PGP SIGNATURE- Version: GnuPG v2 Comment: Using GnuPG with Thunderbird - h

Re: [PHP-DEV] [RFC] jsond

2015-01-11 Thread Xinchen Hui
Hey: On Mon, Jan 12, 2015 at 1:27 AM, Jakub Zelenka wrote: > Hi, > > This rfc about replacing json with jsond is under discussion. > > https://wiki.php.net/rfc/jsond > > I have finally done some benchmarks. You can find a link in the RFC. > > It's been extensively tested for the last few months

Re: [PHP-DEV] [RFC] jsond

2015-01-11 Thread Andrea Faulds
Hey Jakub, > On 11 Jan 2015, at 20:10, Jakub Zelenka wrote: > > Let's first show an example and make clear what the whole thing: > http://3v4l.org/2D72Q > > As you can see, the first type (07, 0xff, .1 and -.1) examples are really > inconsistent and give a different results if there are in the

Re: [PHP-DEV] [RFC] jsond

2015-01-11 Thread Jakub Zelenka
Hi, On Sun, Jan 11, 2015 at 7:07 PM, Paul Dragoonis wrote: > > I feel if we need to make a minor BC break to get this in, then go with it > for PHP 7. > Yes, it's proposed only for PHP 7 as declared in the RFC > I do think that if it's possible for you to detect when users supply this > invalid

Re: [PHP-DEV] [RFC] jsond

2015-01-11 Thread Paul Dragoonis
On 11 Jan 2015 17:28, "Jakub Zelenka" wrote: > > Hi, > > This rfc about replacing json with jsond is under discussion. > > https://wiki.php.net/rfc/jsond > > I have finally done some benchmarks. You can find a link in the RFC. > > It's been extensively tested for the last few months and there is j

Re: [PHP-DEV] [RFC] jsond

2015-01-11 Thread Leigh
On 11 January 2015 at 17:27, Jakub Zelenka wrote: > > I feel that this was a bug in the new parser and we must conform to RFC > 7159 ! > I agree. +1 -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-DEV] [RFC] jsond

2015-01-11 Thread Jakub Zelenka
Hi, This rfc about replacing json with jsond is under discussion. https://wiki.php.net/rfc/jsond I have finally done some benchmarks. You can find a link in the RFC. It's been extensively tested for the last few months and there is just one BC that I'm aware of. The new parser accepts only flo