From: Markus Armbruster <armbru@redhat.com>
To: qemu-devel@nongnu.org
Cc: jsnow@redhat.com, peter.maydell@linaro.org, michael.roth@amd.com
Subject: [PATCH 0/7] qapi: Refactor QAPISchemaVariants
Date: Wed, 20 Mar 2024 08:43:08 +0100 [thread overview]
Message-ID: <20240320074315.23167-1-armbru@redhat.com> (raw)
QAPISchemaVariants represents either a union type's branches, or an
alternate type's alternatives. Much of its code is conditional on
which one it actually is.
This series moves the conditional code to new subtypes
QAPISchemaBranches and QAPISchemaAlternatives.
This also lets us treat QAPISchemaVariants.tag_member like the other
attribute that become known only in .check().
Markus Armbruster (7):
qapi: New QAPISchemaBranches, QAPISchemaAlternatives
qapi: Rename visitor parameter @variants to @branches
qapi: Rename visitor parameter @variants to @alternatives
qapi: Rename QAPISchemaObjectType.variants to .branches
qapi: Rename QAPISchemaAlternateType.variants to .alternatives
qapi: Move conditional code from QAPISchemaVariants to its subtypes
qapi: Simplify QAPISchemaVariants @tag_member
docs/sphinx/qapidoc.py | 21 ++--
scripts/qapi/commands.py | 2 +-
scripts/qapi/events.py | 2 +-
scripts/qapi/gen.py | 2 +-
scripts/qapi/introspect.py | 15 +--
scripts/qapi/schema.py | 223 +++++++++++++++++----------------
scripts/qapi/types.py | 12 +-
scripts/qapi/visit.py | 24 ++--
tests/qapi-schema/test-qapi.py | 9 +-
9 files changed, 163 insertions(+), 147 deletions(-)
--
2.44.0
next reply other threads:[~2024-03-20 7:44 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-20 7:43 Markus Armbruster [this message]
2024-03-20 7:43 ` [PATCH 1/7] qapi: New QAPISchemaBranches, QAPISchemaAlternatives Markus Armbruster
2024-03-20 7:43 ` [PATCH 2/7] qapi: Rename visitor parameter @variants to @branches Markus Armbruster
2024-03-20 7:43 ` [PATCH 3/7] qapi: Rename visitor parameter @variants to @alternatives Markus Armbruster
2024-03-20 7:43 ` [PATCH 4/7] qapi: Rename QAPISchemaObjectType.variants to .branches Markus Armbruster
2024-03-20 7:43 ` [PATCH 5/7] qapi: Rename QAPISchemaAlternateType.variants to .alternatives Markus Armbruster
2024-03-20 7:43 ` [PATCH 6/7] qapi: Move conditional code from QAPISchemaVariants to its subtypes Markus Armbruster
2024-03-20 7:43 ` [PATCH 7/7] qapi: Simplify QAPISchemaVariants @tag_member Markus Armbruster
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240320074315.23167-1-armbru@redhat.com \
--to=armbru@redhat.com \
--cc=jsnow@redhat.com \
--cc=michael.roth@amd.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).