Remove assertions that some compiler say are tautological
authorHeikki Linnakangas
Wed, 20 Mar 2024 07:14:51 +0000 (09:14 +0200)
committerHeikki Linnakangas
Wed, 20 Mar 2024 07:14:51 +0000 (09:14 +0200)
commitd63d486d6c393817810d0477569fc657c388bfb2
treede2b5d708f136f52426798c9a1997781f69e51d6
parent9578393bc513e350e9cbfa4679fc7be7309b41a4
Remove assertions that some compiler say are tautological

To avoid the compiler warnings:

    launch_backend.c:211:39: warning: comparison of constant 16 with expression of type 'BackendType' (aka 'enum BackendType') is always true [-Wtautological-constant-out-of-range-compare]
    launch_backend.c:233:39: warning: comparison of constant 16 with expression of type 'BackendType' (aka 'enum BackendType') is always true [-Wtautological-constant-out-of-range-compare]

The point of the assertions was to fail more explicitly if someone
adds a new BackendType to the end of the enum, but forgets to add it
to the child_process_kinds array. It was a pretty weak assertion to
begin with, because it wouldn't catch if you added a new BackendType
in the middle of the enum. So let's just remove it.

Per buildfarm member ayu and a few others, spotted by Tom Lane.

Discussion: https://www.postgresql.org/message-id/4119680.1710913067@sss.pgh.pa.us
src/backend/postmaster/launch_backend.c
src/include/miscadmin.h