[libvirt] [libvirt-glib 13/23] Remove GVirConfigObject::docHandle

Christophe Fergeau cfergeau at redhat.com
Tue Oct 18 12:35:03 UTC 2011


On Tue, Oct 18, 2011 at 12:44:27PM +0100, Daniel P. Berrange wrote:
> > -    if (priv->docHandle)
> > -        xmlFreeDoc(priv->docHandle);
> > +    /* FIXME: all objects describing a given XML document will share the
> > +     * same document so we can't destroy it here like this, we need some
> > +     * refcounting to know when to destroy it.
> > +     */
> > +    if (priv->node)
> > +        xmlFreeDoc(priv->node->doc);
> 
> This could be reason for creating a trivial GObject to wrap
> the  xmlDocPtr instance, gaining ref counting.

Yes, that's the plan, and that's also why I mentioned this gupnp object in
one of the previous commits.

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/20111018/9b7fd166/attachment-0001.sig>


More information about the libvir-list mailing list