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.
Meanwhile I also found two other tests that needed updating:
test/jdk/sun/net/www/http/ChunkedInputStream/ChunkedEncodingWithProgressMonitorTest.java
test/jdk/sun/net/www/http/ChunkedInputStream/TestAvailable.java
I've added them to the webrev:
http://cr.openjdk.java.net/~dfuchs/webrev_8223573/webrev.01/
best regards,
-- daniel
