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

Reply via email to