[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 5/6] qapi: Minor introspect.py cleanups
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] [PULL 5/6] qapi: Minor introspect.py cleanups |
Date: |
Tue, 28 Aug 2018 21:10:47 +0200 |
From: Eric Blake <address@hidden>
Commit 7d0f982b changed generated introspection output to no longer
produce long lines in the generated .c file, but failed to adjust
comments to match. Add some clarity that the shorter length that
matters most is the overall QMP response on the wire.
Commit 25b1ef31 triggers a pep8 formatting nit.
Signed-off-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>
---
scripts/qapi/introspect.py | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/scripts/qapi/introspect.py b/scripts/qapi/introspect.py
index 189a4edaba..43e81a0693 100644
--- a/scripts/qapi/introspect.py
+++ b/scripts/qapi/introspect.py
@@ -89,7 +89,6 @@ class
QAPISchemaGenIntrospectVisitor(QAPISchemaMonolithicCVisitor):
for typ in self._used_types:
typ.visit(self)
# generate C
- # TODO can generate awfully long lines
name = c_name(self._prefix, protect=False) + 'qmp_schema_qlit'
self._genh.add(mcgen('''
#include "qapi/qmp/qlit.h"
@@ -129,8 +128,8 @@ const QLitObject %(c_name)s = %(c_string)s;
if typ not in self._used_types:
self._used_types.append(typ)
# Clients should examine commands and events, not types. Hide
- # type names to reduce the temptation. Also saves a few
- # characters.
+ # type names as integers to reduce the temptation. Also, it
+ # saves a few characters on the wire.
if isinstance(typ, QAPISchemaBuiltinType):
return typ.name
if isinstance(typ, QAPISchemaArrayType):
@@ -185,7 +184,7 @@ const QLitObject %(c_name)s = %(c_string)s;
arg_type = arg_type or self._schema.the_empty_object_type
ret_type = ret_type or self._schema.the_empty_object_type
obj = {'arg-type': self._use_type(arg_type),
- 'ret-type': self._use_type(ret_type) }
+ 'ret-type': self._use_type(ret_type)}
if allow_oob:
obj['allow-oob'] = allow_oob
self._gen_qlit(name, 'command', obj, ifcond)
--
2.17.1
- [Qemu-devel] [PULL 0/6] QAPI patches for 2018-08-28, Markus Armbruster, 2018/08/28
- [Qemu-devel] [PULL 6/6] qapi: Add comments to aid debugging generated introspection, Markus Armbruster, 2018/08/28
- [Qemu-devel] [PULL 5/6] qapi: Minor introspect.py cleanups,
Markus Armbruster <=
- [Qemu-devel] [PULL 4/6] qapi: Update docs for generator changes since commit 9ee86b85267, Markus Armbruster, 2018/08/28
- [Qemu-devel] [PULL 2/6] qapi: Drop qapi_event_send_FOO()'s Error ** argument, Markus Armbruster, 2018/08/28
- [Qemu-devel] [PULL 1/6] qapi: Fix build_params() for empty parameter list, Markus Armbruster, 2018/08/28
- [Qemu-devel] [PULL 3/6] qapi: Emit a blank line before dummy declaration, Markus Armbruster, 2018/08/28