On Sat, 31 Oct 2020 03:28:24 GMT, Weijun Wang <wei...@openjdk.org> wrote:

> 
> 
> > > The constants in PKCS11Exception are duplicated in PKCS11Constants.
> > > ```
> > >             0x00000000,
> > > ```
> > > 
> > > 
> > > vs
> > > ```
> > > public static final long  CKR_OK                 = 0x00000000L;
> > > ```
> > > 
> > > 
> > > Is there any way to simplify it?
> > 
> > 
> > One defines the value, the other defines the displayed String. I agree that 
> > the way it's currently done is a bit outdated and may be error prone. We 
> > can re-visit this after finishing other pending PKCS11 RFEs..
> 
> Since the value is already defined PKCS11Constants, we can use it in 
> PKCSException and does not need to write 0x00000000 anymore.

Ok, I see what you are saying now. Will change it to use the PKCS11Constants 
constants.

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

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

Reply via email to