From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56577) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YkYsH-0000CI-68 for qemu-devel@nongnu.org; Tue, 21 Apr 2015 10:10:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YkYsC-0004LN-2P for qemu-devel@nongnu.org; Tue, 21 Apr 2015 10:10:05 -0400 Date: Tue, 21 Apr 2015 10:09:02 -0400 From: Luiz Capitulino Message-ID: <20150421100902.10c5dbd7@redhat.com> In-Reply-To: <1428700079-19713-1-git-send-email-eblake@redhat.com> References: <1428700079-19713-1-git-send-email-eblake@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2] qapi: Drop dead genlist parameter List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Blake Cc: qemu-trivial@nongnu.org, qemu-devel@nongnu.org, armbru@redhat.com On Fri, 10 Apr 2015 15:07:59 -0600 Eric Blake 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 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 > @@ -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