docs: minor typo fix for "lower(anymultirange)"

Started by Ian Lawrence Barwickover 1 year ago3 messages
#1Ian Lawrence Barwick
barwick@gmail.com
1 attachment(s)

Hi

Here:

https://www.postgresql.org/docs/current/functions-range.html#MULTIRANGE-FUNCTIONS-TABLE

the description for "lower(anymultirange)":

(NULL if the multirange is empty has no lower bound).

is missing "or" and should be:

(NULL if the multirange is empty or has no lower bound).

Seems to have been omitted with 7539a1b2 (REL_16_STABLE + master).

Very minor issue, but I have seen it and can no longer unsee it.

Regards

Ian Barwick

Attachments:

v1-0001-doc-fix-lower-anymultirange-description.patchtext/x-patch; charset=US-ASCII; name=v1-0001-doc-fix-lower-anymultirange-description.patchDownload
From e444498ee3934cd8d7360388d02b79b880c328f0 Mon Sep 17 00:00:00 2001
From: Ian Barwick <barwick@gmail.com>
Date: Thu, 25 Apr 2024 09:29:30 +0900
Subject: [PATCH v1] doc: fix "lower ( anymultirange )" description

Add missing "or".
---
 doc/src/sgml/func.sgml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/src/sgml/func.sgml b/doc/src/sgml/func.sgml
index ebcd936acb..a46530abee 100644
--- a/doc/src/sgml/func.sgml
+++ b/doc/src/sgml/func.sgml
@@ -21298,7 +21298,7 @@ SELECT NULLIF(value, '(none)') ...
        </para>
        <para>
         Extracts the lower bound of the multirange (<literal>NULL</literal> if the
-        multirange is empty has no lower bound).
+        multirange is empty or has no lower bound).
        </para>
        <para>
         <literal>lower('{[1.1,2.2)}'::nummultirange)</literal>

base-commit: e51c2a46b54ee294c699499bedf76940ce2f392c
-- 
2.39.3

#2Richard Guo
guofenglinux@gmail.com
In reply to: Ian Lawrence Barwick (#1)
Re: docs: minor typo fix for "lower(anymultirange)"

On Thu, Apr 25, 2024 at 8:40 AM Ian Lawrence Barwick <barwick@gmail.com>
wrote:

Hi

Here:

https://www.postgresql.org/docs/current/functions-range.html#MULTIRANGE-FUNCTIONS-TABLE

the description for "lower(anymultirange)":

(NULL if the multirange is empty has no lower bound).

is missing "or" and should be:

(NULL if the multirange is empty or has no lower bound).

Good catch! I checked the descriptions for "upper(anymultirange)",
"lower(anyrange)" and "upper(anyrange)", and they are all correct. We
should fix this one.

Thanks
Richard

#3Michael Paquier
michael@paquier.xyz
In reply to: Richard Guo (#2)
Re: docs: minor typo fix for "lower(anymultirange)"

On Thu, Apr 25, 2024 at 09:02:34AM +0800, Richard Guo wrote:

Good catch! I checked the descriptions for "upper(anymultirange)",
"lower(anyrange)" and "upper(anyrange)", and they are all correct. We
should fix this one.

+1.
--
Michael