Re: RFR: 8227720: Improve ExtendedSocketOptions initialization

2019-07-16 Thread Claes Redestad
On 2019-07-16 17:11, Vyom Tewari26 wrote: Looks good to me. Vyom Thanks for reviewing, Vyom /Claes

Re: RFR: 8227720: Improve ExtendedSocketOptions initialization

2019-07-16 Thread Vyom Tewari26
Looks good to me. Vyom   - Original message -From: Claes Redestad Sent by: "net-dev" To: OpenJDK Network Dev list Cc:Subject: [EXTERNAL] RFR: 8227720: Improve ExtendedSocketOptions initializationDate: Tue, Jul 16, 2019 7:36 PM  Hi,a small cleanup and startup optimization for apps that setup

Re: RFR: 8227720: Improve ExtendedSocketOptions initialization

2019-07-16 Thread Claes Redestad
On 2019-07-16 17:01, Chris Hegarty wrote: On 16 Jul 2019, at 15:06, Claes Redestad wrote: Webrev: http://cr.openjdk.java.net/~redestad/8227720/open.00/ This looks good to me Claes. Thanks for doing it. Reviewed. Thanks for reviewing, Chris! /Claes

Re: RFR: 8227720: Improve ExtendedSocketOptions initialization

2019-07-16 Thread Chris Hegarty
Claes, > On 16 Jul 2019, at 15:06, Claes Redestad wrote: > > Hi, > > a small cleanup and startup optimization for apps that setup and/or use > sockets of various kinds, which pulls in ExtendedSocketOptions. By being > slightly more eager we avoid a few potentially costly streams (featuring > ca