Re: RFR: 8292177: InitialSecurityProperty JFR event [v4]

2022-10-11 Thread Andrey Turbanov
On Mon, 10 Oct 2022 19:23:51 GMT, Sean Coffey  wrote:

>> New JFR event to record state of initial security properties.
>>  
>> Debug output is also now added for these properties via 
>> -Djava.security.debug=properties
>
> Sean Coffey has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Address Oct 10 review comments

test/lib/jdk/test/lib/jfr/EventNames.java line 195:

> 193: public final static String X509Certificate = PREFIX + 
> "X509Certificate";
> 194: public final static String X509Validation = PREFIX + 
> "X509Validation";
> 195: public final static String InitialSecurityProperty = PREFIX + 
> "InitialSecurityProperty";

Let's unify order of modifiers in this file. Some fields use `public final 
static` and some `public static final`

-

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


Re: RFR: 8292177: InitialSecurityProperty JFR event [v4]

2022-10-10 Thread Sean Coffey
> New JFR event to record state of initial security properties.
>  
> Debug output is also now added for these properties via 
> -Djava.security.debug=properties

Sean Coffey has updated the pull request incrementally with one additional 
commit since the last revision:

  Address Oct 10 review comments

-

Changes:
  - all: https://git.openjdk.org/jdk/pull/10394/files
  - new: https://git.openjdk.org/jdk/pull/10394/files/f8faecf4..323938d9

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk=10394=03
 - incr: https://webrevs.openjdk.org/?repo=jdk=10394=02-03

  Stats: 22 lines in 6 files changed: 9 ins; 3 del; 10 mod
  Patch: https://git.openjdk.org/jdk/pull/10394.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10394/head:pull/10394

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