[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check() |
Date: |
Wed, 14 Oct 2015 15:23:15 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 10/14/2015 01:32 AM, Markus Armbruster wrote:
>
>>> Done & pushed to http://repo.or.cz/qemu/armbru.git branch qapi-next.
>>
>> Redone & pushed.
>>
>> Summary of tweaks:
>> * PATCH 01-06.5: unchanged
>> * PATCH 07: your fixup to drop is_implicit() squashed in
>> * PATCH 08: array info plumbing moved to PATCH 12
>> * PATCH 09: trivially rebased
>> * PATCH 12: revert your fixup, so we get the move we want, adjust commit
>> message accordingly, supply missing QAPISchemaArrayType.is_implicit()
>
> You may also want to squash this into 12:
>
> diff --git a/scripts/qapi.py b/scripts/qapi.py
> index fd95864..1e01714 100644
> --- a/scripts/qapi.py
> +++ b/scripts/qapi.py
> @@ -792,10 +792,9 @@ class QAPISchemaEntity(object):
> self.name = name
> # For explicitly defined entities, info points to the (explicit)
> # definition. For builtins (and their arrays), info is None.
> - # For other arrays, info points to an explicit place that uses
> - # the array (there may be more than one such place). For other
> - # implicitly defined entities, it points to the place that
> - # triggered the implicit definition.
> + # For implicitly defined entities, info points to a place that
> + # triggered the implicit definition (there may be more than one
> + # such place).
> self.info = info
>
> def c_name(self):
Done. Thanks!
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), (continued)
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), Markus Armbruster, 2015/10/13
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), Eric Blake, 2015/10/13
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), Markus Armbruster, 2015/10/13
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), Eric Blake, 2015/10/13
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), Markus Armbruster, 2015/10/13
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), Eric Blake, 2015/10/13
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), Eric Blake, 2015/10/13
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), Markus Armbruster, 2015/10/14
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), Markus Armbruster, 2015/10/14
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), Eric Blake, 2015/10/14
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(),
Markus Armbruster <=
[Qemu-devel] [PATCH v8 18/18] qapi: Detect base class loops, Eric Blake, 2015/10/13
[Qemu-devel] [PATCH v8 16/18] qapi: Move duplicate enum value checks to schema check(), Eric Blake, 2015/10/13
[Qemu-devel] [PATCH v8 06.5/18] qapi: Drop redundant args-member-array test, Eric Blake, 2015/10/13
[Qemu-devel] [PATCH v8 06/18] qapi: Drop redundant flat-union-reverse-define test, Eric Blake, 2015/10/15
[Qemu-devel] [PATCH v8 02/18] qapi: Prepare for errors during check(), Eric Blake, 2015/10/15
[Qemu-devel] [PATCH v8 13/18] qapi: Track owner of each object member, Eric Blake, 2015/10/15