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

Alan Bateman Alan.Bateman at oracle.com
Thu May 9 09:52:17 UTC 2019


On 09/05/2019 10:21, Daniel Fuchs wrote:
> On 09/05/2019 08:09, Alan Bateman wrote:
>>> http://cr.openjdk.java.net/~dfuchs/webrev_8223573/webrev.00/
>> I skimmed through the diffs and they look okay with me. Minor nit in 
>> 6550798/test.java is there is a space in "HttpServer.create (". Also 
>> wonder if we should rename this source file too.
>
> Thanks Alan!
>
> I'll grant you that the name of that test is not ideal.
> But since it's not a new test I think I'll prefer to keep its original
> name. I have removed the offending spaces - at several places
> in that test.
Thanks, I see there is no more at L48 (you've removed one of them but 
there is a second).

In any case, the updated webrev looks good.

-Alan.


More information about the net-dev mailing list