[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/22] qapi: qapi_visit.py, make code useable as
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH 04/22] qapi: qapi_visit.py, make code useable as module |
Date: |
Tue, 24 Jul 2012 17:13:27 -0500 |
User-agent: |
Notmuch/0.13.2+60~g7ecf77d (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
Michael Roth <address@hidden> writes:
Reviewed-by: Anthony Liguori <address@hidden>
Regards,
Anthony Liguori
> Signed-off-by: Michael Roth <address@hidden>
> ---
> scripts/qapi_visit.py | 143
> +++++++++++++++++++++++++------------------------
> 1 file changed, 74 insertions(+), 69 deletions(-)
>
> diff --git a/scripts/qapi_visit.py b/scripts/qapi_visit.py
> index 04ef7c4..25707f5 100644
> --- a/scripts/qapi_visit.py
> +++ b/scripts/qapi_visit.py
> @@ -224,55 +224,57 @@ void visit_type_%(name)s(Visitor *m, %(name)s * obj,
> const char *name, Error **e
> ''',
> name=name)
>
> -try:
> - opts, args = getopt.gnu_getopt(sys.argv[1:], "chp:o:",
> - ["source", "header", "prefix=",
> "output-dir="])
> -except getopt.GetoptError, err:
> - print str(err)
> - sys.exit(1)
> -
> -output_dir = ""
> -prefix = ""
> -c_file = 'qapi-visit.c'
> -h_file = 'qapi-visit.h'
> -
> -do_c = False
> -do_h = False
> -
> -for o, a in opts:
> - if o in ("-p", "--prefix"):
> - prefix = a
> - elif o in ("-o", "--output-dir"):
> - output_dir = a + "/"
> - elif o in ("-c", "--source"):
> +def main(argv=[]):
> + try:
> + opts, args = getopt.gnu_getopt(sys.argv[1:], "chp:o:",
> + ["source", "header", "prefix=",
> + "output-dir="])
> + except getopt.GetoptError, err:
> + print str(err)
> + sys.exit(1)
> +
> + output_dir = ""
> + prefix = ""
> + c_file = 'qapi-visit.c'
> + h_file = 'qapi-visit.h'
> +
> + do_c = False
> + do_h = False
> +
> + for o, a in opts:
> + if o in ("-p", "--prefix"):
> + prefix = a
> + elif o in ("-o", "--output-dir"):
> + output_dir = a + "/"
> + elif o in ("-c", "--source"):
> + do_c = True
> + elif o in ("-h", "--header"):
> + do_h = True
> +
> + if not do_c and not do_h:
> do_c = True
> - elif o in ("-h", "--header"):
> do_h = True
>
> -if not do_c and not do_h:
> - do_c = True
> - do_h = True
> + c_file = output_dir + prefix + c_file
> + h_file = output_dir + prefix + h_file
>
> -c_file = output_dir + prefix + c_file
> -h_file = output_dir + prefix + h_file
> + try:
> + os.makedirs(output_dir)
> + except os.error, e:
> + if e.errno != errno.EEXIST:
> + raise
>
> -try:
> - os.makedirs(output_dir)
> -except os.error, e:
> - if e.errno != errno.EEXIST:
> - raise
> -
> -def maybe_open(really, name, opt):
> - if really:
> - return open(name, opt)
> - else:
> - import StringIO
> - return StringIO.StringIO()
> + def maybe_open(really, name, opt):
> + if really:
> + return open(name, opt)
> + else:
> + import StringIO
> + return StringIO.StringIO()
>
> -fdef = maybe_open(do_c, c_file, 'w')
> -fdecl = maybe_open(do_h, h_file, 'w')
> + fdef = maybe_open(do_c, c_file, 'w')
> + fdecl = maybe_open(do_h, h_file, 'w')
>
> -fdef.write(mcgen('''
> + fdef.write(mcgen('''
> /* THIS FILE IS AUTOMATICALLY GENERATED, DO NOT MODIFY */
>
> /*
> @@ -292,7 +294,7 @@ fdef.write(mcgen('''
> ''',
> header=basename(h_file)))
>
> -fdecl.write(mcgen('''
> + fdecl.write(mcgen('''
> /* THIS FILE IS AUTOMATICALLY GENERATED, DO NOT MODIFY */
>
> /*
> @@ -316,37 +318,40 @@ fdecl.write(mcgen('''
> ''',
> prefix=prefix, guard=guardname(h_file)))
>
> -exprs = parse_schema(sys.stdin)
> + exprs = parse_schema(sys.stdin)
>
> -for expr in exprs:
> - if expr.has_key('type'):
> - ret = generate_visit_struct(expr['type'], expr['data'])
> - ret += generate_visit_list(expr['type'], expr['data'])
> - fdef.write(ret)
> + for expr in exprs:
> + if expr.has_key('type'):
> + ret = generate_visit_struct(expr['type'], expr['data'])
> + ret += generate_visit_list(expr['type'], expr['data'])
> + fdef.write(ret)
>
> - ret = generate_declaration(expr['type'], expr['data'])
> - fdecl.write(ret)
> - elif expr.has_key('union'):
> - ret = generate_visit_union(expr['union'], expr['data'])
> - ret += generate_visit_list(expr['union'], expr['data'])
> - fdef.write(ret)
> + ret = generate_declaration(expr['type'], expr['data'])
> + fdecl.write(ret)
> + elif expr.has_key('union'):
> + ret = generate_visit_union(expr['union'], expr['data'])
> + ret += generate_visit_list(expr['union'], expr['data'])
> + fdef.write(ret)
>
> - ret = generate_decl_enum('%sKind' % expr['union'],
> expr['data'].keys())
> - ret += generate_declaration(expr['union'], expr['data'])
> - fdecl.write(ret)
> - elif expr.has_key('enum'):
> - ret = generate_visit_enum(expr['enum'], expr['data'])
> - fdef.write(ret)
> + ret = generate_decl_enum('%sKind' % expr['union'],
> expr['data'].keys())
> + ret += generate_declaration(expr['union'], expr['data'])
> + fdecl.write(ret)
> + elif expr.has_key('enum'):
> + ret = generate_visit_enum(expr['enum'], expr['data'])
> + fdef.write(ret)
>
> - ret = generate_decl_enum(expr['enum'], expr['data'])
> - fdecl.write(ret)
> + ret = generate_decl_enum(expr['enum'], expr['data'])
> + fdecl.write(ret)
>
> -fdecl.write('''
> + fdecl.write('''
> #endif
> -''')
> + ''')
> +
> + fdecl.flush()
> + fdecl.close()
>
> -fdecl.flush()
> -fdecl.close()
> + fdef.flush()
> + fdef.close()
>
> -fdef.flush()
> -fdef.close()
> +if __name__ == '__main__':
> + sys.exit(main(sys.argv))
> --
> 1.7.9.5
- [Qemu-devel] [RFC v2] Use QEMU IDL for device serialization/introspection, Michael Roth, 2012/07/24
- [Qemu-devel] [PATCH 02/22] qapi: qapi-types.py -> qapi_types.py, Michael Roth, 2012/07/24
- [Qemu-devel] [PATCH 01/22] qapi: qapi-visit.py -> qapi_visit.py so we can import, Michael Roth, 2012/07/24
- [Qemu-devel] [PATCH 04/22] qapi: qapi_visit.py, make code useable as module, Michael Roth, 2012/07/24
- Re: [Qemu-devel] [PATCH 04/22] qapi: qapi_visit.py, make code useable as module,
Anthony Liguori <=
- [Qemu-devel] [PATCH 05/22] qapi: qapi_visit.py, support arrays and complex qapi definitions, Michael Roth, 2012/07/24
- [Qemu-devel] [PATCH 03/22] qapi: qapi-commands.py -> qapi_commands.py, Michael Roth, 2012/07/24
- [Qemu-devel] [PATCH 07/22] qapi: qapi_visit.py, support generating static functions, Michael Roth, 2012/07/24
- [Qemu-devel] [PATCH 06/22] qapi: add visitor interfaces for arrays, Michael Roth, 2012/07/24
- [Qemu-devel] [PATCH 08/22] qapi: qapi_visit.py, support for visiting non-pointer/embedded structs, Michael Roth, 2012/07/24
- [Qemu-devel] [PATCH 09/22] qapi: QmpOutputVisitor, implement array handling, Michael Roth, 2012/07/24