Re: Compilation error with buildtype = release

Ashutosh Bapat <ashutosh.bapat.oss@gmail.com> writes:
> I am seeing following error only with buildtype = release

Interesting.  I noticed skink showing the same thing as a warning,
but no other BF animals have shown it (yet anyway).

> Looking at the function, reltuples is indeed initialized in all the
> cases. All the relevant lines of the function are at least 3 years
> old, but I have started seeing this error only after
> 80aa9848befc13c188d2775a859deaf172fdd3a2.

Yeah.  The variable is clearly initialized in all cases where it's
used, but a compiler doing sloppy flow analysis might complain.
I suspect the relevance of 80aa9848b is that when there was a PG_TRY
in the function, these compilers backed off and didn't try to do flow
analysis at all.

> If I make the change as per attached patch, it compiles without an
> error. I don't claim that the patch is the right thing to do, but it
> might provide a hint.

Usually my answer to this sort of thing is to provide an initializer
for the variable.  Will fix.

			regards, tom lane