Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:73481
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain rouvenwessling.de designates 5.35.242.46 as permitted sender)
Content-Type: text/plain; charset=iso-8859-1
Mime-Version: 1.0 (Mac OS X Mail 7.2 \(1874\))
In-Reply-To: <CAEKnhAEFCALpqA8_Gsr-TH_4GUmscrGhLRD70VcAFFV3ij68PQ@mail.gmail.com>
Date: Sun, 30 Mar 2014 20:17:42 +0200
Cc: PHP internals list <internals@lists.php.net>
Content-Transfer-Encoding: quoted-printable
Message-ID: <7B4E1A3B-A762-4FE6-A9FB-7CE9898B495C@rouvenwessling.de>
References: <CAEKnhAEFCALpqA8_Gsr-TH_4GUmscrGhLRD70VcAFFV3ij68PQ@mail.gmail.com>
To: Jakub Zelenka <bukka@php.net>
Subject: Re: [PHP-DEV] New JSON parser
From: me@rouvenwessling.de (=?iso-8859-1?Q?Rouven_We=DFling?=)


On 13.03.2014, at 20:48, Jakub Zelenka <bukka@php.net> wrote:

> Hi,
>=20
> I have create a new JSON parser using conditional re2c and pure pull =
Bison
> parser. It's a native UTF-8 parser licensed under PHP license (it can =
be
> used for Evil though :) ). The extension is available at
>=20
> https://github.com/bukka/php-jsond
>=20
> The encoder is taken from the current ext/json but the decoder has =
been
> completely rewritten.

While looking at this code I noticed that the encoder also does a UTF-8 =
-> UTF-16 conversion that can be avoided. I've written a patch to avoid =
doing so: https://github.com/php/php-src/pull/636=20

The performance improvement is not as awesome as jsond, just 15-30%. =
Interestingly it also makes decoding slightly faster.

In the process I've also spotted some unused code lying around, removal =
is here: https://github.com/php/php-src/pull/637

Best regards
Rouven=