On Tue, Apr 4, 2017 at 9:42 PM, Michael Paquier <[email protected]> wrote: > On Wed, Apr 5, 2017 at 2:54 AM, Tom Lane <[email protected]> wrote: >> (I'm personally not that much in love with PG_USED_FOR_ASSERTS_ONLY, >> because it tends to confuse pgindent.) > > I would be incline to just do that, any other solution I can think of > is uglier than that.
Actually, no. Looking at this issue again the warning is triggered because the Assert() clause is present in USE_ASSERT_CHECKING. So instead of removing PG_USED_FOR_ASSERTS_ONLY, here is a more simple patch that fixes the problem. No need to put the variable *rte within ifdefs as well. -- Michael
costsize-warning-fix-2.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
