| 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(-)
| 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. |