public inbox for [email protected]  

help / color / mirror / Atom feed

From: Brendan Jurd <[email protected]>
To: Robert Haas <[email protected]>
Cc: [email protected]
Subject: Re: Add link to current_schemas in config.sgml
Date: Tue, 14 Jun 2011 02:55:19 +1000
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>

On 14 June 2011 02:39, Robert Haas <[email protected]> wrote:
> On Tue, May 31, 2011 at 2:27 AM, Brendan Jurd <[email protected]> wrote:
>> This is just a quick docs patch to add a link to the mention of the
>> current_schemas function from 18.10.1. Statement Behavior.
>
> Your patch got mangled by my email client, but I committed what I
> believe to be the same change.

Thanks Robert, but I think you missed the other part of my posted
patch, which was removing the trailing empty parens from the function
name ("current_schema()" => "current_schema" in two places in that
paragraph).

The paragraph rationalising this, from my original post:

--
I also removed the parentheses after the function name as this was a)
misleading the reader into thinking the function could be called
without arguments and b) inconsistent with the style for referring to
functions elsewhere in the chapter.
--

Was the exclusion deliberate?

Cheers,
BJ

view thread (6+ messages)  latest in thread

reply

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: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: Add link to current_schemas in config.sgml
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox