On Mon, 26 Sep 2022 13:45:45 GMT, Sean Coffey <coff...@openjdk.org> wrote:

> By moving the JFR event up to the java.security.cert.CertificateFactory 
> class, we can record all generate cert events, including those from 3rd party 
> providers. I've also altered the logic so that an event is genertate for 
> every generate cert call (not just ones missing from the JDK provider 
> implementation cache)
> 
> test case also updated to capture new logic

src/java.base/share/classes/jdk/internal/event/X509CertificateEvent.java line 
34:

> 32: 
> 33: public final class X509CertificateEvent extends Event {
> 34:     private final static X509CertificateEvent EVENT = new 
> X509CertificateEvent();

Use blessed modifiers order
Suggestion:

    private static final X509CertificateEvent EVENT = new 
X509CertificateEvent();

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

PR: https://git.openjdk.org/jdk/pull/10422

Reply via email to