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.
