Re: ask for review of MERGE - Mailing list pgsql-hackers

From Josh Kupershmidt
Subject Re: ask for review of MERGE
Date
Msg-id AANLkTimHA7AXEu7y3K=1jhp3Vf8roQd9WLYU47=YaMoh@mail.gmail.com
Whole thread Raw
In response to Re: ask for review of MERGE  (Greg Smith <greg@2ndquadrant.com>)
List pgsql-hackers
On Wed, Sep 29, 2010 at 2:44 AM, Greg Smith <greg@2ndquadrant.com> wrote:

> The rest of the compiler warnings I saw didn't look related to his code,
> maybe stuff my picky Ubuntu compiler is noticing that was done recently to
> HEAD. I haven't checked HEAD without this patch yet to confirm, and am done
> for the night now. Here's the list if anyone is interested:
>
> Warning in src/backend/parser/scan.c:
>
> gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith
> -Wdeclaration-after-statement -Wendif-labels -fno-strict-aliasing -fwrapv -g
> -I../../../src/include -D_GNU_SOURCE -c -o index.o index.c -MMD -MP -MF
> .deps/index.Po
> In file included from gram.y:12172:
> scan.c: In function ‘yy_try_NUL_trans’:
> scan.c:16256: warning: unused variable ‘yyg’

Known problem: http://archives.postgresql.org/pgsql-hackers/2009-07/msg00657.php

I'm pretty sure I've seen the warn_unused_result warnings on HEAD as well.

Josh


pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: recovery.conf location
Next
From: Robert Haas
Date:
Subject: Re: I: About "Our CLUSTER implementation is pessimal" patch