Re: [HACKERS] Improved \df(+) in psql + backward-compatibility - Mailing list pgsql-patches

From Tom Lane
Subject Re: [HACKERS] Improved \df(+) in psql + backward-compatibility
Date
Msg-id 10046.1125328409@sss.pgh.pa.us
Whole thread Raw
In response to Improved \df(+) in psql + backward-compatibility  (David Fetter <david@fetter.org>)
Responses Re: [HACKERS] Improved \df(+) in psql + backward-compatibility  (David Fetter <david@fetter.org>)
Re: [HACKERS] Improved \df(+) in psql + backward-compatibility  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-patches
David Fetter <david@fetter.org> writes:
> I've noticed that \df doesn't do quite what it might when a function
> is created with named input parameters.  Please find enclosed a patch
> against CVS TIP that does this better.

Meanwhile, getting back to the actual merits of the patch ... this is
not right yet, because it will do the wrong thing when there are OUT
parameters.  (The proargnames array includes both IN and OUT params,
and you can't assume that proargnames and proargtypes have corresponding
subscripts.)  It would probably be a good idea to discuss what display
we want for a function with OUT parameters, anyway.  The strict columnar
representation that \df currently uses doesn't scale very well :-(

            regards, tom lane

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: be-secure.c patch
Next
From: David Fetter
Date:
Subject: Re: [HACKERS] Improved \df(+) in psql + backward-compatibility