From: Bruce Momjian <bruce@momjian.us>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: PostgreSQL-documentation <pgsql-docs@postgresql.org>
Subject: Re: [COMMITTERS] pgsql: remove tags.
Date: Mon, 7 Feb 2011 10:59:53 -0500 (EST)
Message-ID: <201102071559.p17Fxrb07886@momjian.us> (raw)
In-Reply-To: <15595.1297091285@sss.pgh.pa.us>
Tom Lane wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
> > Which way did we more commonly do it before you applied this patch?
>
> We don't have a standard for this, and an undocumented patch applied
> without any discussion doesn't create one. It's hopeless to imagine
> that you'll ever achieve any uniformity that way. It won't last long
> if you do, since you're outnumbered by committers who won't be following
> whatever you think the convention is.
Well, the problem is that we then have paragraphs where a number is
fixed width, and in the next paragraph it isn't; this is particuarly
true in the config.sgml file where we show defaults. Seems worth trying
to make that consistent. I am happy to do whatever we agree on --- I
don't have an opinion.
> I'm not even sure why you're trying --- I don't think it even makes
> sense to try to have a standard about this. I can easily imagine that
> integer constants might read better with <literal> in some contexts
> and better without in others.
Yes, I reviewed all the places and left <literal> where it made sense.
--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-docs@postgresql.org
Cc: bruce@momjian.us, tgl@sss.pgh.pa.us, robertmhaas@gmail.com
Subject: Re: [COMMITTERS] pgsql: remove tags.
In-Reply-To: <201102071559.p17Fxrb07886@momjian.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox