diff options
author | John Snow <jsnow@redhat.com> | 2020-10-09 12:15:28 -0400 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2020-10-10 11:37:47 +0200 |
commit | 5af8263d40c698c47befd4c0bed3d6c452b56d82 (patch) | |
tree | 36e92c1264edfd0b64e4d059440a5d66bb78be94 /scripts/qapi/gen.py | |
parent | 7137a96099644734cd6045313823840d4cecd5e8 (diff) | |
download | qemu-5af8263d40c698c47befd4c0bed3d6c452b56d82.zip qemu-5af8263d40c698c47befd4c0bed3d6c452b56d82.tar.gz qemu-5af8263d40c698c47befd4c0bed3d6c452b56d82.tar.bz2 |
qapi: Remove wildcard includes
Wildcard includes become hard to manage when refactoring and dealing
with circular dependencies with strictly typed mypy.
flake8 also flags each one as a warning, as it is not smart enough to
know which names exist in the imported file.
Remove them and include things explicitly by name instead.
Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Eduardo Habkost <ehabkost@redhat.com>
Reviewed-by: Cleber Rosa <crosa@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20201009161558.107041-7-jsnow@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'scripts/qapi/gen.py')
-rw-r--r-- | scripts/qapi/gen.py | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/scripts/qapi/gen.py b/scripts/qapi/gen.py index fc57fdc..1fed712 100644 --- a/scripts/qapi/gen.py +++ b/scripts/qapi/gen.py @@ -11,13 +11,19 @@ # This work is licensed under the terms of the GNU GPL, version 2. # See the COPYING file in the top-level directory. - +from contextlib import contextmanager import errno import os import re -from contextlib import contextmanager -from .common import * +from .common import ( + c_fname, + gen_endif, + gen_if, + guardend, + guardstart, + mcgen, +) from .schema import QAPISchemaVisitor |