Re: [Qemu-devel] [PATCH 08/11] qjson: Simplify by using json-output-visitor

2015-12-18 Thread Eric Blake
On 12/11/2015 04:10 AM, Paolo Bonzini wrote: > > > On 11/12/2015 00:53, Eric Blake wrote: >> Instead of rolling our own limited JSON outputter, we can just >> wrap the more full-featured JSON output Visitor. >> >> This slightly changes the output (different spacing), but the >> result is still

Re: [Qemu-devel] [PATCH 08/11] qjson: Simplify by using json-output-visitor

2015-12-11 Thread Paolo Bonzini
On 11/12/2015 00:53, Eric Blake wrote: > Instead of rolling our own limited JSON outputter, we can just > wrap the more full-featured JSON output Visitor. > > This slightly changes the output (different spacing), but the > result is still equivalent JSON contents. > > Signed-off-by: Eric Blake

Re: [Qemu-devel] [PATCH 08/11] qjson: Simplify by using json-output-visitor

2015-12-11 Thread Paolo Bonzini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 11/12/2015 14:42, Eric Blake wrote: >>> compiles and is pretty much a 1:1 translation from the qjson.c >>> API to the visitor API (using this patch as a guide). Feel >>> free to include it and remove qjson.c. Alternatively, you can >>> leave

Re: [Qemu-devel] [PATCH 08/11] qjson: Simplify by using json-output-visitor

2015-12-11 Thread Eric Blake
On 12/11/2015 04:10 AM, Paolo Bonzini wrote: > > > On 11/12/2015 00:53, Eric Blake wrote: >> Instead of rolling our own limited JSON outputter, we can just >> wrap the more full-featured JSON output Visitor. >> >> This slightly changes the output (different spacing), but the >> result is still

[Qemu-devel] [PATCH 08/11] qjson: Simplify by using json-output-visitor

2015-12-10 Thread Eric Blake
Instead of rolling our own limited JSON outputter, we can just wrap the more full-featured JSON output Visitor. This slightly changes the output (different spacing), but the result is still equivalent JSON contents. Signed-off-by: Eric Blake --- qjson.c | 61