qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2 00/14] drop qapi nested structs


From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH v2 00/14] drop qapi nested structs
Date: Tue, 05 Aug 2014 19:12:27 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.7.0

On 08/05/2014 04:38 PM, Eric Blake wrote:
> According to this email:
> https://lists.gnu.org/archive/html/qemu-devel/2014-05/msg00708.html
> we want to repurpose 'data': { 'name': {dict...} } in qapi files
> for future use of designating default values of optional parameters.
> But to do that, we must first nuke existing use of that syntax for
> declaring nested structs.  Enhancing the testsuite while at it
> never hurts.
> 
> v2:
>   New patches: 1-2, 5-9
>   consistent TAB usage in Makefile [Fam]
>   catch more bad coding constructs, and test them
>   avoid code duplication in type validity checks (patch 14 [former 7] is 
> simpler because of patch 9)
> 
> Eric Blake (14):
>   qapi: consistent whitespace in tests/Makefile
>   qapi: ignore files created during make check
>   qapi: add some enum tests
>   qapi: better error message for bad enum
>   qapi: add some expr tests
>   qapi: require valid expressions
>   qapi: add some type check tests
>   qapi: add expr_name() helper
>   qapi: add check_type helper function
>   qapi: merge UserDefTwo and UserDefNested in tests
>   qapi: drop tests for inline subtypes
>   qapi: drop inline subtype in query-version
>   qapi: drop inline subtype in query-pci
>   qapi: drop support for inline subtypes

Blah; I messed up the attempt to resume after a failed git send-email.
Better to just send a v3 than risk confusion with two different patches
at v2 for 11-14.


-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]