Re: removing some dead "keep compiler quiet" code

Tom Lane <tgl@sss.pgh.pa.us>

From: Tom Lane <tgl@sss.pgh.pa.us>
To: Peter Eisentraut <peter_e@gmx.net>
Cc: pgsql-hackers@postgresql.org
Date: 2012-11-20T04:09:44Z
Lists: pgsql-hackers
Peter Eisentraut <peter_e@gmx.net> writes:
> Since ereport() can now communicate to the compiler whether it returns
> or not, a fair amount of "keep compiler quiet" code is dead.  Since the
> method that ereport() uses is not dependent on any compiler-specific
> attributes, I think this code can just be removed.  I propose the
> attached patch.

Meh.  This will only work if the compiler understands that abort()
doesn't return --- compilers that don't know that will start bleating
about uninitialized variables again.  And the ones that do know it
will be able to throw away the dead code anyway.

I doubt this is a good idea.

			regards, tom lane