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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Attachment: pgpnEi60q668z.pgp
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]