Thanks Sean and Xuelei

I filed new bug for adding debug option,see https://bugs.openjdk.java.net/browse/JDK-8158620 The codes were also updated follow Sean's suggestion at here: http://cr.openjdk.java.net/~tidu/8158620/webrev.00/
Please help to review again.Thanks.

Regards
Tim
On 2016/6/3 4:38, Sean Mullan wrote:
On 06/02/2016 06:16 AM, Tim Du wrote:
Hi,

Please help to review the path for JDK-8143305

TestEC.java test for ECC algorithms in SSL/TLS with different cipher
suites. It failed with javax.net.ssl.SSLException: Unsupported or
unrecognized SSL message intermittently.

The exception was threw at step of handshaking, I was not able to
reproduce this failures. Suggesting to enable the debug option,so that
it could print more information to logs. Hope get helpful information
and do future evaluation in next failures.

I think a new bug should be opened to add the debug option, since the intermittent failure has not been resolved.


JBS: https://bugs.openjdk.java.net/browse/JDK-8143305
webrev: http://cr.openjdk.java.net/~tidu/8143305/webrev.01/

Can you add braces around the if statement on line 74-5? This is preferred style to accidentally avoid bracing mistakes. See also http://cr.openjdk.java.net/~alundblad/styleguide/index-v6.html#toc-braces

--Sean

Reply via email to