On Sat, Jul 15, 2006 at 04:42:50PM -0700, David Fetter wrote: > Folks, > > This patch makes psql's \df display functions with the names of > parameters and OUT/INOUT if appropriate. Should there be a > regression test for this? A doc patch?
As this doesn't change any documented behavior, it doesn't look like there is space for a doc patch. Anyhow, please find enclosed the context-style diff. Sorry about the mix-up earlier :) Cheers, D -- David Fetter <[EMAIL PROTECTED]> http://fetter.org/ phone: +1 415 235 3778 AIM: dfetter666 Skype: davidfetter Remember to vote!
Index: src/bin/psql/describe.c =================================================================== RCS file: /projects/cvsroot/pgsql/src/bin/psql/describe.c,v retrieving revision 1.140 diff -c -r1.140 describe.c *** src/bin/psql/describe.c 14 Jun 2006 16:49:02 -0000 1.140 --- src/bin/psql/describe.c 16 Jul 2006 06:10:10 -0000 *************** *** 170,177 **** "SELECT n.nspname as \"%s\",\n" " p.proname as \"%s\",\n" " CASE WHEN p.proretset THEN 'setof ' ELSE '' END ||\n" ! " pg_catalog.format_type(p.prorettype, NULL) as \"%s\",\n" ! " pg_catalog.oidvectortypes(p.proargtypes) as \"%s\"", _("Schema"), _("Name"), _("Result data type"), _("Argument data types")); --- 170,198 ---- "SELECT n.nspname as \"%s\",\n" " p.proname as \"%s\",\n" " CASE WHEN p.proretset THEN 'setof ' ELSE '' END ||\n" ! " pg_catalog.format_type(p.prorettype, NULL) as \"%s\",\n" ! " CASE WHEN proallargtypes IS NOT NULL THEN\n" ! " array_to_string(ARRAY(\n" ! " SELECT\n" ! " CASE\n" ! " WHEN p.proargmodes[s.i] = 'i' THEN ''\n" ! " WHEN p.proargmodes[s.i] = 'o' THEN ' OUT '\n" ! " WHEN p.proargmodes[s.i] = 'b' THEN ' INOUT '\n" ! " END ||\n" ! " COALESCE(p.proargnames[s.i] || ' ','') ||\n" ! " format_type(p.proallargtypes[s.i],-1)\n" ! " FROM\n" ! " generate_series(1,array_upper(p.proallargtypes,1)) AS s(i)\n" ! " ), ',')\n" ! " ELSE\n" ! " array_to_string(ARRAY(\n" ! " SELECT\n" ! " COALESCE(p.proargnames[s.i+1] || ' ', '') ||\n" ! " format_type(p.proargtypes[s.i],-1)\n" ! " FROM\n" ! " generate_series(0,array_upper(p.proargtypes,1)) AS s(i)\n" ! " ), ',')\n" ! " END AS \"%s\"", _("Schema"), _("Name"), _("Result data type"), _("Argument data types"));
---------------------------(end of broadcast)--------------------------- TIP 1: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to [EMAIL PROTECTED] so that your message can get through to the mailing list cleanly