JDK 9 RFR of test-only 8180085: (ch) java/nio/channels/SocketChannel/VectorIO.java: add debug instrumentation

Brian Burkhalter brian.burkhalter at oracle.com
Thu May 11 14:23:50 UTC 2017


On May 10, 2017, at 11:52 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:

>> Please review at your convenience.
>> 
>> Issue:	https://bugs.openjdk.java.net/browse/JDK-8180085
>> Patch:	http://cr.openjdk.java.net/~bpb/8180085/webrev.00/
>> 
>> Add some print statements potentially useful in solvinghttps://bugs.openjdk.java.net/browse/JDK-8177801.
>> 
> Looks okay. Style-wise then I guess I would be braces when added the then-else but that is a minor point in the bug for the intermittent test failures.

I prefer that style as well but was retaining the existing style in the test. Might change to that before pushing.

Thanks,

Brian
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20170511/3453f995/attachment-0001.html>


More information about the nio-dev mailing list