[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 13/22] test-qobject-output-visitor: Wean off UserDefListUnion
From: |
Eric Blake |
Subject: |
Re: [PATCH 13/22] test-qobject-output-visitor: Wean off UserDefListUnion |
Date: |
Mon, 13 Sep 2021 10:09:58 -0500 |
User-agent: |
NeoMutt/20210205-772-2b4c52 |
On Mon, Sep 13, 2021 at 02:39:23PM +0200, Markus Armbruster wrote:
> The test_visitor_out_list_union_FOO() use simple union
> UserDefListUnion to cover lists of builtin types. Rewrite as
> test_visitor_out_list_struct(), using struct ArrayStruct and a lot
> less code.
>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
> tests/unit/test-qobject-output-visitor.c | 391 ++++++-----------------
> 1 file changed, 93 insertions(+), 298 deletions(-)
>
Nice simplification.
Reviewed-by: Eric Blake <eblake@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
- Re: [PATCH 03/22] qapi: Convert simple union KeyValue to flat one, (continued)
- [PATCH 07/22] qapi: Convert simple union ChardevBackend to flat one, Markus Armbruster, 2021/09/13
- [PATCH 17/22] tests/qapi-schema: Rewrite simple union TestIfUnion to be flat, Markus Armbruster, 2021/09/13
- [PATCH 09/22] qapi: Convert simple union ImageInfoSpecific to flat one, Markus Armbruster, 2021/09/13
- [PATCH 19/22] tests/qapi-schema: Drop simple union __org.qemu_x-Union1, Markus Armbruster, 2021/09/13
- [PATCH 13/22] test-qobject-output-visitor: Wean off UserDefListUnion, Markus Armbruster, 2021/09/13
- Re: [PATCH 13/22] test-qobject-output-visitor: Wean off UserDefListUnion,
Eric Blake <=
- [PATCH 08/22] qapi: Convert simple union SocketAddressLegacy to flat one, Markus Armbruster, 2021/09/13
- [PATCH 18/22] test-clone-visitor: Wean off __org.qemu_x-Union1, Markus Armbruster, 2021/09/13
- [PATCH 21/22] qapi: Drop simple unions, Markus Armbruster, 2021/09/13
- Re: [PATCH 00/22] qapi: Remove simple unions from the schema language, Markus Armbruster, 2021/09/13