On Fri, Oct 08, 2021 at 02:47:43PM +0200, Laurenz Albe wrote: > On Thu, 2021-10-07 at 16:06 -0400, rir wrote:
> So I suggest that you change the syntax diagram to > > FETCH [ direction ] [ FROM | IN ] cursor_name > Then I agree that the "empty or" can be removed. > I remain of the opinion that the comments should be > retained, but we can leave that for somebody else to > decide. I accept your three points above. The MOVE synopsis shows the same parsing as I presented, should it be changed in the same way (move a square bracket left to be after <direction>)? My guess is yes, but I've never used an SQL cursor. When this convo settles, I send a new patch. Probably here in the group. If I have a few more, or a complex one, I'll check out the other submission method. Rob