Re: [teststabilization] RFR: 8226825: Replace wildcard address with loopback or local host in tests - part 19

2019-06-27 Thread Chris Hegarty
> On 27 Jun 2019, at 14:57, Julia Boes wrote: > > Hi, > > Thanks Brian and Chris for your comments! I incorporated the changes. > > Here's the updated webrev: > > http://cr.openjdk.java.net/~dfuchs/jboes/webrev_8226825/webrev.01/ Thanks for the updated webrev, Julia. You have my Review.

Re: [teststabilization] RFR: 8226825: Replace wildcard address with loopback or local host in tests - part 19

2019-06-27 Thread Julia Boes
Hi, Thanks Brian and Chris for your comments! I incorporated the changes. Here's the updated webrev: http://cr.openjdk.java.net/~dfuchs/jboes/webrev_8226825/webrev.01/ Cheers, Julia On 26/06/2019 21:32, Chris Hegarty wrote: Julia, On 26 Jun 2019, at 18:33, Julia Boes wrote: Hi,

Re: [teststabilization] RFR: 8226825: Replace wildcard address with loopback or local host in tests - part 19

2019-06-26 Thread Chris Hegarty
Julia, > On 26 Jun 2019, at 18:33, Julia Boes wrote: > > Hi, > > Please find below a patch for: > > 8226825: Replace wildcard address with loopback or local host in tests - part > 19 > > https://bugs.openjdk.java.net/browse/JDK-8226825 > > > webrev: > >

Re: [teststabilization] RFR: 8226825: Replace wildcard address with loopback or local host in tests - part 19

2019-06-26 Thread Brian Burkhalter
Hi Julia, (part 19 (!)) In AsyncDisconnect line 57 could be replaced with ioe.printStackTrace() to be consistent with lines 85 and 88. Same thing for B6641309 line 58, and B6660405 line 108. I’ll leave comments on the more substantive changes to the net-dev experts. Thanks, Brian > On Jun

[teststabilization] RFR: 8226825: Replace wildcard address with loopback or local host in tests - part 19

2019-06-26 Thread Julia Boes
Hi, Please find below a patch for: 8226825: Replace wildcard address with loopback or local host in tests - part 19 https://bugs.openjdk.java.net/browse/JDK-8226825 webrev: http://cr.openjdk.java.net/~dfuchs/jboes/webrev_8226825/webrev/ - Replaced wildcard address with loopback or