[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH] qom: Rename object_new_with_props to object
From: |
Daniel P . Berrangé |
Subject: |
Re: [Qemu-devel] [RFC PATCH] qom: Rename object_new_with_props to object_new_child |
Date: |
Fri, 17 Aug 2018 10:48:19 +0100 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Fri, Aug 17, 2018 at 09:33:33AM +0200, Thomas Huth wrote:
> While adding the object_initialize_child() function, Paolo suggested
> to rename the similar object_new_with_props() function accordingly:
>
> http://marc.info/address@hidden
>
> This way it is more obvious that this function creates a new object
> as a child of another object.
I'd expect 'object_new_with_child' to be the same as 'object_new',
but with only 'parent' & 'id' args added, which isn't the case here.
If we want the full & consistent design then we should have
object_new(typename)
object_new_with_child(typename, parent, id)
object_new_props(typename, ...)
object_new_propv(typename, va_arg props)
object_new_with_child_props(typename, parent, id, ...)
object_new_with_child_propv(typename, parent, id, va_arg props)
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|