[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 1/6] qapi: Tweak "command returns a nice type" check for clarity
From: |
Markus Armbruster |
Subject: |
[PATCH 1/6] qapi: Tweak "command returns a nice type" check for clarity |
Date: |
Wed, 20 Nov 2019 19:25:46 +0100 |
Signed-off-by: Markus Armbruster <address@hidden>
---
scripts/qapi/schema.py | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/scripts/qapi/schema.py b/scripts/qapi/schema.py
index cf0045f34e..cfb574c85d 100644
--- a/scripts/qapi/schema.py
+++ b/scripts/qapi/schema.py
@@ -711,10 +711,11 @@ class QAPISchemaCommand(QAPISchemaEntity):
self.ret_type = schema.resolve_type(
self._ret_type_name, self.info, "command's 'returns'")
if self.name not in self.info.pragma.returns_whitelist:
- if not (isinstance(self.ret_type, QAPISchemaObjectType)
- or (isinstance(self.ret_type, QAPISchemaArrayType)
- and isinstance(self.ret_type.element_type,
- QAPISchemaObjectType))):
+ typ = self.ret_type
+ if isinstance(typ, QAPISchemaArrayType):
+ typ = self.ret_type.element_type
+ assert typ
+ if not isinstance(typ, QAPISchemaObjectType):
raise QAPISemError(
self.info,
"command's 'returns' cannot take %s"
--
2.21.0
- Re: [PATCH 4/6] qapi: Proper intermediate representation for modules, (continued)
- [PATCH 3/6] qapi: Generate command registration stuff into separate files, Markus Armbruster, 2019/11/20
- [PATCH 5/6] qapi: Fix code generation for empty modules, Markus Armbruster, 2019/11/20
- [PATCH 2/6] tests/Makefile.include: Fix missing test-qapi-emit-events.[ch], Markus Armbruster, 2019/11/20
- [PATCH 6/6] qapi: Simplify QAPISchemaModularCVisitor, Markus Armbruster, 2019/11/20
- [PATCH 1/6] qapi: Tweak "command returns a nice type" check for clarity,
Markus Armbruster <=
- Re: [PATCH 0/6] qapi: Module fixes and cleanups, Markus Armbruster, 2019/11/20
- Re: [PATCH 0/6] qapi: Module fixes and cleanups, no-reply, 2019/11/20
- Re: [PATCH 0/6] qapi: Module fixes and cleanups, Markus Armbruster, 2019/11/21
- [PATCH 7/6] Makefile: Make Makefile depend on generated qga files, too, Markus Armbruster, 2019/11/29