From: | "Serguei Mokhov" <mokhov(at)cs(dot)concordia(dot)ca> |
---|---|
To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "Manfred Koizar" <mkoi-pg(at)aon(dot)at>, "PostgreSQL Hackers" <pgsql-hackers(at)postgresql(dot)org>, "Peter Eisentraut" <peter_e(at)gmx(dot)net>, "PostgreSQL Patches" <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: pg_dump.options.diff |
Date: | 2003-01-02 22:51:55 |
Message-ID: | 005701c2b2b1$8e923fe0$0301a8c0@gunnymede.lan |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
----- Original Message -----
From: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Sent: January 02, 2003 3:20 PM
> > #if defined(HAVE_GETOPT_LONG)
> > #define xo(long,short,desc) printf("%s %s\n", long, desc)
> > #else
> > #define xo(long,short,desc) printf("%s %s\n", short, desc)
> > #endif
>
> > seems relatively generic, so it could be used by more than one tool.
>
> But there's no good place to put it. I'd say just stick it into each
> tool; it's no worse than repeating the existence of a "usage()"
> subroutine in each tool.
It ended up being in dumputils.h
Attached a patch for pg_dump(all) and pg_restore eliminating dupes
in option descriptions as per above xo "technology".
Please review and apply if there are no too many objections.
thank you,
-s
Attachment | Content-Type | Size |
---|---|---|
pg_dump.options2.diff | application/octet-stream | 27.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2003-01-02 23:07:52 | Re: PostgreSQL Password Cracker |
Previous Message | Bruce Momjian | 2003-01-02 22:31:10 | Re: Bug in pg_get_constraintdef (for deferrable constraints) |
From | Date | Subject | |
---|---|---|---|
Next Message | Nic Ferrier | 2003-01-02 22:57:54 | Re: [PATCHES] Fwd: Patch for streaming resultsets |
Previous Message | Tom Lane | 2003-01-02 20:20:35 | Re: pg_dump.options.diff |