> On May 5, 2020, at 5:42 AM, Daniel Fuchs <daniel.fu...@oracle.com> wrote: > > Hi Mikael, > > I spotted another place where a residual reference to Solaris > remains in a comment: > src/java.base/unix/native/libnet/PlainSocketImpl.c > > 857 #if defined(_AIX) > 858 if (errno == EINVAL) { > 859 // On Solaris setsockopt will set errno to EINVAL if the > socket > 860 // is closed. The default error message is then confusing Fixed in the upcoming webrev! > Otherwise I had a look at the networking related files and > the changes seemed reasonable to me. Thanks a lot for the review! Cheers, Mikael
- Re: RFR: 8244224: Implementation of JEP 381: Remove the S... Alan Bateman
- Re: RFR: 8244224: Implementation of JEP 381: Remove ... Roger Riggs
- Re: RFR: 8244224: Implementation of JEP 381: Rem... Mikael Vidstedt
- Re: RFR: 8244224: Implementation of JEP 381:... Kumar Srinivasan
- Re: RFR: 8244224: Implementation of JEP 381: Remove ... Mikael Vidstedt
- Re: RFR: 8244224: Implementation of JEP 381: Remove the S... Weijun Wang
- Re: RFR: 8244224: Implementation of JEP 381: Remove ... Mikael Vidstedt
- Re: RFR: 8244224: Implementation of JEP 381: Remove the S... naoto . sato
- Re: RFR: 8244224: Implementation of JEP 381: Remove ... Mikael Vidstedt
- Re: RFR: 8244224: Implementation of JEP 381: Remove the S... Daniel Fuchs
- Re: RFR: 8244224: Implementation of JEP 381: Remove ... Mikael Vidstedt
- Re: RFR: 8244224: Implementation of JEP 381: Remove the S... Brent Christian
- Re: RFR: 8244224: Implementation of JEP 381: Remove ... Mikael Vidstedt
- Re: RFR: 8244224: Implementation of JEP 381: Remove the S... Mikael Vidstedt
- Re: RFR: 8244224: Implementation of JEP 381: Remove ... Alan Bateman
- Re: RFR: 8244224: Implementation of JEP 381: Rem... Mikael Vidstedt