Re: Initial review of xslt with no limits patch - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: Initial review of xslt with no limits patch
Date
Msg-id 4C5B8361.5070400@dunslane.net
Whole thread Raw
In response to Re: Initial review of xslt with no limits patch  (Mike Fowler <mike@mlfowler.com>)
Responses Re: Initial review of xslt with no limits patch  (Pavel Stehule <pavel.stehule@gmail.com>)
List pgsql-hackers

On 08/05/2010 06:56 PM, Mike Fowler wrote:
>
> SELECT 
> xslt_process('<employee><name>cim</name><age>30</age><pay>400</pay></employee>'::text, 
> $$<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform" 
> version="1.0">
> <xsl:output method="xml" omit-xml-declaration="yes" indent="yes"/>
>
[snip]
> </xsl:stylesheet>$$::text, 'n1=v1,n2=v2,n3=v3,n4=v4,n5=v5'::text)
>
>

I haven't been paying attention to this, so sorry if this has been 
discussed before, but it just caught my eye. Why are we passing these 
params as a comma-separated list rather than as an array or as a 
variadic list of params? This looks rather ugly. What if you want to 
have a param that includes a comma?

cheers

andrew


pgsql-hackers by date:

Previous
From: Pavel Stehule
Date:
Subject: Re: Initial review of xslt with no limits patch
Next
From: pgsql-hackers@news.hub.org
Date:
Subject: pgsql-hackers@news.hub.org 81% OFF on Pfizer!