"Daniel P. Berrange" <berra...@redhat.com> writes: > On Thu, Nov 19, 2015 at 10:20:22AM +0100, Markus Armbruster wrote: >> Andreas Färber <afaer...@suse.de> writes: >> >> > From: "Daniel P. Berrange" <berra...@redhat.com> >> > >> > Some users of QOM need to be able to iterate over properties >> > defined against an object instance. Currently they are just >> > directly using the QTAIL macros against the object properties >> > data structure. >> > >> > This is bad because it exposes them to changes in the data >> > structure used to store properties, as well as changes in >> > functionality such as ability to register properties against >> > the class. >> > >> > This provides an ObjectPropertyIterator struct which will >> > insulate the callers from the particular data structure >> > used to store properties. It can be used thus >> > >> > ObjectProperty *prop; >> > ObjectPropertyIterator *iter; >> > >> > iter = object_property_iter_init(obj); >> > while ((prop = object_property_iter_next(iter))) { >> > ... do something with prop ... >> > } >> > object_property_iter_free(iter); >> >> I see my review hasn't been addressed, probably because it came late. >> Would you accept a follow-up patch to bring the iterator into line with >> existing ones? > > I'll write such a patch if you like, but i guess waiting for it to merge > till 2.6 is no big deal ?
Not even a little deal :)