Re: new compiler warnings - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: new compiler warnings
Date
Msg-id 1318946787.8007.6.camel@fsopti579.F-Secure.com
Whole thread Raw
In response to Re: new compiler warnings  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: new compiler warnings
List pgsql-hackers
On tis, 2011-10-18 at 09:32 -0400, Tom Lane wrote:
> Andrew Dunstan <andrew@dunslane.net> writes:
> > It is a pity we can't just tell the compiler to turn off the warning in 
> > a particular case.
> 
> I haven't tested, but won't an explicit cast to void silence the
> warning?
> 
>     (void) fwrite(...);

No, tried that already.  You could try
   rc = write(...);   (void) rc;

> There are places, notably the calls in elog.c, where ignoring write
> failures is the right thing.  I think that what Kevin was on about
> was something else entirely, namely whether we need to retry writes
> to disk.  I would hope that we're not simply not bothering to check
> in any cases where it matters.

No, I believe we are OK everywhere else.  We are only ignoring the
result in cases where we are trying to report errors in the first place.




pgsql-hackers by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: pg_ctl restart - behaviour based on wrong instance
Next
From: Robert Haas
Date:
Subject: Re: new compiler warnings