Re: [Boost-docs] XSL optimizations

Subject: Re: [Boost-docs] XSL optimizations
From: Steven Watanabe (watanabesj_at_[hidden])
Date: 2010-03-18 16:11:41

AMDG

Daniel James wrote:
> On 18 March 2010 04:06, Steven Watanabe <watanabesj_at_[hidden]> wrote:
>   
>> Do the changes seem reasonable?
>>     
>
> They look more than reasonable to me. Your syntax highlighter is a lot
> clearer than the old one, so even without the speed increase it'd be
> an improvement. I'll do some testing this weekend.
>   

Update.  The attached patch drops the time to about a minute and
a half (original 25-30 minutes).  After diffing the output against the
output of the original, I only saw two differences:
* Ids like id418384-bb are different.
* The original syntax highlighter doesn't mark up "double" correctly,
  probably because it contains "do."  Mine does handle "double."

> Do you know if xsltproc optimises tail recursion? If it doesn't it
> might be worth moving the search for identifier/non-identifier
> characters into separate templates so that the recursion is a bit
> shallower. That's a minor tweak though, I think the recursion was
> already pretty deep.
>   

I'll take a look at it, since the syntax highlighter is the
bottleneck again in my current patch.

In Christ,
Steven Watanabe