On 12/19/2017 02:45 AM, Peter Xu wrote: > We can simplify object_property_get_str() using the new > qobject_get_try_str(). > > Reviewed-by: Fam Zheng <f...@redhat.com> > Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com> > Signed-off-by: Peter Xu <pet...@redhat.com> > --- > qom/object.c | 9 +++------ > 1 file changed, 3 insertions(+), 6 deletions(-)
Reviewed-by: Eric Blake <ebl...@redhat.com> I'm not opposed to your patch split (particularly since it makes backports easier if it just needs the new function and then your later uses of the new function, without touching existing uses); but I might have merged this with the previous patch so that the new API has a client right away, proving why the new API is worthwhile as part of its introduction. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature