[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] JSON: add %I64d support (Was: Re: [Qemu-devel] system_reset

From: Luiz Capitulino
Subject: Re: [PATCH] JSON: add %I64d support (Was: Re: [Qemu-devel] system_reset command cause assert failed)
Date: Thu, 4 Feb 2010 12:24:14 -0200

On Thu, 4 Feb 2010 10:30:30 +0800
Roy Tam <address@hidden> wrote:

> 2010/2/4 Roy Tam <address@hidden>:
> > 2010/2/3 Luiz Capitulino <address@hidden>:
> OK we are fooled by the json lexer and parser. As we use %I64d to
> print 'long long' variables in Win32, but lexer and parser only deal
> with %lld but not %I64d, this patch add support for %I64d and solve
> 'info pci', 'powser_reset' and 'power_powerdown' assert failure in
> Win32.

 Hm, I guess this has been suggested before... Anthony?

> P.S.: an assert(state.result != NULL) statement in
> qobject_from_jsonv() will be good for asserting failure of parsing
> JSON strings.

 Yes, this change is already in a series I plan to send shortly.

 Thanks a lot for tracking this down.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]