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, [email protected] 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, [email protected] 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
jmm_GetLoadedClasses() defined in services/management.cpp
is not really used in the jmm_interface. This function and
dead code associated with its implementation is removed.
Testing:
The nsk.monitoring and jtreg jdk_management tests are in progress.
Thanks,
Serguei
