Re: xslcache extension

On 28.5.2009 12:57 Uhr, Rob Richards wrote: > Alexey Zakhlestin wrote: >> On Thu, May 28, 2009 at 2:02 PM, Antony Dovgal <tony@daylessday.org> >> wrote: >> >>> On 28.05.2009 13:48, Alexey Zakhlestin wrote: >>> >>>> Not yet. >>>> It is unmaintained for several years, so, fork shouldn't be a problem. >>>> >>> Yes, I know. >>> >>> Btw, you might want to look at this patch: >>> http://dev.daylessday.org/diff/xsltcache.diff >>> (sent upstream 1.5 year ago, still not applied) >>> >> >> I know about this patch. Just recently recommended it to @chregu :-) >> >> p.s. waiting for reply from Jacob Harris (from nytimes) >> >> > Someone had asked me last week at php|tek about xsl caching and we had > tossed around the idea of possibly integrating either this or some of > the fastxsl code into ext/xsl. There are some issues doing this, which > were always the hold up moving on this, so the thought was to make it a > compile time option disabled by default. Wanted to float the idea to > Christian first though. As long as it's disabled by default and can only be enabled by a compile time option or PHP_INI_SYSTEM almost anything is fine for me. At least the nytimes code has some (self-admitted) flawes [1] which make it somehow "dangerous" if it would be used by people not knowing what they do, just thinking they get faster XSLT :) But of course the approach is very very useful for specific cases. chregu [1] http://code.nytimes.com/projects/xslcache/wiki/FutureWork -- Liip AG // Feldstrasse 133 // CH-8004 Zurich Tel +41 43 500 39 81 // Mobile +41 76 561 88 60 www.liip.ch // blog.liip.ch // GnuPG 0x5CE1DECB