[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/7] qom: add object_new_propv / object_new_prop
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH 4/7] qom: add object_new_propv / object_new_proplist constructors |
Date: |
Thu, 30 Apr 2015 16:02:14 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Thu, Apr 30, 2015 at 05:00:28PM +0200, Paolo Bonzini wrote:
>
>
> On 30/04/2015 16:42, Daniel P. Berrange wrote:
> > + propname = va_arg(vargs, char *);
> > + while (propname != NULL) {
> > + const char *value = va_arg(vargs, char *);
> > +
> > + g_assert(value != NULL);
> > + object_property_parse(obj, value, propname, errp);
> > + if (*errp) {
> > + goto error;
> > + }
> > + propname = va_arg(vargs, char *);
> > + }
> > +
> > + object_property_add_child(container_get(object_get_root(), path),
> > + id, obj, errp);
> > + if (*errp) {
> > + goto error;
> > + }
> > +
> > + if (object_dynamic_cast(obj, TYPE_USER_CREATABLE)) {
> > + user_creatable_complete(obj, errp);
> > + if (*errp) {
> > + object_property_del(container_get(object_get_root(), path),
> > + id, &error_abort);
>
> Easier:
>
> object_unparent(obj);
>
> > + goto error;
> > + }
> > + }
> > +
> > + return obj;
>
> And another one: do we want to unref the object here, since we've added
> a stable reference via object_property_add_child?
>
> Do you prefer me too apply patches 1-2-3-5-6-7, minus the testcases, or
> do you simply want to resubmit?
I'll resubmit in a short while.
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
- [Qemu-devel] [PATCH 0/7] qom: misc fixes & enhancements to support TLS work, Daniel P. Berrange, 2015/04/30
- [Qemu-devel] [PATCH 2/7] qom: document user creatable object types in help text, Daniel P. Berrange, 2015/04/30
- [Qemu-devel] [PATCH 1/7] qom: fix typename of 'policy' enum property in hostmem obj, Daniel P. Berrange, 2015/04/30
- [Qemu-devel] [PATCH 5/7] qom: make enum string tables const-correct, Daniel P. Berrange, 2015/04/30
- [Qemu-devel] [PATCH 7/7] qom: don't pass string table to object_get_enum method, Daniel P. Berrange, 2015/04/30
- [Qemu-devel] [PATCH 4/7] qom: add object_new_propv / object_new_proplist constructors, Daniel P. Berrange, 2015/04/30
- [Qemu-devel] [PATCH 3/7] qom: create objects in two phases, Daniel P. Berrange, 2015/04/30
- [Qemu-devel] [PATCH 6/7] qom: add a object_property_add_enum helper method, Daniel P. Berrange, 2015/04/30