pgsql: Rearrange pseudotypes.c to get rid of duplicative code.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Rearrange pseudotypes.c to get rid of duplicative code.
Date: 2020-03-14 19:31:52
Message-ID: E1jDCVk-0000yo-3V@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Rearrange pseudotypes.c to get rid of duplicative code.

Commit a5954de10 replaced a lot of manually-coded stub I/O routines
with code generated by macros. That was a good idea but it didn't
go far enough, because there were still manually-coded stub input
routines for types that had live output routines. Refactor the
macro so that we can generate just a stub input routine at need.

Also create similar macros to generate stub binary I/O routines,
since we have some of those now. The only stub functions that remain
hand-coded are shell_in() and shell_out(), which need to be separate
because they use different error messages.

While here, rearrange the commentary to discuss each type not each
function. This provides a better way to explain the *why* of which
types need which support, rather than just duplicatively annotating
the functions.

Discussion: https://postgr.es/m/24137.1584139352@sss.pgh.pa.us

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/87c9c2571c8146f1594830072253ba39e62ccbc8

Modified Files
--------------
src/backend/utils/adt/pseudotypes.c | 348 +++++++++++++-----------------------
1 file changed, 123 insertions(+), 225 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Bruce Momjian 2020-03-14 21:36:59 pgsql: C comment: correct commented bytes of max_cached_tuplebufs
Previous Message Tom Lane 2020-03-14 18:42:49 pgsql: Restructure polymorphic-type resolution in funcapi.c.