> On 9 Sep 2019, at 17:37, Daniel Fuchs <daniel.fu...@oracle.com> wrote: > > Thanks Chris. > > All good comments. > > new webrev at: > http://cr.openjdk.java.net/~dfuchs/webrev_8230526/webrev.01/
LGTM. > I'm not sure all httpclient tests that use jdk.httpserver have the > appropriate @module tag. But that should be for some further cleanup. Agreed. -Chris