RFR: JDK-7107018: sun.jvm.hotspot.utilities.soql.JSJavaHeap.forEachClass incorrect test

Jini Susan George jini.george at oracle.com
Tue Nov 8 10:56:05 UTC 2016


Thank you, Robbin ! 

-jini

> -----Original Message-----
> From: Robbin Ehn
> Sent: Tuesday, November 08, 2016 4:02 PM
> To: Jini Susan George; serviceability-dev at openjdk.java.net
> Subject: Re: RFR: JDK-7107018:
> sun.jvm.hotspot.utilities.soql.JSJavaHeap.forEachClass incorrect test
> 
> Hi Jini,
> 
> Looks okay!
> 
> I personally would prefer:
> if (k != null && l != null) {
> 
> But I'll leave that up to you.
> 
> Thanks!
> 
> /Robbin ('r'eviewer)
> 
> On 11/03/2016 05:04 AM, Jini Susan George wrote:
> > Please review the trivial fix for the bug below (Unfortunately, the bug
> > is marked confidential):
> >
> >
> >
> > https://bugs.openjdk.java.net/browse/JDK-7107018
> >
> >
> >
> > Webrev:
> http://cr.openjdk.java.net/~jgeorge/7107018/webrev.00/index.html
> >
> >
> >
> > Thanks,
> >
> > - Jini Susan George
> >
> >
> >


More information about the serviceability-dev mailing list