Thank you, Ioi and Dmitry, for your comments. Please check the new webrev addressing these comments at:
http://cr.openjdk.java.net/~jgeorge/8027920/webrev.01/index.html Thanks, Jini. > -----Original Message----- > From: Dmitry Samersoff > Sent: Sunday, September 11, 2016 2:43 PM > To: Jini Susan George; serviceability-dev > Subject: Re: RFR: 8027920: SA: Add default methods to InstanceKlass > > Jini, > > SA changes looks good for me. > > In testcase, please use LingeredApp derived class (see > LingeredAppWithDeadlock as an example) rather that attach to self. > > -Dmitry > > On 2016-09-11 09:01, Jini Susan George wrote: > > Hello, > > > > > > > > Could I please get a review for the following fix ? > > > > > > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8027920 > > > > Webrev: http://cr.openjdk.java.net/~jgeorge/8027920/webrev.00/ > > > > > > > > Thanks, > > > > - Jini Susan George > > > > > > > > > -- > Dmitry Samersoff > Oracle Java development team, Saint Petersburg, Russia > * I would love to change the world, but they won't give me the sources.
