Re: variadic function support
"Pavel Stehule" <[email protected]>
| Newsgroups | gmane.comp.db.postgresql.devel.patches |
|---|---|
| Message-ID | <[email protected]> |
2008/6/25 Tom Lane <[email protected]>: > "Pavel Stehule" <[email protected]> writes: >>> Tom Lane wrote: >>>> Your point about the syntax is good though. It would be better if >>>> the syntax were like >>>> create function foo (a text, variadic b int[]) >>>> or maybe even better >>>> create function foo (a text, variadic b int) > >> I don't see problem with your syntax. It well block combination OUT >> and VARIADIC parameter - my one request, variadic parameter have to be >> array. > > Well, we should certainly store the parameter type as an array in > proargtypes, because that makes this feature transparent to all the > PLs. However, it doesn't follow that the CREATE FUNCTION syntax > has to specify the array type rather than the element type. I think > the Java precedent might be good reason to go with using the element > type in the function declaration. There is only one break - psql functions description. It needs publishing get_element_type function and ofcourse all managers need update. regards Pavel > > regards, tom lane > -- Sent via pgsql-patches mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-patches