From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, 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 18:56:31 |
Message-ID: | 4925B2DF.1080602@hagander.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Tom Lane wrote:
> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
>> 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?
>
> We're definitely *not* fortify-clean, although maybe trying to become so
> would be a good idea. From memory, what I have seen in past builds on
> Red Hat systems is a lot of warnings about ignoring the return value
> from fwrite() and related functions. Which might not be an unreasonable
> thing to try to get rid of, though I think many of the cases are in
> places where there's no useful recovery action to be taken anyway.
> (Whatcha gonna do if you fail to write the postmaster log ... try to
> log a complaint message?)
>
> In any case I agree that anything we try to do about this should be a
> system-wide effort not a one-line hack.
>
>> 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.
>
> +1
So. Should I revert it?
//Magnus
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2008-11-20 19:52:54 | pgsql: Fix breakage of bitmap scan plan creation for special index |
Previous Message | Tom Lane | 2008-11-20 17:28:01 | Re: Re: [COMMITTERS] pgsql: Silence compiler warning about ignored return value. |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2008-11-20 18:59:28 | Re: TODO list request: FK to unique expression indexes |
Previous Message | Magnus Hagander | 2008-11-20 18:49:22 | Re: Autoconf, libpq and replacement function |