[Petal] [Patch]Problem with last patch

Jean-Michel Hiver jhiver at mkdoc.com
Fri Jul 25 11:26:28 BST 2003


> The last patch for the tal-style repeat counter had a problem with nested 
> loops. This explains a large part of the surprising 5x speedup - if it's too 
> good to be true, it's usually not true.
> 
> This new patch fixes that. However for some reason neither patch plays well 
> with Petal's $hash->get() method which does some sort of caching - this 
> explains another big part of the speed up.
> 
> It is however safe to use if you're using Petal::CodePerl as this does no 
> caching of Petales expressions.
> 
> The speedup is 50% for my dictionary template, which does a lot of looping. 
> It's still not too bad I suppose and it means that for my dictionary program 
> Petal is now about 30% faster than Template::Toolkit

If this option is relevant only when using Petal::CodePerl, should this
really be patched into Petal? It's a shame because I like the idea of
having this repeat object.

I'll take a look into it :)

Cheers,
-- 
Building a better web - http://www.mkdoc.com/
---------------------------------------------
Jean-Michel Hiver
jhiver at mkdoc.com  - +44 (0)114 255 8097
Homepage: http://www.webmatrix.net/


More information about the Petal mailing list