Hi Felix, looks ok to me, though I'm no reviewer.
One nit: test/java/net/MulticastSocket/TimeToLive.java, line 40, you could remove the ';' from the try statement. Best regards Christoph > -----Original Message----- > From: net-dev [mailto:net-dev-boun...@openjdk.java.net] On Behalf Of Felix > Yang > Sent: Montag, 26. September 2016 09:57 > To: net-dev@openjdk.java.net > Subject: RFR 8085575_8130657, misc fixes for a few of java.net intermittent > failures > > Hi there, > > please review following patch to a few of java.net tests. > > Bug: > > https://bugs.openjdk.java.net/browse/JDK-8085575 > > https://bugs.openjdk.java.net/browse/JDK-8130657 > > Webrev: > > http://cr.openjdk.java.net/~xiaofeya/8085575_8130657/webrev.00/ > > Add retry for java/net/Socket/InheritHandle.java. Though it may be > unable to resolve all BindException ( I suppose it is the nature of such > close-reuse scenarios), it will be helpful to avoid failures from > asynchronized close, which has been observed especially on Windows. Also > some misc changes on other two tests. > > > Thanks, > > Felix
