Re: Function parameter names

On Sun, 23 Nov 2003, Tom Lane wrote:

> Actually I'd suggest text[], as there is no good reason to pad the
> array entries to a fixed length.

I've implemented this part now and it stores the paremeter names in the 
pg_proc table as a text[] field.

However, in the parser I use IDENT to get the parameter names and already
in the lexer the IDENT tokens are truncated to length NAMEDATALEN.

So I've got 3 options:
1) Leave it as is now where the system table allows any length   but the parser only lets you insert "short"
identifiers.
2) Change the type to name[]
3) Change the parser to accept identifiers of any length and add   the length check in a later phase for the
identifiersthat need   to be shorter.
 

Any opinions or should I just make a choice myself?

-- 
/Dennis