From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Re: [COMMITTERS] pgsql: Silence compiler warning about ignored return value. |
Date: | 2008-11-20 15:29:50 |
Message-ID: | 4925826E.9090604@gmx.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Magnus Hagander wrote:
> Heikki Linnakangas wrote:
>> Magnus Hagander wrote:
>>> Log Message:
>>> -----------
>>> Silence compiler warning about ignored return value. Our comment already
>>> clearly stated that we are aware that we're ignoring it.
>> I think the usual way is to call the function like:
>>
>> (void) function_with_return_value()
>
> I tried that first, of course. gcc is too smart about that - it still
> throws the warning in this case.
Well, the warning is explicitly put in there for this specific function
because you are supposed to process the return value. I'm sure a more
smarter compiler would even warn "variable is assigned a value that is
never used". ;-) (Note that gcc in general doesn't work about unused
return values, only for those functions that glibc explicitly marks as
candidates.)
It looks like you are building in fortify mode? I tried that a while
ago and got a few more warnings. Are we trying to be fortify clean, and
if so, what is our approach?
Also, considering my recent complaint about various brittleness in the
regression test driver, more well hidden ignorings of errors are not
exactly my favorite solution.
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2008-11-20 15:34:19 | Re: Re: [COMMITTERS] pgsql: Silence compiler warning about ignored return value. |
Previous Message | Tom Lane | 2008-11-20 15:27:09 | Re: Re: [COMMITTERS] pgsql: Silence compiler warning about ignored return value. |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2008-11-20 15:31:23 | Re: Updates of SE-PostgreSQL 8.4devel patches (r1197) |
Previous Message | Tom Lane | 2008-11-20 15:27:59 | Re: Error arguments in pl_exec.c |