Re: New JSON parser
| From: | David Soria Parra | Date: | Mon, 17 Mar 2014 17:45:55 +0000 |
| Subject: | Re: New JSON parser | ||
| References: | 1 2 3 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-73226@lists.php.net to get a copy of this message | ||
Jakub Zelenka <bukka@php.net> schrieb:
> --001a11330a7413861304f4aa17a7
> Content-Type: text/plain; charset=ISO-8859-1
>
> On Fri, Mar 14, 2014 at 8:36 AM, Remi Collet <remi@fedoraproject.org> 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