RFR (S): 8176831: Dead code: function jmm_GetLoadedClasses is not used in jmm_interface
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
>>> 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
>>>
>>>
>>>
>>>
>
More information about the serviceability-dev
mailing list