]> git.ipfire.org Git - thirdparty/qemu.git/commit
qapi: Move gen_enum(), gen_enum_lookup() back to qapi/types.py
authorMarkus Armbruster <armbru@redhat.com>
Fri, 18 Oct 2019 07:43:43 +0000 (09:43 +0200)
committerMarkus Armbruster <armbru@redhat.com>
Tue, 22 Oct 2019 07:26:12 +0000 (09:26 +0200)
commit61bfb2e1a4666817b9d94f0a96109f8ef51b812b
treee607d3f320a0c46e5998985e436655a6198566f8
parentf01338cce692ac54109f09bc6c7b5567611e2d24
qapi: Move gen_enum(), gen_enum_lookup() back to qapi/types.py

The next commit will split up qapi/common.py.  gen_enum() needs
QAPISchemaEnumMember, and that's in the way.  Move it to qapi/types.py
along with its buddy gen_enum_lookup().

Permit me a short a digression on history: how did gen_enum() end up
in qapi/common.py?  Commit 21cd70dfc1 "qapi script: add event support"
duplicated qapi-types.py's gen_enum() and gen_enum_lookup() in
qapi-event.py.  Simply importing them would have been cleaner, but
wasn't possible as qapi-types.py was a program, not a module.  Commit
efd2eaa6c2 "qapi: De-duplicate enum code generation" de-duplicated by
moving them to qapi.py, which was a module.

Since then, program qapi-types.py has morphed into module types.py.
It's where gen_enum() and gen_enum_lookup() started, and where they
belong.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20191018074345.24034-6-armbru@redhat.com>
scripts/qapi/common.py
scripts/qapi/events.py
scripts/qapi/types.py