Re: [xslt] xsl:include thing
- From: Daniel Veillard <veillard redhat com>
- To: xslt gnome org
- Subject: Re: [xslt] xsl:include thing
- Date: Tue, 14 May 2002 17:51:08 -0400
On Tue, May 14, 2002 at 07:32:53PM +0200, Igor Zlatkovic wrote:
> Hello there.
> Here is the patch which removes the crash caused by an invalid file
> referenced in xsl:include element.
> What happened is the following:
> * xsltParseStylesheetProcess: Modified not to free the stylesheet which it
> receives as a parameter.
> * xsltParseStylesheetDoc: Modified to free the stylesheet it now carries the
> responsibility for
> * xsltParseStylesheetInclude: Added an int return value to indicate failure.
> * xsltParseStylesheetImport: Added an int return value to indicate failure,
> fixed memory leak incase of error
> * xsltParseStylesheetTop: Added check for these return values and indicated
> an errorneous XSL document accordingly.
looks fine to me,
> The damn diff obviously also contains few differences which have nothing to
> do with the code. My xemacs has a different indenting scheme and a different
> tab size than what Daniel used to make this file. I am still not that
> familiar with the tool to be able to teach it new identing :-) I am
I use VI , more precisely vim with a tab setting set to 8
> Apart from that, if the patch is okay for everyone, I shall apply it.
Daniel Veillard | Red Hat Network https://rhn.redhat.com/
email@example.com | libxml GNOME XML XSLT toolkit http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/
] [Thread Prev