Thread: a2e35b53 added unused variable to ConversionCreate()

a2e35b53 added unused variable to ConversionCreate()

From
Peter Geoghegan
Date:
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



Re: a2e35b53 added unused variable to ConversionCreate()

From
Alvaro Herrera
Date:
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