On 24.03.22 13:09, Michael Paquier wrote:
Hmm. I think that you'd better add a return at the end of each function? Some compilers are dumb in detecting that all the code paths return (aka recent d0083c1) and could generate warnings, even if things are coded to return all the time, like in your patch.
That is a different case. We know that not all compilers understand when elog/ereport return. But no compiler is stupid enough not to understand that
foo() { if (something) return this; else return that; } always reaches a return.