Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:67303
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain fedoraproject.org from 212.27.42.5 cause and error)
Message-ID: <5185271B.9040007@fedoraproject.org>
Date: Sat, 04 May 2013 17:19:55 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130403 Thunderbird/17.0.5
MIME-Version: 1.0
To: PHP Internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Subject: Proposal: new implentation of JSON extension.
From: remi@fedoraproject.org (Remi Collet)

Raise again this discussion as no feedback on pecl ML.

Trying to solve https://bugs.php.net/bug.php?id=63520

I start to work on a dropin alternative for current json extension in php.


Current work:
https://github.com/remicollet/pecl-json-c


It is obviously to late for PHP 5.5, and need probably more feedback,
more tests, and more work before a RFC for PHP 5.6 (or later).

* json_encode : same code than php 5.5, no difference

* json_decode : wrapper over json-c 0.11 library.

Difference:
- error code returned by the parser are different (so only
PHP_JSON_ERROR_SYNTAX and PHP_JSON_ERROR_DEPTH are used)
- number only parse as integer (no automatic cast to double)
- loose of performance (but still acceptable, I think)

New feature: as json-c is an incremental parser, a new class is
available to allow incremental parsing (for large json encoded data)



Proposal:

1/	update existing pecl/json extension

	- on pecl for older PHP 5.3 - 5.5
	- on php-src in master / 5.6

OR

2/	new pecl/json-c extension

	- PHP will keep the non free code


Should I open a RFC ?


Regards,
Remi.


P.S. I have tried to contact Omar, original author of this extension,
without any success.