]> xenbits.xensource.com Git - qemu-xen.git/commitdiff
qapi: Tweak "command returns a nice type" check for clarity
authorMarkus Armbruster <armbru@redhat.com>
Wed, 20 Nov 2019 18:25:46 +0000 (19:25 +0100)
committerMarkus Armbruster <armbru@redhat.com>
Tue, 14 Jan 2020 10:01:58 +0000 (11:01 +0100)
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20191120182551.23795-2-armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
scripts/qapi/schema.py

index cf0045f34e5ed28f0ebec013ad9c88b5430a84d4..cfb574c85d2b166f4a1a927bb7fe0744d5e6e68f 100644 (file)
@@ -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"