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