> On 1 Apr 2020, at 17:51, Alan Bateman <alan.bate...@oracle.com> wrote:
> 
> On 01/04/2020 16:04, Chris Hegarty wrote:
>> :
>> 
>> Updated webrev:
>> https://cr.openjdk.java.net/~chegar/8241988/webrev.01/
>> 
> Looks good although I can't immediately see why options is package private

It can be private. I’ll make that change before pushing.

> I assume the tests runs in /othervm because of the 
> setDatagramSocketImplFactory.

Correct.

-Chris.

Reply via email to