Re: [Qemu-devel] [PATCH v6 00/36] drop qapi nested structs

2015-04-29 Thread Luiz Capitulino
On Wed, 29 Apr 2015 06:34:12 -0600 Eric Blake wrote: > On 04/29/2015 12:48 AM, Markus Armbruster wrote: > > General question first: through which tree should this go? > > > > MAINTAINERS doesn't cover the QAPI generators. Closest related stanza > > is QAPI (Luiz, Michael R.). Should it cover t

Re: [Qemu-devel] [PATCH v6 00/36] drop qapi nested structs

2015-04-29 Thread Eric Blake
On 04/29/2015 12:48 AM, Markus Armbruster wrote: > General question first: through which tree should this go? > > MAINTAINERS doesn't cover the QAPI generators. Closest related stanza > is QAPI (Luiz, Michael R.). Should it cover the generators? Next > closest is QAPI Schema (Luiz, you, myself)

Re: [Qemu-devel] [PATCH v6 00/36] drop qapi nested structs

2015-04-28 Thread Markus Armbruster
General question first: through which tree should this go? MAINTAINERS doesn't cover the QAPI generators. Closest related stanza is QAPI (Luiz, Michael R.). Should it cover the generators? Next closest is QAPI Schema (Luiz, you, myself). For completeness: because we touch qmp_query_pci_bridge(

Re: [Qemu-devel] [PATCH v6 00/36] drop qapi nested structs

2015-04-28 Thread Eric Blake
On 04/28/2015 08:02 AM, Markus Armbruster wrote: >> v6 changes are noted in each patch; in particular, several new >> patches were added (additional tests, split some patches, conversion >> to 'struct' instead of 'type'). But most of the changes were in >> direct response to review comments or reb

Re: [Qemu-devel] [PATCH v6 00/36] drop qapi nested structs

2015-04-28 Thread Markus Armbruster
Eric Blake writes: > We want to eventually allow qapi defaults, by making: > 'data':{'*flag':'bool'} > as shorthand for something like: > 'data':{'flag':{'type':'bool', 'optional':true}} > so that the default can be specified: > 'data':{'flag':{'type':'bool', 'optional':true, 'default':true}}

Re: [Qemu-devel] [PATCH v6 00/36] drop qapi nested structs

2015-04-08 Thread Alberto Garcia
On Mon, Apr 06, 2015 at 07:13:28AM -0600, Eric Blake wrote: > My apologies - I mis-typed Berto's email address, so if you > reply-to-all, you'll get a bounce message if you don't manually fix > it. Is it worth me resending the series with an updated CC list? For me it's not necessary, but thanks

Re: [Qemu-devel] [PATCH v6 00/36] drop qapi nested structs

2015-04-06 Thread Eric Blake
On 04/04/2015 10:07 PM, Eric Blake wrote: > We want to eventually allow qapi defaults, by making: > 'data':{'*flag':'bool'} > as shorthand for something like: > 'data':{'flag':{'type':'bool', 'optional':true}} > so that the default can be specified: > 'data':{'flag':{'type':'bool', 'optional':tr

[Qemu-devel] [PATCH v6 00/36] drop qapi nested structs

2015-04-04 Thread Eric Blake
We want to eventually allow qapi defaults, by making: 'data':{'*flag':'bool'} as shorthand for something like: 'data':{'flag':{'type':'bool', 'optional':true}} so that the default can be specified: 'data':{'flag':{'type':'bool', 'optional':true, 'default':true}} This series does not quite get u