qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 00/36] Rip out dynamic JSON parsing


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH 00/36] Rip out dynamic JSON parsing
Date: Wed, 30 Nov 2016 15:42:25 -0500 (EST)


----- Original Message -----
> From: "Eric Blake" <address@hidden>
> To: address@hidden
> Cc: address@hidden, address@hidden
> Sent: Wednesday, November 30, 2016 8:50:05 PM
> Subject: Re: [Qemu-devel] [PATCH 00/36] Rip out dynamic JSON parsing
> 
> On 11/30/2016 01:44 PM, Eric Blake wrote:
> > Followup series to conversation about PRId64 and MacOS:
> > https://lists.gnu.org/archive/html/qemu-devel/2016-11/msg04226.html
> > 
> 
> And I meant to add:
> 
> Available as a tag at:
> git fetch git://repo.or.cz/qemu/ericb.git qapi-dynamic-json-v1
> 
> on top of Markus' qapi-next branch (but presumably will apply without
> much effort on top of the final 2.8 release, since this is 2.9 material)

I don't see the point, honestly.  It's harder to use and hardly
removes any code (the balance is -160 lines, but about 140 of them
come from the parser's test suite).

Using the functionality only in qtest is fine, but it is damn useful
there.  If we want to remove the % escaping, let's rewrite the qtests
in Python as they probably should have been since the beginning.

Paolo



reply via email to

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