RFR: JDK-8159127: hprof heap dumps broken for lambda classdata
serguei.spitsyn at oracle.com
serguei.spitsyn at oracle.com
Thu Dec 15 09:27:56 UTC 2016
Jini,
It looks good.
Thank you for the update!
Serguei
On 12/15/16 00:57, Jini Susan George wrote:
> Thank you, Serguei for the review. Comments inline.
>
>> -----Original Message-----
>>
>>
>> It seems there is no need to check address for null as it is done
>> inside the instantiateWrapperFor call.
> Makes sense. I have removed those.
>
>> At the lines 522-547 it is possible to define and use the same callback class
>> for traversing both the SystemDictionary classes and the anonymous classes.
>> The same is applied to the lines 961-990.
>> But I leave it up to you to decide if it needs to be updated.
> I have left this as it is. The modified webrev is at:
>
> http://cr.openjdk.java.net/~jgeorge/8159127/webrev.02/
>
> Thanks,
> Jini.
More information about the serviceability-dev
mailing list