On 03/13/2017 01:18 AM, Markus Armbruster wrote: > This fixes the errors uncovered by the previous commit. > > Signed-off-by: Markus Armbruster <arm...@redhat.com> > ---
>
> expr = self.get_expr(False)
> if 'include' in expr:
> + self.reject_expr_doc()
> if len(expr) != 1:
Do you also need to handle 'pragma' expressions added earlier in the series?
/me goes and experiments:
diff --git i/tests/qapi-schema/doc-bad-expr.json
w/tests/qapi-schema/doc-bad-expr.json
index 0caa0ae..6c1204d 100644
--- i/tests/qapi-schema/doc-bad-expr.json
+++ w/tests/qapi-schema/doc-bad-expr.json
@@ -4,4 +4,5 @@
# @foo:
##
{ 'include': 'empty.json' }
+{ 'pragma': {} }
{ 'struct': 'foo', 'data': {} }
Oops - back to accepting the program.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
