[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/4] qapi: Handle some keyval fallout
|
From: |
Marc-André Lureau |
|
Subject: |
Re: [Qemu-devel] [PATCH 0/4] qapi: Handle some keyval fallout |
|
Date: |
Fri, 26 May 2017 10:30:21 +0000 |
On Mon, May 22, 2017 at 8:45 PM Markus Armbruster <address@hidden> wrote:
> Markus Armbruster (4):
> qobject-input-visitor: Reject non-finite numbers with keyval
> qapi: Document visit_type_any() issues with keyval input
> tests/qapi-schema: Avoid 'str' in alternate test cases
> qapi: Reject alternates that can't work with keyval_parse()
>
> include/qapi/visitor.h | 4 ++
> qapi/qobject-input-visitor.c | 3 +-
> scripts/qapi.py | 19 ++++++-
> tests/Makefile.include | 2 +
> tests/qapi-schema/alternate-clash.json | 2 +-
> tests/qapi-schema/alternate-conflict-dict.json | 2 +-
> tests/qapi-schema/alternate-conflict-enum-bool.err | 1 +
> .../qapi-schema/alternate-conflict-enum-bool.exit | 1 +
> .../qapi-schema/alternate-conflict-enum-bool.json | 6 +++
> tests/qapi-schema/alternate-conflict-enum-bool.out | 0
> tests/qapi-schema/alternate-conflict-enum-int.err | 1 +
> tests/qapi-schema/alternate-conflict-enum-int.exit | 1 +
> tests/qapi-schema/alternate-conflict-enum-int.json | 6 +++
> tests/qapi-schema/alternate-conflict-enum-int.out | 0
> tests/qapi-schema/alternate-conflict-string.err | 2 +-
> tests/qapi-schema/alternate-conflict-string.json | 6 +--
> tests/qapi-schema/alternate-nested.json | 2 +-
> tests/qapi-schema/args-alternate.json | 2 +-
> tests/qapi-schema/doc-bad-alternate-member.json | 2 +-
> tests/qapi-schema/qapi-schema-test.json | 13 +++--
> tests/qapi-schema/qapi-schema-test.out | 34 ++++++------
> tests/qapi-schema/returns-alternate.json | 2 +-
> tests/test-clone-visitor.c | 23 ++++----
> tests/test-keyval.c | 18 ++++---
> tests/test-qobject-input-visitor.c | 62
> ++++++++++++----------
> tests/test-qobject-output-visitor.c | 4 +-
> util/keyval.c | 10 ++--
> 27 files changed, 140 insertions(+), 88 deletions(-)
> create mode 100644 tests/qapi-schema/alternate-conflict-enum-bool.err
> create mode 100644 tests/qapi-schema/alternate-conflict-enum-bool.exit
> create mode 100644 tests/qapi-schema/alternate-conflict-enum-bool.json
> create mode 100644 tests/qapi-schema/alternate-conflict-enum-bool.out
> create mode 100644 tests/qapi-schema/alternate-conflict-enum-int.err
> create mode 100644 tests/qapi-schema/alternate-conflict-enum-int.exit
> create mode 100644 tests/qapi-schema/alternate-conflict-enum-int.json
> create mode 100644 tests/qapi-schema/alternate-conflict-enum-int.out
>
>
Reviewed-by: Marc-André Lureau <address@hidden>
--
Marc-André Lureau