Re: [xslt] Patch to implement date:sum(node-set)



On Fri, 2004-09-24 at 14:01, Daniel Veillard wrote:

>   Hum ... I tried ... and hit the fact that there is already
> a date sum function provided in recent versions.
>   http://cvs.gnome.org/viewcvs/libxslt/libexslt/date.c?rev=1.30&view=auto
> The version in CVs obviously got more testing since it was added in
> 1.1.9 last month (EXSLT date:sum added (Joel Reed))
>    http://xmlsoft.org/XSLT/news.html
> So I think I will keep the existing implementation, which seems
> to work just as well except it generate one difference when running
> the regression tests:
> 
> paphio:~/XSLT/tests/exslt/date -> ../../../xsltproc/xsltproc sum.2.xsl sum.2.xmlxmlXPathCompiledEval: evaluation failed
> runtime error: file sum.2.xsl line 12 element value-of
> xsltValueOf: text copy failed
> xmlXPathCompiledEval: evaluation failed
> [...]

Daniel,

Doh!  I guess I should have looked in CVS first.  Personally, I prefer
my implementation because it reuses more of existing code.  But of
course you are the maintainter so you should trust your own judgment.

It would be nice to if you could get the exslt.org page updated. 
Currently, on http://exslt.org/date/functions/sum/ it says that no XSLT
processors implement date:sum natively.

Derek


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]