[libvirt] [libvirt-glib] API to get/set custom metadata from/to domain config

Christophe Fergeau cfergeau at redhat.com
Tue Jan 24 09:18:59 UTC 2012


On Mon, Jan 23, 2012 at 11:31:57PM +0200, Zeeshan Ali (Khattak) wrote:
> On Mon, Jan 23, 2012 at 7:19 PM, Christophe Fergeau <cfergeau at redhat.com> wrote:
> > Based on a patch from Zeeshan Ali (Khattak) <zeeshanak at gnome.org>
> 
> Looks good! I'll test it later

I've only compile-tested it, if it doesn't work, don't waste time on it,
just tell me to test my stuff

> 
> > +    gvir_config_object_set_namespace(custom_xml, ns, ns_uri, TRUE);
> 
> You meant to pass 'FALSE' in the last arg?

Dunno, depends on what we decide to do :) I prefer FALSE, your initial
patch uses TRUE so I kept that for now.

> 
> > +    g_return_val_if_fail(GVIR_CONFIG_IS_OBJECT(object), FALSE);
> > +    g_return_val_if_fail(ns != NULL, FALSE);
> > +    g_return_val_if_fail(ns_uri != NULL, FALSE);
> > +
> > +    namespace = xmlNewNs(object->priv->node,
> > +                         (xmlChar *)ns_uri, (xmlChar *)ns);
> > +    if (namespace == NULL)
> > +        return FALSE;
> > +    if (recursive) {
> > +        set_namespace_on_tree(object->priv->node, namespace);
> > +    } else {
> > +        xmlSetNs(object->priv->node, namespace);
> > +    }
> 
> You don't need the curly braces here.

I know, but I like them :) I don't mind dropping them

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120124/6c25c2dc/attachment-0001.sig>


More information about the libvir-list mailing list