Bill Mitchell wrote:
Hello Senaka,
No, existing axiom_node_detach calls do not need to change. They can all
stay the same, and in general they need to stay the same in order to receive
the fix that the detached tree declares all the namespaces that it
references.
One of the concerns raised in the discussion, and obvious during testing, is
that it is silly to redeclare the namespaces during a detach when the intent
of the detach is to free the tree. So, in the internal routines where the
code detached and freed the tree, I changed them to free the tree directly
without a separate detach. I believe it was Samisa who agreed with my
suggestion that, when free is passed a tree that is still attached, a
situation not currently handled nor diagnosed as an error, free could go
ahead and detach the tree before freeing it.
Yes I did. And also, I went through the patch you have attached to issue
675. The fix looks OK, and I think, we should ship that for 1.3.
Bill, thank you for fixing this.
Regards,
Samisa...
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
