Re: [HACKERS] a2e35b53 added unused variable to ConversionCreate()

2015-03-05 Thread Alvaro Herrera
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 Herrerahttp://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training  Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


[HACKERS] a2e35b53 added unused variable to ConversionCreate()

2015-03-04 Thread Peter Geoghegan
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.

-- 
Peter Geoghegan


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers