Peter Geoghegan wrote: > I'm seeing this on a the master branch tip when building at -O2: > > pg_conversion.c: In function ‘ConversionCreate’: > pg_conversion.c:53:6: error: variable ‘oid’ set but not used > [-Werror=unused-but-set-variable] > Oid oid; > ^ > cc1: all warnings being treated as errors > > I think that commit a2e35b53 did this.
Obviously the commit did not introduce an unused variable, but instead made another variable take its place as the function's return value. In an assert-enabled build it was used by an assert, which is why I didn't notice the problem. I removed the assert and the variable so this should be fixed now. Thanks for reporting. -- Álvaro Herrera http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers