On Tue, Jul 29, 2025 at 6:36 PM Tom Lane <t...@sss.pgh.pa.us> wrote: > > Ashutosh Bapat <ashutosh.bapat....@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.
Thanks. -- Best Wishes, Ashutosh Bapat
