[8u backport] Request for review: 8133193: Memory leak in G1 because G1RootProcessor doesn't have desctructor

[Mikael Gerdin]

Thanks for the reviews Jesper and Jon.


On 2015-09-29 18:36, Jesper Wilhelmsson wrote:
> Looks good.
>
> To push to 8u you also need to request approval on hotspot-dev. There is
> a template here:

According to the template the correct list is jdk8u-dev, I'll send the 
mail there.

/Mikael

> http://openjdk.java.net/projects/jdk8u/approval-template.html
>
> /Jesper
>
> Den 29/9/15 kl. 17:31, skrev Mikael Gerdin:
>> Hi all,
>>
>> Please review the backport of 8133193.
>> The backport didn't apply cleanly due to some changes in the
>> surrounding code.
>>
>> Webrev: http://cr.openjdk.java.net/~mgerdin/8133193_8u/webrev/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8133193
>>
>> JDK 9 Webrev: http://cr.openjdk.java.net/~david/JDK-8133193/webrev.00
>> JDK 9 Review Thread:
>> http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2015-August/014387.html
>>
>>
>> I also created a unified diff of the different changesets at
>> http://cr.openjdk.java.net/~mgerdin/8133193_8u/diffdiff
>>
>> Thanks
>> /Mikael