On 23/01/2026 14:13, Fujii Masao wrote:
> Hi,
>
> While reviewing the patch [1], I found that WalSndShutdown() calls abort()
> with the comment "keep the compiler quiet" just after proc_exit(0).
>
> static void
> WalSndShutdown(void)
> {
> /*
> * Reset whereToSendOutput to prevent ereport from attempting to send any
> * more messages to the standby.
> */
> if (whereToSendOutput == DestRemote)
> whereToSendOutput = DestNone;
>
> proc_exit(0);
> abort(); /* keep the compiler quiet */
> }
>
> This may have been necessary in the past, but is it still required?
> Other functions, such as CheckpointerMain(), simply call proc_exit(0)
> without an abort(), which doesn't seem to cause compiler warnings.
> That made me wonder whether the abort() in WalSndShutdown() is
> still needed, or which compiler would actually warn if WalSndLoop()
> does not end with an abort().
Seems useless to me. Looking at the git history, long time ago the
proc_exit(0) call was in a function that returned "int", and I can see
how the compiler would complain about that if it didn't know that the
function doesn't return. But WalSendShutdown() returns void these days,
so you should not get a compiler warning, whether or not the compiler
understands that proc_exit(0) doesn't return.
- Heikki
