On 8 Mar 2016, at 20:49, Michael McMahon <michael.x.mcma...@oracle.com> wrote:

> This is a small change, where an internal completion result
> was getting lost.
> 
> http://cr.openjdk.java.net/~michaelm/8151441/webrev.1/

The source changes look ok, maybe just line up the dots of the
method invocations of sendBodyAsync and thenCompose.

Can you update the bug synopsis to reflect the actual change, it 
looks like a test only issue ( which it is not ).

In the test you can replace read(InputStream) with is.readAllBytes.

There are also a number of places where you can/should close I/O
streams/sockets with try-with-resources.

-Chris.

Reply via email to