qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v7 05/18] qapi: Test for various name collisions


From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH v7 05/18] qapi: Test for various name collisions
Date: Thu, 1 Oct 2015 12:51:53 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0

On 10/01/2015 11:39 AM, Markus Armbruster wrote:

>> I'm leaning towards A (calling self-inheritance a name collision is a
>> bit of a stretch in the first place; and leaving it untested until 16/46
>> goes in doesn't hurt).
> 
> Okay, A. it is.  I pushed to branch qapi-next at
> http://repo.or.cz/qemu/armbru.git

Branch looks good to go with one nit: 15/18 commit message (currently
e7462ff) has a typo in the text you massaged:

    The generated code has fewer blank line in qapi-event.c functions,
    but has no semantic difference.

s/line/lines/

I'll rebase my remaining patches on top of your tip, and let you decide
how long to wait before sending the pull request.  And with your rewrite
of 17/18, this part of the series no longer depends on the python 2.6
configure patch.

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