From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | rir <rirans(at)comcast(dot)net> |
Cc: | Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>, pgsql-docs(at)lists(dot)postgresql(dot)org |
Subject: | Re: small patch |
Date: | 2021-10-07 20:24:16 |
Message-ID: | 175281.1633638256@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs |
rir <rirans(at)comcast(dot)net> writes:
> On Thu, Oct 07, 2021 at 07:58:47AM +0200, Laurenz Albe wrote:
>> Can you think of a way to modify the syntax diagram so that it
>> expresses that and still remains comprehensible?
> For myself,
> 'FETCH [ <direction> [ FROM | IN ] ] <cursor_name>'
> clearly indicates that 'direction' is optional.
Right, but if we don't say that <direction> can be
empty, then this diagram disallows
FETCH FROM cursor_name
which in fact is legal. I think however that we could make it read
FETCH [ <direction> ] [ FROM | IN ] <cursor_name>'
and have a correct description without requiring <direction>
to be allowed to be empty.
BTW, as it stands, the diagram is ambiguous
because there are two ways to parse
FETCH cursor_name
... is <direction> present but empty, or omitted altogether?
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Anitha P | 2021-10-08 07:34:14 | Fwd: PostgreSQL 12 Authentication type questions |
Previous Message | Alvaro Herrera | 2021-10-07 20:18:49 | Re: small patch |