Received: from maia.hub.org (maia-5.hub.org [200.46.204.29])
	by mail.postgresql.org (Postfix) with ESMTP id 4F98C632BCA
	for <pgsql-docs-postgresql.org@mail.postgresql.org>;
	Sun, 29 Aug 2010 18:21:41 -0300 (ADT)
Received: from mail.postgresql.org ([200.46.204.86])
	by maia.hub.org (mx1.hub.org [200.46.204.29]) (amavisd-maia, port 10024)
	with ESMTP id 65582-03 for
	<pgsql-docs-postgresql.org@mail.postgresql.org>;
	Sun, 29 Aug 2010 21:21:34 +0000 (UTC)
X-Greylist: from auto-whitelisted by SQLgrey-1.7.6
Received: from sss.pgh.pa.us (sss.pgh.pa.us [66.207.139.130])
	by mail.postgresql.org (Postfix) with ESMTP id F1828632762
	for <pgsql-docs@postgresql.org>; Sun, 29 Aug 2010 18:21:33 -0300 (ADT)
Received: from sss2.sss.pgh.pa.us (tgl@localhost [127.0.0.1])
	by sss.pgh.pa.us (8.14.2/8.14.2) with ESMTP id o7TLLVMI028205;
	Sun, 29 Aug 2010 17:21:31 -0400 (EDT)
To: Thom Brown <thom@linux.com>
cc: pgsql-docs <pgsql-docs@postgresql.org>
Subject: Re: Doc fixes and improvements 
In-reply-to: <AANLkTinDvXCg3KKrif-egktfJMREU3d=uRGsRHQ6YKNL@mail.gmail.com> 
References: <AANLkTimPYapJDx1N7LjvL5vPmxuQeHNnTT2vLkc2+04i@mail.gmail.com>
	<AANLkTi=DxLhia2teAS=oGdDrXBtNFdP2bjRP4QczWVsF@mail.gmail.com>
	<AANLkTikrmCYo905XbCMsS-kkUDOQK+2ncjUEkYtLM12n@mail.gmail.com>
	<AANLkTik=sM=jPr2+VMGZNsfjuZtTDR2R0ye5mQ8qn-QX@mail.gmail.com>
	<27296.1283113864@sss.pgh.pa.us>
	<AANLkTinDvXCg3KKrif-egktfJMREU3d=uRGsRHQ6YKNL@mail.gmail.com>
Comments: In-reply-to Thom Brown <thom@linux.com>
	message dated "Sun, 29 Aug 2010 21:42:31 +0100"
Date: Sun, 29 Aug 2010 17:21:31 -0400
Message-ID: <28204.1283116891@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
X-Virus-Scanned: Maia Mailguard 1.0.1
X-Spam-Status: No, hits=-1.91 tagged_above=-5 required=5 tests=BAYES_00=-1.9,
	T_RP_MATCHES_RCVD=-0.01
X-Spam-Level: 
X-Archive-Number: 201008/142
X-Sequence-Number: 5966

Thom Brown <thom@linux.com> writes:
> On 29 August 2010 21:31, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> Ummm ... this seems to be almost entirely pointless whitespace changes.
>> Sure you attached the right diff?

> Yes, the whitespace changes you're seeing are because I closed the
> <para> tag early, then re-opened it again for the next paragraph,
> which meant I reduced the indent for the content between them.  The
> paragraph is also closed again after, rather than containing
> everything after it, so I removed the excess closing paragraph tag and
> reduce the indentation of the affected sections.

> This is because of what is rendered on:
> http://www.postgresql.org/docs/9.0/static/functions-logical.html

> The sentence "SQL uses a three-valued boolean..." isn't contained in a
> paragraph at all and should be.  Otherwise it's free text in a div.

Uh ... it is too enclosed in a <para>.  What your changes seem to
accomplish is to take the <simplelist> and <informaltable> constructs
outside any <para>, but what is the point of that?  I don't see anything
obviously wrong with the rendering on the page you cite.

			regards, tom lane