Re: [HACKERS][PATCH] adding simple sock check for windows - Mailing list pgsql-hackers

From CharSyam
Subject Re: [HACKERS][PATCH] adding simple sock check for windows
Date
Msg-id CAMrLSE7Sy1erv+QuyBRDoGXQzp2Jv7dtT_tkBqk8Xi2fcO9i4g@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS][PATCH] adding simple sock check for windows  (Amit Kapila <amit.kapila16@gmail.com>)
Responses Re: [HACKERS][PATCH] adding simple sock check for windows
Re: [HACKERS][PATCH] adding simple sock check for windows
List pgsql-hackers
Thanks Amit.
I had a mistake. Thank you again to point it out :)


2018-03-31 19:33 GMT+09:00 Amit Kapila <amit.kapila16@gmail.com>:
> On Sat, Mar 31, 2018 at 12:05 PM, CharSyam <charsyam@gmail.com> wrote:
>> Amit, I agree with you.
>>
>> I changed my patch :) to remove old check.
>>
>
> - if (slot->sock < 0)
> + if (slot->sock == PGINVALID_SOCKET || slot->sock < 0)
>
> I still see the same check. I think by mistake you have attached old patch.
>
> --
> With Regards,
> Amit Kapila.
> EnterpriseDB: http://www.enterprisedb.com

Attachment

pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Problem while setting the fpw with SIGHUP
Next
From: Michael Banck
Date:
Subject: Re: [PATCH] Verify Checksums during Basebackups