RFR JMC-6075: Avoid conversion for RECORDING_SETTING event from JDK9/10 in SettingsTransformer

Jayathirth D V jayathirth.d.v at oracle.com
Fri Jul 20 08:04:55 UTC 2018


Hello All,

 

Please review the following fix in JMC 7 :

 

Bug : https://bugs.openjdk.java.net/browse/JMC-6075 

Webrev : http://cr.openjdk.java.net/~jdv/6075/webrev.00/

 

Issue: In SettingTransformer and SyntheticAttributeExtension, whenever we have RECORDING_SETTING from JDK9/10 we translate these events to JDK11 type.

 

Solution: We have static variables which we use for Pre JDK9 events in JdkTypeIDsPreJdk11. For JDK9/10 RECORDING_SETTING event type also we can maintain a static variable in JdkTypeIDsPreJdk11 and use it wherever applicable. This will avoid conversion of JDK9/10 RECORDING_SETTING event to JDK11 type. I have removed usage of JdkTypeIDsPreJdk11.needTransform() because we will be creating SettingTransform object only when have Pre-JDK 11 RECORDING_SETTING event type. Also there is some cleanup done in  SettingTransformer.

 

I have ran unit tests and there are no failures. Also used JDK 7/8/9/10/11 recordings and I don't see any difference in Automated-analysis results before and after change.

 

Thanks,

Jay

 


More information about the jmc-dev mailing list