Subject: Re: exsl:node-set in xalan
From: David Tolpin <dvd@xxxxxxxxxxxxxx>
Date: Sat, 17 Jan 2004 16:23:15 +0400 (AMT)
|
> I haven't had a chance to analyze this yet (and I may not be able to
> this week), but one quick point is that wherever the Saxon performance
> problem is, it's not actually in exsl:node-set(). That's because all
> that exsl:node-set() does is to flip a bit saying that the tree can be
> used as a normal document.
>
> The problem (or opportunity for improvement!) is much more likely to be
> in optimization of copying operations on trees.
>
> An observation, on line 65 of incelim.xsl
>
> <xsl:copy-of select="exsl:node-set($rng)"/>
>
> there is no need for a call on exsl:node-set() here. You can copy an RTF
> directly.
Michael,
I've followed your suggestion and fixed the typo -- it was an oversight on my
side instead. Note, though, that this operation is only executed once
per the stylesheet, and as you said, the only thing node-set() does is to
flip a bit.
The execution time didn't change at all, and it should not change.
David
XSL-List info and archive: http://www.mulberrytech.com/xsl/xsl-list
| Current Thread |
- Re: exsl:node-set in xalan, (continued)
- Kevin Jones - Fri, 16 Jan 2004 22:10:58 -0500 (EST)
- David Tolpin - Sat, 17 Jan 2004 00:49:57 -0500 (EST)
- David Tolpin - Sat, 17 Jan 2004 01:10:29 -0500 (EST)
- Michael Kay - Sat, 17 Jan 2004 06:36:59 -0500 (EST)
- David Tolpin - Sat, 17 Jan 2004 07:24:15 -0500 (EST) <=
- David Tolpin - Sat, 17 Jan 2004 09:45:22 -0500 (EST)
- Julian Reschke - Sat, 17 Jan 2004 10:04:07 -0500 (EST)
- Michael Kay - Sat, 17 Jan 2004 11:28:30 -0500 (EST)
- David Tolpin - Sat, 17 Jan 2004 12:34:01 -0500 (EST)
|
|