[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 03/25] qapi: More robust conditions for when labels a
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] [PULL 03/25] qapi: More robust conditions for when labels are needed |
Date: |
Fri, 30 Oct 2015 16:42:32 +0100 |
From: Eric Blake <address@hidden>
We were using regular expressions to see if ret included
any earlier text that emitted a 'goto out;' line, to decide
whether we needed to output an 'out:' label. But this is
fragile, if the ret text can possibly combine more than one
generated function body, where the first function used a
goto but the second does not. Change the code to just check
for the known conditions which cause an error check to be
needed. Besides, it's slightly more efficient to use plain
checks than regular expression searching.
No change to generated code.
Signed-off-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>
---
scripts/qapi-commands.py | 4 +++-
scripts/qapi-visit.py | 3 ++-
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/scripts/qapi-commands.py b/scripts/qapi-commands.py
index 43a893b..561e47a 100644
--- a/scripts/qapi-commands.py
+++ b/scripts/qapi-commands.py
@@ -175,7 +175,9 @@ def gen_marshal(name, arg_type, ret_type):
ret += gen_marshal_input_visit(arg_type)
ret += gen_call(name, arg_type, ret_type)
- if re.search('^ *goto out;', ret, re.MULTILINE):
+ # 'goto out' produced by gen_marshal_input_visit->gen_visit_fields()
+ # for each arg_type member, and by gen_call() for ret_type
+ if (arg_type and arg_type.members) or ret_type:
ret += mcgen('''
out:
diff --git a/scripts/qapi-visit.py b/scripts/qapi-visit.py
index d0759d7..2dc3aed 100644
--- a/scripts/qapi-visit.py
+++ b/scripts/qapi-visit.py
@@ -87,7 +87,8 @@ static void visit_type_%(c_name)s_fields(Visitor *v,
%(c_name)s **obj, Error **e
ret += gen_visit_fields(members, prefix='(*obj)->')
- if re.search('^ *goto out;', ret, re.MULTILINE):
+ # 'goto out' produced for base, and by gen_visit_fields() for each member
+ if base or members:
ret += mcgen('''
out:
--
2.4.3
- [Qemu-devel] [PULL 00/25] QAPI patches, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 02/25] qapi: More idiomatic string operations, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 07/25] qapi-visit: Split off visit_type_FOO_fields forward decl, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 08/25] qapi-types: Refactor base fields output, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 04/25] qapi: Reserve '*List' type names for list types, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 03/25] qapi: More robust conditions for when labels are needed,
Markus Armbruster <=
- [Qemu-devel] [PULL 16/25] sockets: Convert to new qapi union layout, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 13/25] qapi-visit: Convert to new qapi union layout, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 12/25] qapi: Start converting to new qapi union layout, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 21/25] tpm: Convert to new qapi union layout, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 25/25] qapi-schema: mark InetSocketAddress as mandatory again, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 24/25] qapi: Simplify gen_struct_field(), Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 05/25] qapi: Reserve 'q_*' and 'has_*' member names, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 10/25] qapi: Unbox base members, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 06/25] vnc: Hoist allocation of VncBasicInfo to callers, Markus Armbruster, 2015/10/30
- [Qemu-devel] [PULL 09/25] qapi: Prefer typesafe upcasts to qapi base classes, Markus Armbruster, 2015/10/30