From: | rir <rirans(at)comcast(dot)net> |
---|---|
To: | Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at> |
Cc: | rir <rirans(at)comcast(dot)net>, pgsql-docs(at)lists(dot)postgresql(dot)org |
Subject: | Re: small patch |
Date: | 2021-10-07 20:06:12 |
Message-ID: | 20211007200612.rkxxgeorw6w4fpjp@localhost |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs |
On Thu, Oct 07, 2021 at 07:58:47AM +0200, Laurenz Albe wrote:
> On Wed, 2021-10-06 at 23:39 -0400, rir wrote:
> > On Mon, Oct 04, 2021 at 08:18:22AM +0200, Laurenz Albe wrote:
> > > On Fri, 2021-10-01 at 21:06 -0400, rir wrote:
> > > > Minor changes to move.sgml and fetch.sgml.
> > > However, I don't think that is an improvement:
> > >
> > > - the comments pointing from MOVE to FETCH and vice versa are
> > > helpful for people who edit the documentation like you did
> > > - we should retain "empty or one of", otherwise the following syntax
> > > would be undocumented:
> > >
> > > FETCH FROM c;
> >
> > Your view is completely reasonable, but it suggests that
> > many of the synopses are leaving syntax undocumented.
> > The 'empty or one of:' is only used in these two synopses.
> 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.
Rob
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2021-10-07 20:18:49 | Re: small patch |
Previous Message | Sanjeev Adwal | 2021-10-07 08:14:16 | Re: Correction to documentation at https://www.postgresql.org/docs/12/kernel-resources.html |