Alvaro Herrera wrote:
> Kevin Grittner wrote:
> > Bruce Momjian <br...@momjian.us> wrote:
> >  
> > > \df[S+] [PATTERN]      list functions
> > > \df[antwS+] [PATTERN]  list only agg/normal/trigger/window functions
> >  
> > Shouldn't that second line have some curly braces?  Like maybe:
> >  
> > \df{antw}[S+] [PATTERN]  list only agg/normal/trigger/window functions
> >  
> > Technically, it should probably be even more verbose, but this might
> > be adequate.
> 
> I suggested this to Bruce over IM:
> 
> \df[antw][S+]     list [only agg/normal/trigger/window] functions
> 
> (one line only, removing the second redundant line).  This seems
> clea[nr]er to me.  Bruce says it would confuse users.  But really, if
> you can understand the [S+] stuff you should be able to get the [antw]
> stuff too, right?

I merged the entries into one line:

        \df[antwS+] [PATTERN]  list (only agg/normal/trigger/window) functions

I didn't feel I had room to do [][] like Alvaro suggested.  I also went
with parntheses in the description.  Are brackets better there?

-- 
  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: doc/src/sgml/ref/psql-ref.sgml
===================================================================
RCS file: /cvsroot/pgsql/doc/src/sgml/ref/psql-ref.sgml,v
retrieving revision 1.224
diff -c -c -r1.224 psql-ref.sgml
*** doc/src/sgml/ref/psql-ref.sgml	21 Apr 2009 15:49:06 -0000	1.224
--- doc/src/sgml/ref/psql-ref.sgml	21 Apr 2009 17:21:56 -0000
***************
*** 1038,1045 ****
  
  
        <varlistentry>
!         <term><literal>\df[S+] [ <replaceable class="parameter">pattern</replaceable> ]</literal></term>
!         <term><literal>\df[antw][S+] [ <replaceable class="parameter">pattern</replaceable> ]</literal></term>
  
          <listitem>
          <para>
--- 1038,1044 ----
  
  
        <varlistentry>
!         <term><literal>\df[antwS+] [ <replaceable class="parameter">pattern</replaceable> ]</literal></term>
  
          <listitem>
          <para>
Index: src/bin/psql/help.c
===================================================================
RCS file: /cvsroot/pgsql/src/bin/psql/help.c,v
retrieving revision 1.147
diff -c -c -r1.147 help.c
*** src/bin/psql/help.c	21 Apr 2009 15:49:06 -0000	1.147
--- src/bin/psql/help.c	21 Apr 2009 17:21:57 -0000
***************
*** 205,212 ****
  	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[S+] [PATTERN]      list functions\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[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"));
-- 
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