> 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.