JDK 9 RFR of 8054720: Modifications of I/O methods for instrumentation purposes
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, Brian
Hi Brian, Looks fine, thanks for the cleanup. Roger On 8/20/2014 11:08 AM, 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,
Brian
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
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.
participants (3)
-
Alan Bateman
-
Brian Burkhalter
-
roger riggs