commit c2c7065e1752a3be1b437d1ea359cde35d28ee3b
author Markus Armbruster <armbru@redhat.com> Sat Sep 14 17:34:58 2019 +0200
committer Markus Armbruster <armbru@redhat.com> Tue Sep 24 14:07:23 2019 +0200
tree 309fd28ba39f32c26ebaaf9ca0b06852a5c67f2d
parent 887a2069f76fa99b9755467126dd171a9bad34a3

qapi: Reject blank 'if' conditions in addition to empty ones

"'if': 'COND'" generates "#if COND".  We reject empty COND because it
won't compile.  Blank COND won't compile any better, so reject that,
too.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20190914153506.2151-12-armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>

tests/qapi-schema/bad-if-list.json

diff --git a/tests/qapi-schema/bad-if-list.json b/tests/qapi-schema/bad-if-list.json
index 49ced9b..ea3d95b 100644
--- a/tests/qapi-schema/bad-if-list.json
+++ b/tests/qapi-schema/bad-if-list.json
@@ -1,3 +1,3 @@
 # check invalid 'if' content
 { 'struct': 'TestIfStruct', 'data': { 'foo': 'int' },
-  'if': ['foo', ''] }
+  'if': ['foo', ' '] }