[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] qom: change object property iterator API con
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH v2] qom: change object property iterator API contract |
Date: |
Mon, 11 Jan 2016 16:29:28 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 |
Am 17.12.2015 um 09:54 schrieb Markus Armbruster:
> "Daniel P. Berrange" <address@hidden> writes:
>
>> Currently the object property iterator API works as follows
>>
>> ObjectPropertyIterator *iter;
>>
>> iter = object_property_iter_init(obj);
>> while ((prop = object_property_iter_next(iter))) {
>> ...
>> }
>> object_property_iter_free(iter);
>>
>> This has the benefit that the ObjectPropertyIterator struct
>> can be opaque, but has the downside that callers need to
>> explicitly call a free function. It is also not in keeping
>> with iterator style used elsewhere in QEMU/glib2
>>
>> This patch changes the API to use stack allocation instead
>>
>> ObjectPropertyIterator iter;
>>
>> object_property_iter_init(&iter, obj);
>> while ((prop = object_property_iter_next(&iter))) {
>> ...
>> }
>>
>> Reviewed-by: Eric Blake <address@hidden>
>> Signed-off-by: Daniel P. Berrange <address@hidden>
> [...]
>> diff --git a/include/qom/object.h b/include/qom/object.h
>> index 9630c5b..275a21b 100644
>> --- a/include/qom/object.h
>> +++ b/include/qom/object.h
>> @@ -971,6 +971,11 @@ ObjectProperty *object_class_property_find(ObjectClass
>> *klass, const char *name,
>>
>> typedef struct ObjectPropertyIterator ObjectPropertyIterator;
>>
>> +struct ObjectPropertyIterator {
>> + ObjectClass *nextclass;
>> + GHashTableIter iter;
>> +};
>> +
>
> Please fuse the two declarations. Perhaps Andreas can do that on
> commit.
Done.
>> /**
>> * object_property_iter_init:
>> * @obj: the object
>
> Reviewed-by: Markus Armbruster <address@hidden>
Queued on qom-next:
https://github.com/afaerber/qemu-cpu/commits/qom-next
Thanks,
Andreas
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton; HRB 21284 (AG Nürnberg)
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v2] qom: change object property iterator API contract,
Andreas Färber <=