[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: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check() |
Date: |
Wed, 14 Oct 2015 06:59:21 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
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):
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), (continued)
- [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(), 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 <=
- Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check(), Markus Armbruster, 2015/10/14
[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