On Mon, Jan 12, 2015 at 6:34 PM, Xinchen Hui <larue...@php.net> wrote: > Hey: > > On Tue, Jan 13, 2015 at 2:06 AM, Jakub Zelenka <bu...@php.net> wrote: >> On Mon, Jan 12, 2015 at 1:40 PM, Xinchen Hui <larue...@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 -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
