Re: [12] RFR 8129310: java/net/Socket/asyncClose/AsyncClose.java fails intermittently

2018-11-19 Thread Chris Yin
Thank you, Daniel Regards, Chris > On 19 Nov 2018, at 5:40 PM, Daniel Fuchs wrote: > > Looks good to me Chris! > > best regards, > > -- daniel > > On 19/11/2018 03:03, Chris Yin wrote: >> Hi, Daniel >>> On 16 Nov 2018, at 7:13 PM, Daniel Fuchs wrote: >>> >>> Hi Chris, >>> >>> This looks g

Re: [12] RFR 8129310: java/net/Socket/asyncClose/AsyncClose.java fails intermittently

2018-11-19 Thread Daniel Fuchs
Looks good to me Chris! best regards, -- daniel On 19/11/2018 03:03, Chris Yin wrote: Hi, Daniel On 16 Nov 2018, at 7:13 PM, Daniel Fuchs wrote: Hi Chris, This looks good to me, though I wonder if the check for reason.length() > 0 should be made in 51 public synchronized String fai