[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 33/36] qapi/types.py: remove one-letter variables
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v5 33/36] qapi/types.py: remove one-letter variables |
Date: |
Wed, 07 Oct 2020 14:42:00 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
John Snow <jsnow@redhat.com> writes:
> "John, if pylint told you to jump off a bridge, would you?"
> Hey, if it looked like fun, I might.
>
> Now that this file is clean, enable pylint checks on this file.
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> Reviewed-by: Eduardo Habkost <ehabkost@redhat.com>
> Reviewed-by: Cleber Rosa <crosa@redhat.com>
> ---
> scripts/qapi/pylintrc | 1 -
> scripts/qapi/types.py | 29 +++++++++++++++--------------
> 2 files changed, 15 insertions(+), 15 deletions(-)
>
> diff --git a/scripts/qapi/pylintrc b/scripts/qapi/pylintrc
> index 8badcb11cda..b3c4cf46dbf 100644
> --- a/scripts/qapi/pylintrc
> +++ b/scripts/qapi/pylintrc
> @@ -6,7 +6,6 @@ ignore-patterns=error.py,
> expr.py,
> parser.py,
> schema.py,
> - types.py,
> visit.py,
>
>
> diff --git a/scripts/qapi/types.py b/scripts/qapi/types.py
> index 766822feb3a..9d1e79d503d 100644
> --- a/scripts/qapi/types.py
> +++ b/scripts/qapi/types.py
> @@ -49,14 +49,14 @@ def gen_enum_lookup(name: str,
> .array = (const char *const[]) {
> ''',
> c_name=c_name(name))
> - for m in members:
> - ret += gen_if(m.ifcond)
> - index = c_enum_const(name, m.name, prefix)
> + for member in members:
Let's use @memb. It's more visually distinct from @members, and
gen_struct_members() already uses it.
> + ret += gen_if(member.ifcond)
> + index = c_enum_const(name, member.name, prefix)
> ret += mcgen('''
> [%(index)s] = "%(name)s",
> ''',
> - index=index, name=m.name)
> - ret += gen_endif(m.ifcond)
> + index=index, name=member.name)
> + ret += gen_endif(member.ifcond)
>
> ret += mcgen('''
> },
> @@ -79,13 +79,13 @@ def gen_enum(name: str,
> ''',
> c_name=c_name(name))
>
> - for m in enum_members:
> - ret += gen_if(m.ifcond)
> + for member in enum_members:
> + ret += gen_if(member.ifcond)
> ret += mcgen('''
> %(c_enum)s,
> ''',
> - c_enum=c_enum_const(name, m.name, prefix))
> - ret += gen_endif(m.ifcond)
> + c_enum=c_enum_const(name, member.name, prefix))
> + ret += gen_endif(member.ifcond)
>
> ret += mcgen('''
> } %(c_name)s;
> @@ -148,11 +148,12 @@ def gen_object(name: str, ifcond: List[str],
> objects_seen.add(name)
>
> ret = ''
> - if variants:
> - for v in variants.variants:
> - if isinstance(v.type, QAPISchemaObjectType):
> - ret += gen_object(v.type.name, v.type.ifcond, v.type.base,
> - v.type.local_members, v.type.variants)
> + for variant in variants.variants if variants else ():
Let's use @var, for consistency with gen_variants().
> + obj = variant.type
> + if not isinstance(obj, QAPISchemaObjectType):
> + continue
> + ret += gen_object(obj.name, obj.ifcond, obj.base,
> + obj.local_members, obj.variants)
>
> ret += mcgen('''
- Re: [PATCH v5 30/36] qapi/gen.py: update write() to be more idiomatic, (continued)
[PATCH v5 33/36] qapi/types.py: remove one-letter variables, John Snow, 2020/10/05
- Re: [PATCH v5 33/36] qapi/types.py: remove one-letter variables,
Markus Armbruster <=
[PATCH v5 35/36] qapi/visit.py: remove unused parameters from gen_visit_object, John Snow, 2020/10/05
[PATCH v5 36/36] qapi/visit.py: add type hint annotations, John Snow, 2020/10/05
Re: [PATCH v5 00/36] qapi: static typing conversion, pt1, Cleber Rosa, 2020/10/05
Re: [PATCH v5 00/36] qapi: static typing conversion, pt1, Markus Armbruster, 2020/10/07