Re: Warning in new GIN code

From: Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)postgreSQL(dot)org
Subject: Re: Warning in new GIN code
Date: 2014-01-23 20:16:06
Message-ID: 52E17886.7050301@vmware.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 01/23/2014 08:41 PM, Tom Lane wrote:
> The rather ancient gcc on my HPUX box is complaining thusly about HEAD:
>
> ginbtree.c: In function `ginPlaceToPage':
> ginbtree.c:602: warning: control reaches end of non-void function
>
> I would imagine this would happen on any compiler that doesn't recognize
> the hint about elog(ERROR) not returning. Could we stick a dummy return
> at the end?
>
> }
> else
> + {
> elog(ERROR, "unknown return code from GIN placeToPage method: %d", rc);
> + return false; /* keep compiler quiet */
> + }
> }

Fixed, thanks.

- Heikki

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2014-01-23 20:20:54 Re: Add CREATE support to event triggers
Previous Message Tom Lane 2014-01-23 19:50:57 Re: Add %z support to elog/ereport?