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
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 |