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