Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:73226 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 37328 invoked from network); 17 Mar 2014 17:45:56 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 17 Mar 2014 17:45:56 -0000 X-Host-Fingerprint: 217.114.215.11 experimentalworks.net Date: Mon, 17 Mar 2014 12:45:55 -0500 Received: from [217.114.215.11] ([217.114.215.11:23758] helo=localhost.localdomain) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 3E/8A-17561-3D437235 for ; Mon, 17 Mar 2014 12:45:55 -0500 To: internals@lists.php.net References: <5322BFA6.7060707@fedoraproject.org> User-Agent: slrn/pre1.0.0-18 (Linux) Message-ID: X-Posted-By: 217.114.215.11 Subject: Re: [PHP-DEV] New JSON parser From: dsp@php.net (David Soria Parra) Jakub Zelenka schrieb: > --001a11330a7413861304f4aa17a7 > Content-Type: text/plain; charset=ISO-8859-1 > > On Fri, Mar 14, 2014 at 8:36 AM, Remi Collet wrote: > >> >> Can you pull all the test suite from official ext/json to check if it is >> a dropin alternative ? > > > Done! ;) I just renamed json to jsond. All tests pass for me. Just one skip > for 32bit test (pass001.1.phpt) but it's 64bit variant is fine. If someone > could try that test, that would be great. > > It's tested only with PHP master branch and apparently not compile under > 5.5. I will have a look on older PHP versions and fix it later. Also need > to improve build and do more benchmarking. > > Any testing, ideas and comments will be appreciated... ;) > > Thanks > > Jakub > > --001a11330a7413861304f4aa17a7-- have you been testing this with a json testing library like https://code.google.com/p/json-test-suite/ ? I really like your project, but we have to be very very careful when it comes to json adn make sure it supports everything the current json lib supports. -David