Improve unreachability recognition in elog() macro.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 10 Sep 2016 21:54:23 +0000 (17:54 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 10 Sep 2016 21:54:23 +0000 (17:54 -0400)
commit43ef6abcbb6010593a000a0d42303f138b9f5b4c
tree34fd6a37e6d1fb93fc2ffa3cc59c7d113ecc9285
parentbca6eeb82e856bc378fb31781d519caed5123b63
Improve unreachability recognition in elog() macro.

Some experimentation with an older version of gcc showed that it is able
to determine whether "if (elevel_ >= ERROR)" is compile-time constant
if elevel_ is declared "const", but otherwise not so much.  We had
accounted for that in ereport() but were too miserly with braces to
make it so in elog().  I don't know how many currently-interesting
compilers have the same quirk, but in case it will save some code
space, let's make sure that elog() is on the same footing as ereport()
for this purpose.

Back-patch to 9.3 where we introduced pg_unreachable() calls into
elog/ereport.
src/include/utils/elog.h