Re: Initial review of xslt with no limits patch

Tom Lane wrote:
> Bruce Momjian <bruce@momjian.us> writes:
> > Tom Lane wrote:
> >> I think we have a few TODO items here:
> >> 
> >> * Invent ... and document ... an API that permits safe assembly of a
> >> parameter list from non-constant (and perhaps untrustworthy) values.
> >> 
> >> * Fix xslt_process' failure to report (some?) errors detected by libxslt.
> >> 
> >> * Move the functionality to a less deprecated place.
> >> 
> >> None of these are within the scope of the current patch though.
> 
> > Should any of these be added to our TODO list under XML?
> 
> Yes, all of them, since nothing's been done about any of 'em ...

OK, TODO items added:
Move XSLT from contrib/xml2 to a more reasonable location    *
http://archives.postgresql.org/pgsql-hackers/2010-08/msg00539.phpReport errors returned by the XSLT library    *
http://archives.postgresql.org/pgsql-hackers/2010-08/msg00562.phpImprove the XSLT parameter passing API    *
http://archives.postgresql.org/pgsql-hackers/2010-08/msg00416.php
 

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +