On Wed, Jan 12, 2011 at 9:23 AM, Stephen Frost <sfr...@snowman.net> wrote: > Minor enhancement, but a valuable one imv. Hopefully there aren't any > issues with it. :)
1. Why %o? That's not obviously mnemonic. Perhaps %U? 2. It won't be clear to people reading this what the difference is between %u and this. You probably need to reword the documentation for the existing option as well as documenting the new one. 3. Please attach the patch rather than including it inline, if possible. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
