RFR (S): 8176831: Dead code: function jmm_GetLoadedClasses is not used in jmm_interface (original) (raw)

serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Fri Mar 17 01:38:14 UTC 2017

I've reloaded the webrev with pulled latest jdk10 changes. It looks almost the same but has the recent update from Coleen.

Thanks, Serguei

On 3/16/17 18:20, serguei.spitsyn at oracle.com wrote:

David,

Thank you for the review!

On 3/16/17 18:15, David Holmes wrote: Looks good!

Don't forget to update copyright years. Yes, of course. Thanks, Serguei Thanks, David On 17/03/2017 10:59 AM, serguei.spitsyn at oracle.com wrote: Please, review the jdk 10 fix for: https://bugs.openjdk.java.net/browse/JDK-8176831

Webrev: http://cr.openjdk.java.net/~sspitsyn/webrevs/2017/hotspot/8176831-jmm-dead.1/

Summary: It was found at the review of the 8155672 that the function jmmGetLoadedClasses() defined in services/management.cpp is not really used in the jmminterface. This function and dead code associated with its implementation is removed. Testing: The nsk.monitoring and jtreg jdkmanagement tests are in progress. Thanks, Serguei

More information about the hotspot-dev mailing list