Re: Warning in new GIN code - Mailing list pgsql-hackers

From Heikki Linnakangas
Subject Re: Warning in new GIN code
Date
Msg-id 52E17886.7050301@vmware.com
Whole thread Raw
In response to Warning in new GIN code  (Tom Lane <tgl@sss.pgh.pa.us>)
List 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



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Add %z support to elog/ereport?
Next
From: Tom Lane
Date:
Subject: Re: Add CREATE support to event triggers