Looks ok to me too. Brad
On 9/18/2012 5:49 AM, Sean Mullan wrote:
Looks good to me. Don't forget to add the noreg-trivial keyword to the CR before pushing. --Sean On 9/17/12 9:38 PM, Xuelei Fan wrote:Hi, Please review the trivial fix of a typo in internal method name. The bug has not been shown in bug database. The method name, "getSuportedCipherSuiteList", of SSLContextImpl should be "getSupportedCipherSuiteList" webrev: http://cr.openjdk.java.net/~xuelei/7199066/webrev.00/ Thanks, Xuelei