Message-ID: <16601466752e40543051369fac722c0a200bb4e6.camel@cybertec.at>
Subject: Re: Documentation of return values of range functions lower and
 upper
From: Laurenz Albe <laurenz.albe@cybertec.at>
To: jani.rahkola@iki.fi, pgsql-docs@lists.postgresql.org
Date: Thu, 12 Nov 2020 09:14:12 +0100
In-Reply-To: <53ec9f116bea9252d7fa9341e024530553656c42.camel@cybertec.at>
References: <160508672127.25505.8356390205508789564@wrigleys.postgresql.org>
	 <53ec9f116bea9252d7fa9341e024530553656c42.camel@cybertec.at>
Content-Type: multipart/mixed; boundary="=-1qtsEYjRS/qk1sVQC6l7"
User-Agent: Evolution 3.36.5 (3.36.5-1.fc32) 
MIME-Version: 1.0
Precedence: bulk


--=-1qtsEYjRS/qk1sVQC6l7
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit

On Wed, 2020-11-11 at 18:19 +0100, Laurenz Albe wrote:
> > Table 9.54 in page
> > https://www.postgresql.org/docs/current/functions-range.html states that the
> > functions lower and upper return NULL if the requested bound is infinite. If
> > the element type of the range contains the special values infinity and
> > -infinity, this is not correct, as those values are returned if explicitly
> > used as either bound.
> 
> +1
> 
> Perhaps it would be better to say
> 
>   NULL if the range is empty or has no lower/upper bound

Here is a patch for this.

Yours,
Laurenz Albe

--=-1qtsEYjRS/qk1sVQC6l7
Content-Disposition: attachment;
	filename="0001-Improve-upper-lower-documentation-for-ranges.patch"
Content-Transfer-Encoding: base64
Content-Type: text/x-patch;
	name="0001-Improve-upper-lower-documentation-for-ranges.patch";
	charset="UTF-8"

RnJvbSBiMTdkMmQ5NjgyNjgyNTdjNWQxMzg1YmI4MzQ3NDdmZTNjMzM4MzRiIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBMYXVyZW56IEFsYmUgPGxhdXJlbnouYWxiZUBjeWJlcnRlYy5h
dD4KRGF0ZTogVGh1LCAxMiBOb3YgMjAyMCAwOToxMzowOSArMDEwMApTdWJqZWN0OiBbUEFUQ0hd
IEltcHJvdmUgdXBwZXIvbG93ZXIgZG9jdW1lbnRhdGlvbiBmb3IgcmFuZ2VzCgpUaGUgb2xkIHdv
cmRpbmcgY291bGQgZ2l2ZSB0aGUgaW1wcmVzc2lvbiB0aGF0IHRoZXNlCmZ1bmN0aW9ucyB3b3Vs
ZCByZXR1cm4gTlVMTCBpZiB0aGUgcmFuZ2UgaXMgYm91bmRlZApieSBhIHZhbHVlICJpbmZpbml0
eSIuICBSZXBvcnQgYnkgSmFuaSBSYWhrb2xhLgoKRGlzY3Vzc2lvbjogaHR0cHM6Ly9wb3N0Z3Iu
ZXMvbS8xNjA1MDg2NzIxMjcuMjU1MDUuODM1NjM5MDIwNTUwODc4OTU2NEB3cmlnbGV5cy5wb3N0
Z3Jlc3FsLm9yZwotLS0KIGRvYy9zcmMvc2dtbC9mdW5jLnNnbWwgfCA0ICsrLS0KIDEgZmlsZSBj
aGFuZ2VkLCAyIGluc2VydGlvbnMoKyksIDIgZGVsZXRpb25zKC0pCgpkaWZmIC0tZ2l0IGEvZG9j
L3NyYy9zZ21sL2Z1bmMuc2dtbCBiL2RvYy9zcmMvc2dtbC9mdW5jLnNnbWwKaW5kZXggMjc4Mzk4
NWI1NS4uZDI5MDY1M2FlOCAxMDA2NDQKLS0tIGEvZG9jL3NyYy9zZ21sL2Z1bmMuc2dtbAorKysg
Yi9kb2Mvc3JjL3NnbWwvZnVuYy5zZ21sCkBAIC0xODE3Niw3ICsxODE3Niw3IEBAIFNFTEVDVCBO
VUxMSUYodmFsdWUsICcobm9uZSknKSAuLi4KICAgICAgICA8L3BhcmE+CiAgICAgICAgPHBhcmE+
CiAgICAgICAgIEV4dHJhY3RzIHRoZSBsb3dlciBib3VuZCBvZiB0aGUgcmFuZ2UgKDxsaXRlcmFs
Pk5VTEw8L2xpdGVyYWw+IGlmIHRoZQotICAgICAgICByYW5nZSBpcyBlbXB0eSBvciB0aGUgbG93
ZXIgYm91bmQgaXMgaW5maW5pdGUpLgorICAgICAgICByYW5nZSBpcyBlbXB0eSBvciBoYXMgbm8g
bG93ZXIgYm91bmQpLgogICAgICAgIDwvcGFyYT4KICAgICAgICA8cGFyYT4KICAgICAgICAgPGxp
dGVyYWw+bG93ZXIobnVtcmFuZ2UoMS4xLDIuMikpPC9saXRlcmFsPgpAQCAtMTgxOTQsNyArMTgx
OTQsNyBAQCBTRUxFQ1QgTlVMTElGKHZhbHVlLCAnKG5vbmUpJykgLi4uCiAgICAgICAgPC9wYXJh
PgogICAgICAgIDxwYXJhPgogICAgICAgICBFeHRyYWN0cyB0aGUgdXBwZXIgYm91bmQgb2YgdGhl
IHJhbmdlICg8bGl0ZXJhbD5OVUxMPC9saXRlcmFsPiBpZiB0aGUKLSAgICAgICAgcmFuZ2UgaXMg
ZW1wdHkgb3IgdGhlIHVwcGVyIGJvdW5kIGlzIGluZmluaXRlKS4KKyAgICAgICAgcmFuZ2UgaXMg
ZW1wdHkgb3IgaGFzIG5vIHVwcGVyIGJvdW5kKS4KICAgICAgICA8L3BhcmE+CiAgICAgICAgPHBh
cmE+CiAgICAgICAgIDxsaXRlcmFsPnVwcGVyKG51bXJhbmdlKDEuMSwyLjIpKTwvbGl0ZXJhbD4K
LS0gCjIuMjYuMgoK


--=-1qtsEYjRS/qk1sVQC6l7--