On 08/04/2015 09:57 AM, Markus Armbruster wrote: > Fixes flat unions to get the base's base members. Test case is from > commit 2fc0043, in qapi-schema-test.json: >
> > Flat union visitors remain broken. They'll be fixed next. > > Signed-off-by: Markus Armbruster <arm...@redhat.com> > --- > docs/qapi-code-gen.txt | 51 +++--- Thanks for updating the docs. The diffstat of the actual generated code is large, but the extra commits prove that in the end it's the same material, just in different ordering. Meanwhile, the doc example is correct, and being much shorter, makes it easier to see the sort of reordering that happened. > scripts/qapi-types.py | 273 > ++++++++++++++------------------ > tests/qapi-schema/qapi-schema-test.json | 4 +- > 3 files changed, 144 insertions(+), 184 deletions(-) > > + > +# If you link code generated from multiple schemata, you want only one > +# instance include the code generated for built-in types. Generate it s/include/of/ > +# only when do_builtins, enabled by command line option -b. See also > +# QAPISchemaGenTypeVisitor.visit_end(). > do_builtins = False With the grammar fix, Reviewed-by: Eric Blake <ebl...@redhat.com> -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
