Arthur, > On 12 Mar 2019, at 16:59, Arthur Eubanks <aeuba...@google.com> wrote: > > Thanks for creating the umbrella bug. > > Split long line. > Added the HTTPTestServer.java and UrgentDataTest.java changes in Chris's > suggestions.
Thanks. > The other 2 involve URI/URLs, and there are a lot of those changes, will > group them in a later change. That is true, and it is absolutely fine to leave them until a follow on. The reason I suggested the change is because almost all of the other HTTP Client tests run successfully on an IPv6-only environment. The reason is that the URL-string when converted to a URI has the square brackets added by URI.create. Anyway, not critical for now. > PTAL: http://cr.openjdk.java.net/~aeubanks/8220083/webrev.02/ > <http://cr.openjdk.java.net/~aeubanks/8220083/webrev.02/> Looks good. You are probably all set, but if you need a sponsor just ask. -Chris.
