[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 06/27] qom: add QObject-based property get/se
|
From: |
Luiz Capitulino |
|
Subject: |
Re: [Qemu-devel] [PATCH v2 06/27] qom: add QObject-based property get/set wrappers |
|
Date: |
Wed, 8 Feb 2012 10:29:24 -0200 |
On Tue, 07 Feb 2012 10:12:34 +0100
Paolo Bonzini <address@hidden> wrote:
> On 02/06/2012 03:16 PM, Anthony Liguori wrote:
> >
> >> Move the creation of QmpInputVisitor and QmpOutputVisitor from
> >> qmp.c to qom/object.c, since it's the only practical way to access
> >> object properties.
> >>
> >> Signed-off-by: Paolo Bonzini<address@hidden>
> >
> > Reluctantly-Reviewed-by: Anthony Liguori <address@hidden>
>
> Fair enough, thanks very much. :)
>
> > As long as we keep this isolated such that it's easy to remove. At some
> > point, we need to remove all usage of QObject in the tree and replace it
> > with GVariant.
>
> Agreed, we'll revisit after Luiz completes QAPI.
Commands are not far away. Events are still missing, although they could be
converted along with the introduction of GVariant.
- [Qemu-devel] [PATCH v2 08/27] qom: fix off-by-one, (continued)
- [Qemu-devel] [PATCH v2 14/27] qdev: allow reusing get/set for legacy property, Paolo Bonzini, 2012/02/04
- [Qemu-devel] [PATCH v2 13/27] qdev: remove direct calls to print/parse, Paolo Bonzini, 2012/02/04
- [Qemu-devel] [PATCH v2 07/27] qom: add property get/set wrappers for C types, Paolo Bonzini, 2012/02/04
- [Qemu-devel] [PATCH v2 10/27] qom: use object_resolve_path_type for links, Paolo Bonzini, 2012/02/04
- [Qemu-devel] [PATCH v2 17/27] qdev: remove parse/print methods for mac properties, Paolo Bonzini, 2012/02/04