[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 23/23] test-clone-visitor: Correct an accidental rename
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v2 23/23] test-clone-visitor: Correct an accidental rename |
Date: |
Tue, 21 Sep 2021 07:17:53 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.1.0 |
On 9/17/21 16:31, Markus Armbruster wrote:
> Commit b359f4b203 "tests: Rename UserDefNativeListUnion to
> UserDefListUnion" renamed test_clone_native_list() to
> test_clone_list_union(). The function has nothing to do with unions.
> Rename it to test_clone_list().
>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> Reviewed-by: Eric Blake <eblake@redhat.com>
> ---
> tests/unit/test-clone-visitor.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Maybe nitpicking, while this patch is related to the series,
although I understand you noticed the mistake while working
on this series, I'd move this cleanup as 1/23.
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
- [PATCH v2 01/23] qapi: Tidy up unusual line breaks, (continued)
- [PATCH v2 01/23] qapi: Tidy up unusual line breaks, Markus Armbruster, 2021/09/17
- [PATCH v2 06/23] qapi: Convert simple union MemoryDeviceInfo to flat one, Markus Armbruster, 2021/09/17
- [PATCH v2 16/23] tests/qapi-schema: Simple union UserDefListUnion is now unused, drop, Markus Armbruster, 2021/09/17
- [PATCH v2 15/23] tests/qapi-schema: Wean off UserDefListUnion, Markus Armbruster, 2021/09/17
- [PATCH v2 05/23] qapi: Convert simple union TpmTypeOptions to flat one, Markus Armbruster, 2021/09/17
- [PATCH v2 03/23] qapi: Convert simple union KeyValue to flat one, Markus Armbruster, 2021/09/17
- [PATCH v2 11/23] tests/qapi-schema: Prepare for simple union UserDefListUnion removal, Markus Armbruster, 2021/09/17
- [PATCH v2 07/23] qapi: Convert simple union ChardevBackend to flat one, Markus Armbruster, 2021/09/17
- [PATCH v2 19/23] tests/qapi-schema: Drop simple union __org.qemu_x-Union1, Markus Armbruster, 2021/09/17
- [PATCH v2 23/23] test-clone-visitor: Correct an accidental rename, Markus Armbruster, 2021/09/17
- Re: [PATCH v2 23/23] test-clone-visitor: Correct an accidental rename,
Philippe Mathieu-Daudé <=
- [PATCH v2 13/23] test-qobject-output-visitor: Wean off UserDefListUnion, Markus Armbruster, 2021/09/17
- [PATCH v2 20/23] tests/qapi-schema: Purge simple unions from tests, Markus Armbruster, 2021/09/17
- [PATCH v2 21/23] qapi: Drop simple unions, Markus Armbruster, 2021/09/17
- [PATCH v2 08/23] qapi: Convert simple union SocketAddressLegacy to flat one, Markus Armbruster, 2021/09/17
- [PATCH v2 14/23] test-clone-visitor: Wean off UserDefListUnion, Markus Armbruster, 2021/09/17
- [PATCH v2 18/23] test-clone-visitor: Wean off __org.qemu_x-Union1, Markus Armbruster, 2021/09/17
- [PATCH v2 22/23] tests/qapi-schema: Rename flat-union-* test cases to union-*, Markus Armbruster, 2021/09/17
- [PATCH v2 02/23] qapi: Stop enforcing "type name should not end in 'Kind', Markus Armbruster, 2021/09/17