[Top][All Lists]

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

Re: [Qemu-devel] [PATCH v2 3/4] qapi: Correctly handle downstream extens

From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH v2 3/4] qapi: Correctly handle downstream extensions in more locations
Date: Thu, 30 Apr 2015 16:30:23 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0

On 04/29/2015 09:30 AM, Eric Blake wrote:
> On 04/29/2015 05:29 AM, Markus Armbruster wrote:
>> Eric Blake <address@hidden> writes:
>>> Now that c_var() handles '.' in downstream extension names, fix
>>> the generator to support such names as additional types, enums,
>>> members within an enum, branches of a union or alternate, and
>>> in arrays.

>>>  def generate_visit_struct_fields(name, field_prefix, fn_prefix, members, 
>>> base = None):
>>> +    assert field_prefix == ""
>> Makes me wonder why we have a field_prefix parameter.
>> fn_prefix is also always ""...
> Hmm - looks like I was debugging whether the code was dead (by whether I
> could trigger the assertion) and forgot to clean it up.  I'll split that
> into a separate patch (I've already done another related cleanup that I
> found while reviewing this code, in commit 6540e9f).

In fact, the code was NOT dead until I killed inline structs ("qapi:
Drop support for inline nested types").  Which means my cleanup wasn't
complete - but as that series is already at v7, I'll do the additional
cleanup as its own 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]