On Thu, Oct 22, 2020 at 3:15 PM Mark Dilger <mark.dil...@enterprisedb.com> wrote: > The 0001 attached patch addresses the -Werror=maybe-uninitialized problem.
I am skeptical. Why so much code churn to fix a compiler warning? And even in the revised code, *status isn't set in all cases, so I don't see why this would satisfy the compiler. Even if it satisfies this particular compiler for some other reason, some other compiler is bound to be unhappy sometime. It's better to just arrange to set *status always, and use a dummy value in cases where it doesn't matter. Also, "return XID_BOUNDS_OK;;" has exceeded its recommended allowance of semicolons. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
