[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/10] qom: add get_id
From: |
malc |
Subject: |
Re: [Qemu-devel] [PATCH 04/10] qom: add get_id |
Date: |
Fri, 25 May 2012 21:39:34 +0400 (MSK) |
User-agent: |
Alpine 2.00 (LNX 1167 2008-08-23) |
On Fri, 25 May 2012, Andreas F?rber wrote:
> Am 23.05.2012 17:44, schrieb Paolo Bonzini:
> > Some classes may present objects differently in errors, for example if they
> > are not part of the composition tree or if they are not assigned an id by
> > the user. Let them do this with a get_id method on Object, and use the
> > method consistently where a %(device) appears in the error.
> >
> > Signed-off-by: Paolo Bonzini <address@hidden>
>
> This patch looks new, so it's not covered by Anthony's carte blanche to
> take Paolo's patches into qom-next.
>
> Generally I'm fine with it, some formal issues below.
>
> > ---
> > hw/qdev-properties.c | 6 +++---
> > hw/qdev.c | 15 ++++++++++++++-
> > include/qemu/object.h | 11 +++++++++++
> > qom/object.c | 26 +++++++++++++++++++++++++-
> > 4 files changed, 53 insertions(+), 5 deletions(-)
>
> > diff --git a/hw/qdev.c b/hw/qdev.c
> > index e909f3b..5d6dc1f 100644
> > --- a/hw/qdev.c
> > +++ b/hw/qdev.c
> [...]
> > @@ -714,6 +714,13 @@ static void device_finalize(Object *obj)
> > }
> > }
> >
> > +static const char *qdev_get_id(Object *obj)
> > +{
> > + DeviceState *dev = DEVICE(obj);
> > +
> > + return dev->id?:object_get_typename(obj);
>
> Spaces around ?: please.
Even better is to not use gcc extensions like this one.
>
> > +}
> > +
> > static void device_class_base_init(ObjectClass *class, void *data)
> > {
> > DeviceClass *klass = DEVICE_CLASS(class);
> [...]
> > diff --git a/qom/object.c b/qom/object.c
> > index 68a4c57..b19ef94 100644
> > --- a/qom/object.c
> > +++ b/qom/object.c
> > @@ -334,6 +334,24 @@ static void object_property_del_child(Object *obj,
> > Object *child, Error **errp)
> > }
> > }
> >
> > +static const char *_object_get_id(Object *obj)
>
> malc's alarm bells will be ringing: Use of an identifier starting with
> underscore.
Given that it's forbidden in this context - yeah.
[..snip..]
--
mailto:address@hidden
- [Qemu-devel] [PATCH 05/10] qdev: push state up to Object, (continued)
- [Qemu-devel] [PATCH 05/10] qdev: push state up to Object, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 06/10] qdev: generalize properties to Objects, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 07/10] qdev: move bulk of qdev-properties.c to qom/object.c, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 09/10] qom: add realized property, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 10/10] qom: Add QERR_PROPERTY_SET_AFTER_REALIZE, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 04/10] qom: add get_id, Paolo Bonzini, 2012/05/23
- [Qemu-devel] [PATCH 08/10] qom: push static properties to Object, Paolo Bonzini, 2012/05/23