[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] qapi: Drop dead genlist parameter
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH v2] qapi: Drop dead genlist parameter |
Date: |
Tue, 21 Apr 2015 10:09:02 -0400 |
On Fri, 10 Apr 2015 15:07:59 -0600
Eric Blake <address@hidden> wrote:
> Defaulting a parameter to True, then having all callers omit or
> pass an explicit True for that parameter, is pointless. Looks
> like it has been dead since introduction in commit 06d64c6, more
> than 4 years ago.
>
> Signed-off-by: Eric Blake <address@hidden>
Applied to the qmp branch, thanks.
> ---
>
> v2: tweak commit message, don't depend on pending series
>
> scripts/qapi-visit.py | 38 +++++++++++++++++---------------------
> 1 file changed, 17 insertions(+), 21 deletions(-)
>
> diff --git a/scripts/qapi-visit.py b/scripts/qapi-visit.py
> index 8f845a2..1be4d67 100644
> --- a/scripts/qapi-visit.py
> +++ b/scripts/qapi-visit.py
> @@ -2,7 +2,7 @@
> # QAPI visitor generator
> #
> # Copyright IBM, Corp. 2011
> -# Copyright (C) 2014 Red Hat, Inc.
> +# Copyright (C) 2014-2015 Red Hat, Inc.
> #
> # Authors:
> # Anthony Liguori <address@hidden>
> @@ -401,34 +401,31 @@ out:
>
> return ret
>
> -def generate_declaration(name, members, genlist=True, builtin_type=False):
> +def generate_declaration(name, members, builtin_type=False):
> ret = ""
> if not builtin_type:
> ret += mcgen('''
>
> void visit_type_%(name)s(Visitor *m, %(name)s **obj, const char *name, Error
> **errp);
> ''',
> - name=name)
> -
> - if genlist:
> - ret += mcgen('''
> -void visit_type_%(name)sList(Visitor *m, %(name)sList **obj, const char
> *name, Error **errp);
> -''',
> - name=name)
> -
> - return ret
> -
> -def generate_enum_declaration(name, members, genlist=True):
> - ret = ""
> - if genlist:
> - ret += mcgen('''
> -void visit_type_%(name)sList(Visitor *m, %(name)sList **obj, const char
> *name, Error **errp);
> -''',
> name=name)
>
> + ret += mcgen('''
> +void visit_type_%(name)sList(Visitor *m, %(name)sList **obj, const char
> *name, Error **errp);
> +''',
> + name=name)
> +
> + return ret
> +
> +def generate_enum_declaration(name, members):
> + ret = mcgen('''
> +void visit_type_%(name)sList(Visitor *m, %(name)sList **obj, const char
> *name, Error **errp);
> +''',
> + name=name)
> +
> return ret
>
> -def generate_decl_enum(name, members, genlist=True):
> +def generate_decl_enum(name, members):
> return mcgen('''
>
> void visit_type_%(name)s(Visitor *m, %(name)s *obj, const char *name, Error
> **errp);
> @@ -542,8 +539,7 @@ exprs = parse_schema(input_file)
> # 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(generate_declaration(typename, None, builtin_type=True))
> fdecl.write(guardend("QAPI_VISIT_BUILTIN_VISITOR_DECL"))
>
> # ...this doesn't work for cases where we link in multiple objects that