Re: [Xen-devel] Re: [PATCH V7 6/7] libxl: Introduce JSON parsing stuff.

[Anthony PERARD <anthony.perard@xxxxxxxxxx>]

On Thu, Jul 21, 2011 at 15:40, Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx> wrote:
> On Wed, 2011-07-20 at 22:24 +0100, Anthony PERARD wrote:
>> +    /* parse the input */
>> +    if (yajl_ctx.hand == NULL) {
>> +        /* allow comments */
>> +        yajl_parser_config cfg = { 1, 1 };
>
> Looks like YALJ v2 has enums for these config values but v1 doesn't.
> However
>            yajl_parse_config cfg = {
>                .allowComments = 1,
>                .checkUTF8 = 1,
>            };
> is a bit more self documenting.
>
> (supporting yajl 1 and 2 is going to be a pain. but lets cross that
> bridge when we come to it...)

With the V2, there is "yajl_tree_parse" that basicly do the same as
libxl__json_parse. I try to be closer to the tree implementation of
the V2, but yes, it will sill be painfull to support both. If we can
only support the v1, and change to the v2 later (when widly availlable
in distribution), it should be OK.

-- 
Anthony PERARD

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel