Tom Lane wrote:
> Alvaro Herrera <alvhe...@commandprompt.com> writes:
> > Still, my original proposal was \df[antw][S+].  The extra brackets are
> > obviously redundant, but if we're about providing cues, this is a good
> > cue IMO.  It allows the [S+] to match the other lines.
> 
> I'm for that too.  Bruce was complaining that it'd make the column
> wider, but I don't find that a compelling objection.

OK, I switch to brackets and added separate brackets around the "antw"
options.  I changed "PATTERN" to "PATRN" so I would not have to shift
over _every_ other \? option to align properly;  patch attached and
applied.

-- 
  Bruce Momjian  <br...@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +
Index: src/bin/psql/help.c
===================================================================
RCS file: /cvsroot/pgsql/src/bin/psql/help.c,v
retrieving revision 1.148
diff -c -c -r1.148 help.c
*** src/bin/psql/help.c	21 Apr 2009 17:23:05 -0000	1.148
--- src/bin/psql/help.c	22 Apr 2009 14:58:16 -0000
***************
*** 205,211 ****
  	fprintf(output, _("  \\des[+] [PATTERN]      list foreign servers\n"));
  	fprintf(output, _("  \\deu[+] [PATTERN]      list user mappings\n"));
  	fprintf(output, _("  \\dew[+] [PATTERN]      list foreign-data wrappers\n"));
! 	fprintf(output, _("  \\df[antwS+] [PATTERN]  list (only agg/normal/trigger/window) functions\n"));
  	fprintf(output, _("  \\dF[+]  [PATTERN]      list text search configurations\n"));
  	fprintf(output, _("  \\dFd[+] [PATTERN]      list text search dictionaries\n"));
  	fprintf(output, _("  \\dFp[+] [PATTERN]      list text search parsers\n"));
--- 205,211 ----
  	fprintf(output, _("  \\des[+] [PATTERN]      list foreign servers\n"));
  	fprintf(output, _("  \\deu[+] [PATTERN]      list user mappings\n"));
  	fprintf(output, _("  \\dew[+] [PATTERN]      list foreign-data wrappers\n"));
! 	fprintf(output, _("  \\df[antw][S+] [PATRN]  list [only agg/normal/trigger/window] functions\n"));
  	fprintf(output, _("  \\dF[+]  [PATTERN]      list text search configurations\n"));
  	fprintf(output, _("  \\dFd[+] [PATTERN]      list text search dictionaries\n"));
  	fprintf(output, _("  \\dFp[+] [PATTERN]      list text search parsers\n"));
-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to