libxmlplusplus-general

[Libxmlplusplus-general] Re: [gnomemm] ANNOUNCE: libxml++ is now sane.

[Paul D. <pa...@li...>]

>libxml++ was a mess in the past but it has been overhauled recently. The
>new 0.16.0 release is now organised and usable. I invite you to try it 
>and help us to improve it by submitting patches (to the sourceforge
>patch manager).

if you're going to try to improve it, i would pass on one
recommendation. this API:

const Node * add_child (const std::string &name) 
      Node * add_child (Node *node) 

has caused dozens of stupid bugs in my code, because it doesn't make
it clear whether the added child is copied or not (its not).

ardour has had its own copy of libxml++ for some time, and we have
been unable to get the author to accept any of our patches. we changed
the API to:

  XMLNode *add_child(const string &);
  XMLNode *add_child_copy(const XMLNode&);
  void     add_child_nocopy (XMLNode&);

--p

Re: [Libxmlplusplus-general] Re: [gnomemm] ANNOUNCE: libxml++ is now sane.

[Christophe de V. <cde...@ne...>]

On Mercredi 20 Novembre 2002 22:57, Paul Davis wrote:
>
> const Node * add_child (const std::string &name)
>       Node * add_child (Node *node)
>
> has caused dozens of stupid bugs in my code, because it doesn't make
> it clear whether the added child is copied or not (its not).
>

It's one thing I wanted to improve. For me, if not not standalone, a node 
should have all it's life cycle managed by its owner : creation and deletion.
We'll try to do somethind about this soon, probably partly based on your patch 
if your could post it again in the patch tracker.

Regards,

Christophe