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
