On 20/08/2014 16:08, Brian Burkhalter wrote:
JDK 9 reviewers:
I would like to reprise this thread
http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-August/028181.html
with an updated patch
Issue: https://bugs.openjdk.java.net/browse/JDK-8054720
Patch: http://cr.openjdk.java.net/~bpb/8054720/webrev.01/
This changes builds on all platforms and passes the jdk_io and jdk_net tests on
all platforms.
Note that the patch posted in the original discussion thread has been moved to
http://cr.openjdk.java.net/~bpb/8054720/webrev.00/
Thanks for dropping SSLSocketImpl, it looks much better now.
The new comments "wrap native call to allow instrumentation" seem to be
added in really odd place, can't you just use @implNote?
-Alan.
