qemu-devel
[Top][All Lists]
Advanced

[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: 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




reply via email to

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