On Fri, 14 May 2021 00:15:09 GMT, Xue-Lei Andrew Fan <[email protected]> wrote:

>> Valerie Peng has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   PKCS11Exception class refactoring - use enum instead of HashMap
>
> src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/wrapper/PKCS11Exception.java
>  line 212:
> 
>> 210:         long value() {
>> 211:             return value;
>> 212:         }
> 
> I guess it is safe to use the 'value' variable directly for enum.  The 
> value() method may be not necessary.  Just a personal preference, ignore it 
> if you like the current way.

Sure, I like it too. Will update.

-------------

PR: https://git.openjdk.java.net/jdk/pull/3709

Reply via email to