Minor changes to move.sgml and fetch.sgml. The text 'or empty' is inconsistent by restating what the synopsis notation has expressed.
The comments on sharing a language feature, while likely helpful during review, seem verbose compared to the non-commenting in other similar files. Thanks again, Rob
:From a297377ad3b760f10e951cf67ad0cea704214f34 Mon Sep 17 00:00:00 2001 From: rir <rir...@comcast.net> Date: Wed, 29 Sep 2021 19:06:01 -0400 Subject: [PATCH] "empty or" is unusual and redundant --- doc/src/sgml/ref/fetch.sgml | 4 ++-- doc/src/sgml/ref/move.sgml | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/doc/src/sgml/ref/fetch.sgml b/doc/src/sgml/ref/fetch.sgml index ec843f5684..e76b6fc69d 100644 --- a/doc/src/sgml/ref/fetch.sgml +++ b/doc/src/sgml/ref/fetch.sgml @@ -25,11 +25,11 @@ PostgreSQL documentation </refnamediv> <refsynopsisdiv> -<!-- Note the "direction" bit is also in ref/move.sgml --> + <synopsis> FETCH [ <replaceable class="parameter">direction</replaceable> [ FROM | IN ] ] <replaceable class="parameter">cursor_name</replaceable> -<phrase>where <replaceable class="parameter">direction</replaceable> can be empty or one of:</phrase> +<phrase>where <replaceable class="parameter">direction</replaceable> can one of:</phrase> NEXT PRIOR diff --git a/doc/src/sgml/ref/move.sgml b/doc/src/sgml/ref/move.sgml index 4c7d1dca39..1ed06e4c9a 100644 --- a/doc/src/sgml/ref/move.sgml +++ b/doc/src/sgml/ref/move.sgml @@ -25,11 +25,11 @@ PostgreSQL documentation </refnamediv> <refsynopsisdiv> -<!-- Note the "direction" bit is also in ref/fetch.sgml --> + <synopsis> MOVE [ <replaceable class="parameter">direction</replaceable> [ FROM | IN ] ] <replaceable class="parameter">cursor_name</replaceable> -<phrase>where <replaceable class="parameter">direction</replaceable> can be empty or one of:</phrase> +<phrase>where <replaceable class="parameter">direction</replaceable> can one of:</phrase> NEXT PRIOR -- 2.20.1