Re: Initial review of xslt with no limits patch

Bruce Momjian <bruce@momjian.us>

From: Bruce Momjian <bruce@momjian.us>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Mike Fowler <mike@mlfowler.com>, Pavel Stehule <pavel.stehule@gmail.com>, pgsql-hackers@postgresql.org
Date: 2011-02-18T22:59:38Z
Lists: pgsql-hackers
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.php 
	
	Report errors returned by the XSLT library
	
	    * http://archives.postgresql.org/pgsql-hackers/2010-08/msg00562.php 
	
	Improve 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. +