pgsql: Refactor datetime functions' timezone lookup code to reduce dupl

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Refactor datetime functions' timezone lookup code to reduce dupl
Date: 2023-03-17 21:47:26
Message-ID: E1pdHv4-0042sd-39@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Refactor datetime functions' timezone lookup code to reduce duplication.

We already had five copies of essentially the same logic, and an
upcoming patch introduces yet another use-case. That's past my
threshold of pain, so introduce a common subroutine. There's not
that much net code savings, but the chance of typos should go down.

Inspired by a patch from Przemysław Sztoch, but different in detail.

Discussion: https://postgr.es/m/01a84551-48dd-1359-bf7e-f6b0203a6bd0@sztoch.pl

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/3e59e5048d0f20debe4ad79b2e02ca2a76c5daed

Modified Files
--------------
src/backend/utils/adt/date.c | 51 +++--------
src/backend/utils/adt/datetime.c | 85 ++++++++++++++++++
src/backend/utils/adt/timestamp.c | 178 ++++++++------------------------------
src/include/utils/datetime.h | 8 ++
4 files changed, 142 insertions(+), 180 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Michael Paquier 2023-03-18 09:06:20 pgsql: Add files related to query jumbling in src/include/nodes/ for me
Previous Message Peter Eisentraut 2023-03-17 20:41:50 pgsql: Fix typo