public inbox for [email protected]  

help / color / mirror / Atom feed
From: Pavel Stehule <[email protected]>
To: Tom Lane <[email protected]>
Cc: Bruce Momjian <[email protected]>
Cc: pgsql-docs <[email protected]>
Subject: Re: remove undocumented assign syntax from plpgsql doc
Date: Sat, 11 Jan 2014 22:06:27 +0100
Message-ID: <CAFj8pRDukaWkUtQRZujhbLRCh6haOY-FDunVXjf2X2XNdNOKGQ@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <CAFj8pRDHJqRwZd-GO1t3WnLnS--E943xPrEUSOT_nL8xr9mE2g@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgsql-docs>

2014/1/11 Tom Lane <[email protected]>

> Bruce Momjian <[email protected]> writes:
> > Oh, I think you are right.  I have reverted the patch.  Attached is
> > proposed documentation for '='.
>
> Meh.  Variable initialization is only one of multiple cases (assignment,
> GET DIAGNOSTICS; maybe others, I've not examined the grammar).  Also,
> if we do it like this, we're implying that both := and = are equally
> preferred, which might not be the impression we want to leave.
>

GET DIAGNOSTICS is defined by standard - and there "=" should be allowed
only - although we allow ":=" too. It is a embedded SQL statement -
although it is implemented as plpgsql statement.

Same situation is with UPDATE statement - we don't allow ":=" there.


>
> I'd be a bit inclined to just stick a NOTE somewhere saying that "="
> can be used in place of ":=" for assignment.
>

ok

If we accept it and we close this topic, then following comment should be
removed

assign_operator : '='   /* not documented because it might be removed
someday */
                                | COLON_EQUALS
                                ;

Regards

Pavel



>
>                         regards, tom lane
>


view thread (27+ 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], [email protected]
  Subject: Re: remove undocumented assign syntax from plpgsql doc
  In-Reply-To: <CAFj8pRDukaWkUtQRZujhbLRCh6haOY-FDunVXjf2X2XNdNOKGQ@mail.gmail.com>

* 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