Re: pgsql: Check dup2() results in syslogger

From: Stephen Frost <sfrost(at)snowman(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Check dup2() results in syslogger
Date: 2014-01-28 13:45:39
Message-ID: 20140128134539.GJ31026@tamriel.snowman.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

* Tom Lane (tgl(at)sss(dot)pgh(dot)pa(dot)us) wrote:
> Stephen Frost <sfrost(at)snowman(dot)net> writes:
> > * Tom Lane (tgl(at)sss(dot)pgh(dot)pa(dot)us) wrote:
> >> In short, this patch was ill considered. Please revert. If we need
> >> to silence a Coverity complaint, perhaps a cast-to-void will do?
>
> > Sure, I'll adjust it accordingly.
>
> Feel free to improve the comment if you think it could be clearer.

I hemmed and hawed over it and tried to improve it but I'm not convinced
that I did. Still, I went ahead and at least got the revert committed.
If anyone feels the comment change hurts more than helps, let me know.

Thanks,

Stephen

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Bruce Momjian 2014-01-28 16:29:45 pgsql: Remove orphaned prototype
Previous Message Stephen Frost 2014-01-28 13:41:27 pgsql: Revert dup2() checking in syslogger.c