[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/11] json parser (v2)
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH 0/11] json parser (v2) |
Date: |
Sun, 18 Oct 2009 19:34:52 -0200 |
On Sat, 17 Oct 2009 08:36:00 -0500
Anthony Liguori <address@hidden> wrote:
> I fat-fingered the git send-email command the first time so here's the second
> iteration.
Got it now..
> This series introduces a json parsing framework. The parser is a recursive
> decent parser and implements the full json spec except for the null keyword.
> We
> need to decide how we want to model null in order to support this.
>
> In addition, we implement an extension to allow single quote strings. This
> is very useful for defining dictionaries in C.
>
> The parser has a full test suite and introduces a boolean and float type.
Very nice, this is obviously way beyond the simple approach I was following,
I have some minor comments and will reply some patches. Will also run some
tests later today, but I don't think I'll find any issue (given that you have
written unit-tests!).
What's you merge plan for your own patches? :)
- [Qemu-devel] [PATCH 07/11] qjson: Use QBool for true/false keywords, (continued)
- [Qemu-devel] [PATCH 05/11] Add unit test for json parser, Anthony Liguori, 2009/10/17
- [Qemu-devel] [PATCH 09/11] qjson: add unit test for varargs bool parsing, Anthony Liguori, 2009/10/17
- [Qemu-devel] [PATCH 10/11] qjson: add vararg format for embedded qobjects, Anthony Liguori, 2009/10/17
- [Qemu-devel] [PATCH 11/11] qjson: add unit test to check %p format, Anthony Liguori, 2009/10/17
- Re: [Qemu-devel] [PATCH 0/11] json parser (v2),
Luiz Capitulino <=