|
From: | Anthony Liguori |
Subject: | Re: [PATCH] JSON: add %I64d support (Was: Re: [Qemu-devel] system_reset command cause assert failed) |
Date: | Mon, 08 Feb 2010 10:21:54 -0600 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.5) Gecko/20091209 Fedora/3.0-4.fc12 Lightning/1.0pre Thunderbird/3.0 |
On 02/04/2010 10:59 AM, Roy Tam wrote:
2010/2/4 Luiz Capitulino<address@hidden>: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?OK I just missed this. And the wheel was reinvented. :-S http://www.mail-archive.com/address@hidden/msg23983.html
I asked for the json parser changes to be split from the PRId64 changes. It was never resubmitted. I'll apply your patch.
Regards, Anthony Liguori
[Prev in Thread] | Current Thread | [Next in Thread] |