On Thu, 8 Apr 2021 17:02:20 GMT, Sean Mullan <mul...@openjdk.org> wrote:

>> 8264681: Use the blessed modifier order in java.security
>
> src/java.xml.crypto/share/classes/org/jcp/xml/dsig/internal/dom/DOMSignatureMethod.java
>  line 390:
> 
>> 388:         }
>> 389: 
>> 390:         public abstract PSSParameterSpec getPSSParameterSpec();
> 
> This is 3rd-party (Apache) code. It would be better to change this at Apache. 
> I would leave this one out.

@alblue, this is the unresolved comment. Please fix it?

> src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/wrapper/Constants.java
>  line 53:
> 
>> 51: 
>> 52: /**
>> 53:  * This class holds only static final member variables that are constants
> 
> Probably best to leave this file unchanged as it is 3rd-party code.

Same here...

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

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

Reply via email to