qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v6 00/15] post-introspection cleanups, subset C


From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH v6 00/15] post-introspection cleanups, subset C
Date: Fri, 16 Oct 2015 09:23:25 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0

On 10/07/2015 08:00 PM, Eric Blake wrote:
> Pending prerequisite: Markus' qapi-next branch (which has my
> subset A patches):
> git://repo.or.cz/qemu/armbru.git qapi-next
> http://thread.gmane.org/gmane.comp.emulators.qemu/365827/focus=366351
> as well as my subset B patches (currently at v7):
> http://article.gmane.org/gmane.comp.emulators.qemu/366810
> http://repo.or.cz/qemu/ericb.git/shortlog/refs/tags/qapi-cleanupv7b
> 

I'm about to post a rebase of this series, depending on my recent subset
B' v9, if that affects any reviewer's plans.
https://lists.gnu.org/archive/html/qemu-devel/2015-10/msg03730.html

> Also available as a tag at this location:
> git fetch git://repo.or.cz/qemu/ericb.git qapi-cleanupv6c
> 
> and I plan to eventually forcefully update my branch with the rest
> of the v5 series, at:
> http://repo.or.cz/qemu/ericb.git/shortlog/refs/heads/qapi
> 
> v6 notes:
> Add some patches and rebase onto work on subset B. Rearrange some
> patches from v5 (this set includes 17-20, 23, 25-27). Backport diff
> gets a bit confused by one patch title changing.
> 
> 001/15:[down] 'qapi: Move empty-enum to compile-time test'
> 002/15:[down] 'qapi: Drop redundant returns-int test'
> 003/15:[down] 'qapi: Drop redundant flat-union-reverse-define test'

Meanwhile, these three already got hoisted into subset B and are in a
pending pull request.

> 004/15:[down] 'qapi: Use generated TestStruct machinery in tests'
> 005/15:[----] [--] 'qapi: Provide nicer array names in introspection'
> 006/15:[----] [--] 'qapi-introspect: Guarantee particular sorting'
> 007/15:[down] 'qapi: Change alternate layout to use 'type''

and this one may disappear completely, since v9B does the rename to type
in a cleaner fashion.

> 008/15:[0141] [FC] 'qapi: Simplify visiting of alternate types'
> 009/15:[0023] [FC] 'qapi: Fix alternates that accept 'number' but not 'int''
> 010/15:[----] [--] 'qapi: Remove dead visitor code'
> 011/15:[down] 'qapi: Plug leaks in test-qmp-*'
> 012/15:[down] 'qapi: Simplify error testing in test-qmp-*'
> 013/15:[0007] [FC] 'qapi: Test failure in middle of array parse'
> 014/15:[down] 'qapi: More tests of input arrays'
> 015/15:[0021] [FC] 'qapi: Simplify visits of optional fields'
> 
> Not much direct review comments, although some of the changes here
> are updated based on comments made on other patches in the v5 series.
> 
> Subset D (and more?) will come later.
> 

Still true.

-- 
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]