mirror of https://github.com/xemu-project/xemu.git
qapi: qapi-visit.py, native list support
Teach visitor generators about native types so they can generate the appropriate visitor routines. Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com> Reviewed-by: Laszlo Ersek <lersek@redhat.com> Reviewed-by: Amos Kong <akong@redhat.com> Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
This commit is contained in:
parent
c664aef551
commit
7c946bc418
|
@ -202,12 +202,14 @@ void visit_type_%(name)s(Visitor *m, %(name)s ** obj, const char *name, Error **
|
||||||
|
|
||||||
return ret
|
return ret
|
||||||
|
|
||||||
def generate_declaration(name, members, genlist=True):
|
def generate_declaration(name, members, genlist=True, builtin_type=False):
|
||||||
ret = mcgen('''
|
ret = ""
|
||||||
|
if not builtin_type:
|
||||||
|
ret += mcgen('''
|
||||||
|
|
||||||
void visit_type_%(name)s(Visitor *m, %(name)s ** obj, const char *name, Error **errp);
|
void visit_type_%(name)s(Visitor *m, %(name)s ** obj, const char *name, Error **errp);
|
||||||
''',
|
''',
|
||||||
name=name)
|
name=name)
|
||||||
|
|
||||||
if genlist:
|
if genlist:
|
||||||
ret += mcgen('''
|
ret += mcgen('''
|
||||||
|
@ -235,8 +237,9 @@ void visit_type_%(name)s(Visitor *m, %(name)s * obj, const char *name, Error **e
|
||||||
name=name)
|
name=name)
|
||||||
|
|
||||||
try:
|
try:
|
||||||
opts, args = getopt.gnu_getopt(sys.argv[1:], "chp:o:",
|
opts, args = getopt.gnu_getopt(sys.argv[1:], "chbp:o:",
|
||||||
["source", "header", "prefix=", "output-dir="])
|
["source", "header", "builtins", "prefix=",
|
||||||
|
"output-dir="])
|
||||||
except getopt.GetoptError, err:
|
except getopt.GetoptError, err:
|
||||||
print str(err)
|
print str(err)
|
||||||
sys.exit(1)
|
sys.exit(1)
|
||||||
|
@ -248,6 +251,7 @@ h_file = 'qapi-visit.h'
|
||||||
|
|
||||||
do_c = False
|
do_c = False
|
||||||
do_h = False
|
do_h = False
|
||||||
|
do_builtins = False
|
||||||
|
|
||||||
for o, a in opts:
|
for o, a in opts:
|
||||||
if o in ("-p", "--prefix"):
|
if o in ("-p", "--prefix"):
|
||||||
|
@ -258,6 +262,8 @@ for o, a in opts:
|
||||||
do_c = True
|
do_c = True
|
||||||
elif o in ("-h", "--header"):
|
elif o in ("-h", "--header"):
|
||||||
do_h = True
|
do_h = True
|
||||||
|
elif o in ("-b", "--builtins"):
|
||||||
|
do_builtins = True
|
||||||
|
|
||||||
if not do_c and not do_h:
|
if not do_c and not do_h:
|
||||||
do_c = True
|
do_c = True
|
||||||
|
@ -324,11 +330,29 @@ fdecl.write(mcgen('''
|
||||||
|
|
||||||
#include "qapi/visitor.h"
|
#include "qapi/visitor.h"
|
||||||
#include "%(prefix)sqapi-types.h"
|
#include "%(prefix)sqapi-types.h"
|
||||||
|
|
||||||
''',
|
''',
|
||||||
prefix=prefix, guard=guardname(h_file)))
|
prefix=prefix, guard=guardname(h_file)))
|
||||||
|
|
||||||
exprs = parse_schema(sys.stdin)
|
exprs = parse_schema(sys.stdin)
|
||||||
|
|
||||||
|
# to avoid header dependency hell, we always generate declarations
|
||||||
|
# for built-in types in our header files and simply guard them
|
||||||
|
fdecl.write(guardstart("QAPI_VISIT_BUILTIN_VISITOR_DECL"))
|
||||||
|
for typename in builtin_types:
|
||||||
|
fdecl.write(generate_declaration(typename, None, genlist=True,
|
||||||
|
builtin_type=True))
|
||||||
|
fdecl.write(guardend("QAPI_VISIT_BUILTIN_VISITOR_DECL"))
|
||||||
|
|
||||||
|
# ...this doesn't work for cases where we link in multiple objects that
|
||||||
|
# have the functions defined, so we use -b option to provide control
|
||||||
|
# over these cases
|
||||||
|
if do_builtins:
|
||||||
|
fdef.write(guardstart("QAPI_VISIT_BUILTIN_VISITOR_DEF"))
|
||||||
|
for typename in builtin_types:
|
||||||
|
fdef.write(generate_visit_list(typename, None))
|
||||||
|
fdef.write(guardend("QAPI_VISIT_BUILTIN_VISITOR_DEF"))
|
||||||
|
|
||||||
for expr in exprs:
|
for expr in exprs:
|
||||||
if expr.has_key('type'):
|
if expr.has_key('type'):
|
||||||
ret = generate_visit_struct(expr['type'], expr['data'])
|
ret = generate_visit_struct(expr['type'], expr['data'])
|
||||||
|
|
Loading…
Reference in New Issue