pgsql: Mark multirange_constructor0() and multirange_constructor2() str

From: Alexander Korotkov <akorotkov(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Mark multirange_constructor0() and multirange_constructor2() str
Date: 2021-04-23 10:26:23
Message-ID: E1lZt0x-0003Dv-3c@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Mark multirange_constructor0() and multirange_constructor2() strict

These functions shouldn't receive null arguments: multirange_constructor0()
doesn't have any arguments while multirange_constructor2() has a single array
argument, which is never null.

But mark them strict anyway for the sake of uniformity.

Also, make checks for null arguments use elog() instead of ereport() as these
errors should normally be never thrown. And adjust corresponding comments.

Catversion is bumped.

Reported-by: Peter Eisentraut
Discussion: https://postgr.es/m/0f783a96-8d67-9e71-996b-f34a7352eeef%40enterprisedb.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/6bbcff096f932a1fe43ac3208c5c8b0acac29cda

Modified Files
--------------
src/backend/commands/typecmds.c | 4 ++--
src/backend/utils/adt/multirangetypes.c | 25 +++++++++++++------------
src/include/catalog/catversion.h | 2 +-
src/include/catalog/pg_proc.dat | 24 ++++++++++++------------
4 files changed, 28 insertions(+), 27 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2021-04-23 12:36:51 pgsql: Factor out system call names from error messages
Previous Message Fujii Masao 2021-04-23 10:11:15 pgsql: Reorder COMPRESSION option in gram.y and parsenodes.h into alpha