Code review request, CR 6996372: synchronizing handshaking hash

Weijun Wang weijun.wang at oracle.com
Fri Apr 27 06:49:13 UTC 2012


Looks fine.

It seems people normally put "static" before "final", but since there 
are already 2 "final static" there, you're free to do anything.

Thanks
Max

On 04/27/2012 12:07 PM, Xuelei Fan wrote:
> On 4/27/2012 11:46 AM, David Schlosnagle wrote:
>> On Apr 26, 2012, at 10:07 PM, Xuelei Fan<xuelei.fan at oracle.com>  wrote:
>>
>>> webrev: http://cr.openjdk.java.net/~xuelei/6996372/webrev.00/
>>
>> It might be worthwhile to add a comment that both disabledAlgorithmsMap and keySizeConstraintsMap are locked by the disabledAlgorithmsMap instance. Also, is there any reason not to make both of those static maps final?
>>
> Good suggestion.  See the updated webrev:
>     http://cr.openjdk.java.net/~xuelei/6996372/webrev.01/
>
> Thanks,
> Xuelei
>
>> Thanks,
>> Dave
>



More information about the security-dev mailing list