Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:80434
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.215.53 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CABBJUpdRhuyPHZRjrCFqD8eR=V-p7Zde3QXKKrxZpeBD8N1WQA@mail.gmail.com>
References: <CAEKnhAFvQpPxkoZKJ-2A-Fjm_ABqxPNMuJoqOsby3ELZgoyQ9g@mail.gmail.com>
	<CABBJUpdxT0mnqAB4Tit_F=s8RjX9Ydh_jreU3McnzUzOc7LuOQ@mail.gmail.com>
	<CAEKnhAFi7HdJpVbRooy==1fGvg-50z76LehRxs60vZ7zb_wQ+g@mail.gmail.com>
	<CABBJUpd_WqOEdVG86rjcSNxOXnxPJj5+WKz+J6DUXAh_XGLikw@mail.gmail.com>
	<CAEKnhAFuaURMQEHHbfWFMSQW55GPWpy-oEQ9v8EF4NO1GvOq8A@mail.gmail.com>
	<CABBJUpdRhuyPHZRjrCFqD8eR=V-p7Zde3QXKKrxZpeBD8N1WQA@mail.gmail.com>
Date: Mon, 12 Jan 2015 18:51:27 -0800
Message-ID: <CAEZPtU5rFX6NFfKR4ZVThkXAAMUHckZrjBLArXiY9LbcrpMSgQ@mail.gmail.com>
To: Xinchen Hui <laruence@php.net>
Cc: Jakub Zelenka <bukka@php.net>, PHP internals list <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] [RFC] jsond
From: pierre.php@gmail.com (Pierre Joye)

On Mon, Jan 12, 2015 at 6:34 PM, Xinchen Hui <laruence@php.net> wrote:
> Hey:
>
> On Tue, Jan 13, 2015 at 2:06 AM, Jakub Zelenka <bukka@php.net> wrote:
>> On Mon, Jan 12, 2015 at 1:40 PM, Xinchen Hui <laruence@php.net> 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
>>
>>
>> To be honest I expected that there won't be a big difference unless
>> json_decode is heavily used for bigger json instances.
>>
>> Anyway the decoder is not fully optimized yet and there definitely is a room
>> for improvents. It's basically just an initial port for PHP 7. But I think
>> that it's already good enough to replace the existing json ext as my
>> benchmarks for instances containing large strings already gives much better
>> results.
>>
>> And, of course, it is a PHP licensed parser... :)
>>
> sure, of course, :)

btw, it is important that this point alone justifies this RFC (and us
accepting it as there is no impact on perf and could be better). We
have an issue with the current extension due to the license of the
underlying implementation. Many distros already use alternative
implementations. To solve this problem will make everyone work easier.



-- 
Pierre

@pierrejoye | http://www.libgd.org