Code review request, CR 6996372: synchronizing handshaking hash
weijun.wang at oracle.com
Thu Apr 26 23:49:13 PDT 2012
It seems people normally put "static" before "final", but since there
are already 2 "final static" there, you're free to do anything.
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:
More information about the security-dev