[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 0/3] string-input-visitor: Support enum/bool alte
From: |
Eduardo Habkost |
Subject: |
[Qemu-devel] [PATCH v2 0/3] string-input-visitor: Support enum/bool alternate types |
Date: |
Fri, 5 May 2017 17:11:25 -0300 |
v1 was originally part of the following series:
[PATCH 0/4] x86: Support "-cpu feature=force"
But as I am not sure yet "feature=force" is the right way to go,
I decided to send only the string-input-visitor alternate
support, in case the QAPI maintainers want to merge it anyway.
Changes v1 -> v2:
* Updated string_input_visitor_new() documentation
to mention alternate support (Markus Armbruster)
* Detect ambiguous alternates at runtime. Test case included.
* Removed support for integers. We don't need it yet, and
it would require sorting out the parse_str() mess.
* Change supported_qtypes to uint32_t (Eric Blake)
* Replace assert() on all generated visitor functions with a
single QEMU_BUILD_BUG_ON() on visit_start_alternate()
(Eric Blake)
* Extra spaces around "|" on generated visitor code
(Eric Blake)
* Don't use bitops.h and just use (1U << QTYPE_FOO)
(Markus Armbruster)
* Update tests/qapi-schema/qapi-schema-test.out to match
qapi-schema-test.json updates
* New patch: qapi: Add enum_table[] parameter to start_alternate
(needed to detect ambiguous alternates)
* Code indentation fix (Markus Armbruster)
Eduardo Habkost (3):
visitor: Add 'supported_qtypes' parameter to visit_start_alternate()
qapi: Add enum_table[] parameter to start_alternate
string-input-visitor: Support alternate types
include/qapi/visitor.h | 13 ++++-
include/qapi/visitor-impl.h | 4 +-
scripts/qapi-visit.py | 17 ++++--
include/qapi/string-input-visitor.h | 6 +-
qapi/qapi-visit-core.c | 12 +++-
qapi/qapi-clone-visitor.c | 4 +-
qapi/qapi-dealloc-visitor.c | 4 +-
qapi/qobject-input-visitor.c | 7 ++-
qapi/string-input-visitor.c | 99 +++++++++++++++++++++++++++++----
tests/test-string-input-visitor.c | 76 +++++++++++++++++++++++++
qapi/trace-events | 2 +-
tests/qapi-schema/qapi-schema-test.json | 8 +++
tests/qapi-schema/qapi-schema-test.out | 9 +++
13 files changed, 234 insertions(+), 27 deletions(-)
--
2.11.0.259.g40922b1