Re: Code review request, CR 7153184: NullPointerException when calling SSLEngineImpl.getSupportedCipherSuites

[Weijun Wang]

The fix is good, but I think you are over-commenting. Everyone seeing 
the synchronized keyword knows what it means. You can keep the new lines 
at 380-381.

Thanks
Max
On 05/04/2012 12:37 PM, Xuelei Fan wrote:
Hi,

Please review the synchronization issue in SSLContextImpl.

bug detail: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7153184
webrev: http://cr.openjdk.java.net/~xuelei/7153184/webrev.00/

No new regression test, simple fix and hard to reproduce the issue.

Thanks,
Xuelei