Re: Warnings from fwrite() in git head

Поиск
Список
Период
Сортировка
От Bruce Momjian
Тема Re: Warnings from fwrite() in git head
Дата
Msg-id 20121011021710.GJ11890@momjian.us
обсуждение исходный текст
Ответ на Re: Warnings from fwrite() in git head  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: Warnings from fwrite() in git head
Список pgsql-hackers
On Mon, Oct  8, 2012 at 12:12:54PM -0400, Tom Lane wrote:
> 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.

My warnings have disappeared now that _FORTIFY_SOURCE has been removed
on Linux.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Bruce Momjian
Дата:
Сообщение: Re: change in LOCK behavior
Следующее
От: Greg Stark
Дата:
Сообщение: Re: [PATCH 8/8] Introduce wal decoding via catalog timetravel