Hi, > Im Auftrag von Daniel Veillard > Gesendet: Dienstag, 11. April 2006 19:09 > > On Tue, Apr 11, 2006 at 09:29:47AM -0700, Jerome Pesenti wrote:
[...] > > > The suggested fix is not acceptable as this > > > create the bug again. > > > You could try to force the reuse of the same > > > xmlXPathContextPtr for > > > all the compilations instead of reallocating a new > > > one each time, > > > that could work, but the current patch is not > > > acceptable. > > > > I'd like to submit a patch but I want to make sure > > that my solution is acceptable. > > okay, > > > 1/ Add a dummy xmlXPathContextPtr to the stylesheet > > structure and initialize it once with > > xmlXPathNewContext when creating the stylsheet > > structure. > > yes something like that except it should go into a compilation > context (which doesn't exist yet so in the style for now, yes). > > > 2/ Reuse that structure each time when compiling > > XPaths (is that safe?) > > It's safe because an XPath compilation can't call the XSLT back. > Just make sure you reinitialize the context fields if needed. > > > 3/ Free the structure when freeing the stylesheet > > structure. > > yes > > > I don't want to create a global to avoid having to > > lock it... > > definitely, that would be plain broken. > > thanks in advance :-) This is great. I'm trying to refactor some of the parts related to compilation. There's already a compilation context on my side. So Jerome, if you could just wait a couple of days: I'm debugging my changes and will be able to commit them in the next few days; the changes will be IFDEFed out until they are accepted as a sane way to go. Regards, Kasimier _______________________________________________ xslt mailing list, project page http://xmlsoft.org/XSLT/ [email protected] http://mail.gnome.org/mailman/listinfo/xslt
