Daniel, > On 14 Aug 2020, at 16:43, Daniel Fuchs <daniel.fu...@oracle.com> wrote: > > Hi, > > After discussion with Chris & Alan I have slightly reworded > the API documentation, and ensured the MultiExchange is only > weakly referenced from the MinimalFuture to avoid the risk of > pinning the exchange in memory after it is no longer needed. > > Also I added a trivial test for the case where the thread is > interrupted just before calling HttpClient::send. > > The new webrev is at: > http://cr.openjdk.java.net/~dfuchs/webrev_8245462/webrev.03/
I provided some comments to you on a previous iteration, they seem to be addressed in this latest version. Looks good to me. -Chris.
