RFR 8201510 : Merge TwoStacksPlainSocketImpl into DualStackPlainSocketImpl [win]

Chris Hegarty chris.hegarty at oracle.com
Wed Apr 18 16:06:44 UTC 2018


Ivan,

On 16/04/18 17:29, Ivan Gerasimov wrote:
> ...
> WEBREV: http://cr.openjdk.java.net/~igerasim/8201510/00/webrev/

I think this is mostly good. Just one comment.


I'm not sure that this is correct.

--- OLD ---

   60         String exclBindProp = AccessController.doPrivileged(
   61                 new GetPropertyAction("sun.net.useExclusiveBind", 
""));
   62         exclusiveBind = (exclBindProp.isEmpty())
   63                 ? true
   64                 : Boolean.parseBoolean(exclBindProp);

--- NEW ---
  private static final boolean useExclusiveBind =
   55             Boolean.parseBoolean(AccessController.doPrivileged(
   56                 new GetPropertyAction("sun.net.useExclusiveBind", 
"true")));

Exclusive bind should be true iif:
   1) it is defined and has no value, or
   2) if is defined and has a value of `true`.

I thought we had tests for this, but maybe not if you are not
seeing test failures.

-Chris.


More information about the net-dev mailing list