Re: [xslt] Result Tree Fragments

[Daniel Veillard <veillard redhat com>]

On Wed, Apr 09, 2003 at 10:20:42AM -0600, Vakoc, Mark wrote:
> Well I haven't heard anything but this fixes my own testcase in bug 110023
> so I think it's safe to apply.

 yep, sorry didn't had time yet to look in details.

> > -----Original Message-----
> > From: Vakoc, Mark 
> > Sent: Tuesday, April 08, 2003 2:28 PM
> > To: 'xslt@gnome.org'
> > Subject: RE: [xslt] Result Tree Fragments
> > 
> > 
> > Perhaps an answer to my own question:
> > 
> > Daniel, can you verify that the attached patch for 
> > str:tokenize() is the
> > correct method for returning Result Tree Fragments from 
> > extension functions.

  Hum, right, creating a real document node is what libxslt
does now for RTFs, it's more unified, simplifies the template
processing for those, etc ...
  context->document becoming NULL is an annoyance but really
a side effect of the node-set() extension which wasn't in
the initial spec and which I didn't design for. It's annoying
to change some design at this point but it's really due to 
the huge deployment of this extension function.

> > This also fixes what appeared to be a memory leak in that 'str' and
> > 'delimiters' were never freed if the operation was successful.
> > 
> > I've also attached the patch to the bug.

  I applied the patch, 

    thanks !

Daniel

-- 
Daniel Veillard      | Red Hat Network https://rhn.redhat.com/
veillard@redhat.com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/