Looks good. Thanks, /Staffan
> On 23 aug. 2016, at 19:36, Jini Susan George <jini.geo...@oracle.com> wrote: > > I need one more reviewer, please ! > > Thanks, > Jini. > >> -----Original Message----- >> From: Jini Susan George >> Sent: Tuesday, August 23, 2016 11:48 AM >> To: Dmitry Samersoff; serviceability-dev@openjdk.java.net >> Subject: RE: RFR: 8163150: SA: CLHSDB printmdo throws an exception with >> "java.lang.InternalError: missing reason for 22" >> >> Thank you, Dmitry ! >> >> -jini >> >>> -----Original Message----- >>> From: Dmitry Samersoff >>> Sent: Tuesday, August 23, 2016 11:44 AM >>> To: Jini Susan George; serviceability-dev@openjdk.java.net >>> Subject: Re: RFR: 8163150: SA: CLHSDB printmdo throws an exception with >>> "java.lang.InternalError: missing reason for 22" >>> >>> Jini, >>> >>> Looks good for me. >>> >>> -Dmitry >>> >>> On 2016-08-22 06:29, Jini Susan George wrote: >>>> Hello, >>>> >>>> >>>> >>>> Could I please get a review for: >>>> >>>> >>>> >>>> https://bugs.openjdk.java.net/browse/JDK-8163150 >>>> >>>> >>>> >>>> Webrev: http://cr.openjdk.java.net/~jgeorge/8163150/webrev.00/ >>>> >>>> >>>> >>>> The test BasicLauncherTest.java includes code leveraged from >> Yasumasa’s >>>> webrev for JDK-8163581. >>>> >>>> >>>> >>>> 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.
