Re: Compiler warning in master branch

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: Peter Geoghegan <pg(at)heroku(dot)com>
Cc: Pg Hackers <pgsql-hackers(at)postgresql(dot)org>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Subject: Re: Compiler warning in master branch
Date: 2014-11-10 07:29:58
Message-ID: CAApHDvrQcjRjUChEycp2BOv+xtJ8innEMGWoAgaqnkeeAtGVRQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Nov 10, 2014 at 4:31 PM, Peter Geoghegan <pg(at)heroku(dot)com> wrote:

> I see this when I build at -O2 from master's tip:
>
> gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith
> -Wdeclaration-after-statement -Wendif-labels
> -Wmissing-format-attribute -Wformat-security -fno-strict-aliasing
> -fwrapv -fexcess-precision=standard -Werror -I../../../../src/include
> -D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -I/usr/include/libxml2 -c -o
> brin_revmap.o brin_revmap.c -MMD -MP -MF .deps/brin_revmap.Po
> brin_revmap.c: In function ‘brinRevmapExtend’:
> brin_revmap.c:113:14: error: variable ‘mapBlk’ set but not used
> [-Werror=unused-but-set-variable]
> BlockNumber mapBlk;
> ^
>
> I'm using gcc 4.8.2 here.
>
>
It would appear just to need the attached.

Regards

David Rowley

Attachment Content-Type Size
brin_used_for_asserts_only_fix.diff text/plain 467 bytes

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2014-11-10 07:37:25 Re: [v9.5] Custom Plan API
Previous Message Etsuro Fujita 2014-11-10 07:15:06 Re: PENDING_LIST_CLEANUP_SIZE - maximum size of GIN pending list Re: HEAD seems to generate larger WAL regarding GIN index