pgsql: Fix maintenance hazards caused by ill-considered use of default:

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix maintenance hazards caused by ill-considered use of default:
Date: 2017-05-14 17:33:05
Message-ID: E1d9xOD-0006e8-1F@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix maintenance hazards caused by ill-considered use of default: cases.

Remove default cases from assorted switches over ObjectClass and some
related enum types, so that we'll get compiler warnings when someone
adds a new enum value without accounting for it in all these places.

In passing, re-order some switch cases as needed to match the declaration
of enum ObjectClass. OK, that's just neatnik-ism, but I dislike code
that looks like it was assembled with the help of a dartboard.

Discussion: https://postgr.es/m/20170512221010.nglatgt5azzdxjlj@alvherre.pgsql

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/e84c0195980f24b1c7f857b88834c1dcaf20a102

Modified Files
--------------
src/backend/catalog/dependency.c | 21 ++-
src/backend/catalog/objectaddress.c | 305 ++++++++++++++++++-----------------
src/backend/commands/alter.c | 38 ++++-
src/backend/commands/event_trigger.c | 75 ++++++---
4 files changed, 259 insertions(+), 180 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2017-05-14 23:15:59 pgsql: Edit SGML documentation related to extended statistics.
Previous Message Tom Lane 2017-05-14 16:22:34 pgsql: Fix handling of extended statistics during ALTER COLUMN TYPE.