qapi: Drop useless gen_err_check()

Ever since commit 12f254f removed the last parameterization
of gen_err_check(), it no longer makes sense to hide the three
lines of generated C code behind a macro call. Just inline it
into the remaining users.

No change to generated code.

Backports commit fa274ed6fb788866ed3a2cfd54a2ddf78f04f2c0 from qemu
This commit is contained in:
Eric Blake 2018-02-25 20:10:43 -05:00 committed by Lioncash
parent d7014c66df
commit b5220a6867
No known key found for this signature in database
GPG Key ID: 4E3C3CC1031BA9C7
2 changed files with 6 additions and 10 deletions

View File

@ -46,9 +46,11 @@ void visit_type_%(c_name)s_members(Visitor *v, %(c_name)s *obj, Error **errp)
if base:
ret += mcgen('''
visit_type_%(c_type)s_members(v, (%(c_type)s *)obj, &err);
if (err) {
goto out;
}
''',
c_type=base.c_name())
ret += gen_err_check()
for memb in members:
if memb.optional:
@ -59,10 +61,12 @@ void visit_type_%(c_name)s_members(Visitor *v, %(c_name)s *obj, Error **errp)
push_indent()
ret += mcgen('''
visit_type_%(c_type)s(v, "%(name)s", &obj->%(c_name)s, &err);
if (err) {
goto out;
}
''',
c_type=memb.type.c_name(), name=memb.name,
c_name=c_name(memb.name))
ret += gen_err_check()
if memb.optional:
pop_indent()
ret += mcgen('''

View File

@ -1669,14 +1669,6 @@ def gen_params(arg_type, extra):
return ret
def gen_err_check(label='out', skiperr=False):
return mcgen('''
if (err) {
goto out;
}
''')
#
# Common command line parsing
#