On 29 August 2010 22:21, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> 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>.
I'm referring to the rendered output. It isn't contained in <p> tags.
> What your changes seem to
> accomplish is to take the <simplelist> and <informaltable> constructs
> outside any <para>, but what is the point of that?
Well, those can be in their own <para> container too, although I don't
think that's an issue.
But I shalln't pursue it further if you think there's really no issue.
--
Thom Brown
Twitter: @darkixion
IRC (freenode): dark_ixion
Registered Linux user: #516935