Re: Initial review of xslt with no limits patch

Поиск
Список
Период
Сортировка
От Pavel Stehule
Тема Re: Initial review of xslt with no limits patch
Дата
Msg-id AANLkTinCJixED+KtgBJgv7H0hC6PiVpx4Ec4HxsjhVa=@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Initial review of xslt with no limits patch  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: Initial review of xslt with no limits patch  (Mike Fowler <mike@mlfowler.com>)
Список pgsql-hackers
Hello

attached updated patch with regression test



2010/8/6 Tom Lane <tgl@sss.pgh.pa.us>:
> Mike Fowler <mike@mlfowler.com> writes:
>> SELECT
>> xslt_process( ... , ... ,
>>              'n1=v1,n2=v2,n3=v3,n4=v4,n5=v5'::text)
>
> produces
>
>> <samples>
>>    <sample>v1</sample>
>>    <sample>v2</sample>
>>    <sample>v3</sample>
>>    <sample>v4</sample>
>>    <sample>v5</sample>
>> </samples>
>
>> Sadly I get the following in both versions:
>
>> <samples>
>>    <sample/>
>>    <sample/>
>>    <sample/>
>>    <sample/>
>>    <sample/>
>> </samples>
>
> Some examination of
> http://www.xmlsoft.org/XSLT/tutorial/libxslttutorial.html
> suggests that the parameter values need to be single-quoted,
> and indeed when I change the last part of your example to
>
>        'n1=''v1'',n2=''v2'',n3=''v3'',n4=''v4'',n5=''v5'''::text);
>
> I get
>
>     xslt_process
> -----------------------
>  <samples>            +
>   <sample>v1</sample>+
>   <sample>v2</sample>+
>   <sample>v3</sample>+
>   <sample>v4</sample>+
>   <sample>v5</sample>+
>  </samples>           +
>
> (1 row)
>
> So this seems to be a documentation problem more than a code problem.
>
> (It's a bit distressing to notice that the regression tests for the
> module fail to exercise 3-parameter xslt_process at all, though.)
>

??? I don't see it

Regards

Pavel Stehule

>                        regards, tom lane
>

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Robert Haas
Дата:
Сообщение: Re: refactoring comment.c
Следующее
От: Andrew Dunstan
Дата:
Сообщение: Re: Initial review of xslt with no limits patch