[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 05/11] qapi: Add visitor for implicit struct
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [RFC PATCH 05/11] qapi: Add visitor for implicit structs |
Date: |
Thu, 11 Jul 2013 06:41:01 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7 |
On 07/09/2013 03:53 AM, Kevin Wolf wrote:
> These can be used when an embedded struct is parsed and members not
> belonging to the struct may be present in the input (parsing flat
> namespect QMP union with discriminator)
namespect? Not sure if you meant 'namespaced'?
Again, a comment demonstrating in sample QMP on what this is attempting
would be helpful in the commit message. Without looking at later
patches yet, I'm guessing from your description that your intent is to
reach a point where:
{ 'type': 'Extra1',
'data': { 'foo': 'str' } }
{ 'type': 'Extra2',
'data': { 'bar': 'str' } }
{ 'union': 'Type',
'data': { 'variant1': 'Extra1',
'variant2': 'Extra2' } }
{ 'command': 'test',
'data': { 'arg': 'Type' } }
which currently requires this over the wire:
{ "execute": "test",
"arguments": { "arg": { "type": "variant1",
"data": { "foo": "hello" } } } }
can be shortened to:
{ "execute": "test",
"arguments": { "arg": { "type": "variant1", "foo": "hello" } } }
by allowing the struct passed as "arg" to inline the contents of the
union rather than require another nesting level through "data". This
patch sets up the entry points so that a future patch can parse the
struct of arg as an implicit struct, and not choke over the key "foo",
instead of the current visitor of a union type that expects exactly the
two keys "type" and "data".
>
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
> include/qapi/visitor-impl.h | 4 ++++
> include/qapi/visitor.h | 3 +++
> qapi/qapi-visit-core.c | 16 ++++++++++++++++
> qapi/qmp-input-visitor.c | 14 ++++++++++++++
> 4 files changed, 37 insertions(+)
Although the commit message left me floundering, the code itself seems
reasonable - you are just adding two new entry points to allow a new
type of parse. It remains to later patches to see how the new parsers
are useful, but that doesn't stop me from giving:
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [RFC PATCH 01/11] qapi-types.py: Split off generate_struct_fields(), (continued)
- [Qemu-devel] [RFC PATCH 01/11] qapi-types.py: Split off generate_struct_fields(), Kevin Wolf, 2013/07/09
- [Qemu-devel] [RFC PATCH 03/11] qapi-visit.py: Split off generate_visit_struct_fields(), Kevin Wolf, 2013/07/09
- [Qemu-devel] [RFC PATCH 04/11] qapi-visit.py: Implement 'base' for unions, Kevin Wolf, 2013/07/09
- [Qemu-devel] [RFC PATCH 02/11] qapi-types.py: Implement 'base' for unions, Kevin Wolf, 2013/07/09
- [Qemu-devel] [RFC PATCH 05/11] qapi: Add visitor for implicit structs, Kevin Wolf, 2013/07/09
- Re: [Qemu-devel] [RFC PATCH 05/11] qapi: Add visitor for implicit structs,
Eric Blake <=
- [Qemu-devel] [RFC PATCH 06/11] qapi: Flat unions with arbitrary discriminator, Kevin Wolf, 2013/07/09
- [Qemu-devel] [RFC PATCH 09/11] Implement qdict_flatten(), Kevin Wolf, 2013/07/09
- [Qemu-devel] [RFC PATCH 07/11] qapi: Add consume argument to qmp_input_get_object(), Kevin Wolf, 2013/07/09
- [Qemu-devel] [RFC PATCH 08/11] qapi: Anonymous unions, Kevin Wolf, 2013/07/09