Re: Warnings from fwrite() in git head - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Warnings from fwrite() in git head
Date
Msg-id 2801.1349712774@sss.pgh.pa.us
Whole thread Raw
In response to Warnings from fwrite() in git head  (Bruce Momjian <bruce@momjian.us>)
Responses Re: Warnings from fwrite() in git head  (Bruce Momjian <bruce@momjian.us>)
List pgsql-hackers
Bruce Momjian <bruce@momjian.us> writes:
> I am seeing the following warnings in git head from zic.c:
>     zic.c:1505: warning: ignoring return value of ‘fwrite’, declared with attribute warn_unused_result

Yeah, this is probably a consequence of the _FORTIFY_SOURCE addition.
I believe that ratchets up warning pickiness as well as other things.

I'm inclined to think we should make zic.c bail out on write errors.
Otherwise, "make install" could fail to notice an out-of-disk-space
situation during install.  The analogy you are drawing to ignoring
errors while writing log messages is quite faulty.

We're overdue for another round of syncing the tz code with upstream,
too.  Might be best to do that first, just in case Olson's crew already
fixed this.
        regards, tom lane



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: static-if-inline
Next
From: Jeff Janes
Date:
Subject: Re: Warnings from fwrite() in git head