[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/25] qdev: use object_property_print in info q
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH 07/25] qdev: use object_property_print in info qtree |
Date: |
Fri, 11 May 2012 16:10:34 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120421 Thunderbird/12.0 |
Am 03.04.2012 15:05, schrieb Paolo Bonzini:
> Il 03/04/2012 14:28, Jan Kiszka ha scritto:
>>>> if (object_property_get_type(OBJECT(dev), legacy_name, NULL)) {
>>>> value = object_property_get_str(OBJECT(dev), legacy_name,
>>>> &err);
>> This is not a criticism on this patch, but I'd like to underline that
>> the line above only works because legacy props also define print
> ^^^^
>
> You mean always, I guess.
>
>> But this statement above is still inconsistent. We should either
>> assert(print && parse) or handle their non-existence properly.
>
> Yes, asserting that print/parse are always present in pairs (if at all)
> would be good.
Paolo, has this issue been addressed somewhere?
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- Re: [Qemu-devel] [PATCH 07/25] qdev: use object_property_print in info qtree,
Andreas Färber <=