Looks good! Thanks, /Staffan
On 18 okt 2013, at 15:41, Fredrik Arvidsson <fredrik.arvids...@oracle.com> wrote: > Hi > > Please help me with this tiny review: > > Webrev: http://cr.openjdk.java.net/~farvidsson/8026808/webrev.00/index.html > <http://cr.openjdk.java.net/%7Efarvidsson/8026808/webrev.00/index.html> > Bug: https://bugs.openjdk.java.net/browse/JDK-8026808 > > About this bug: > This bug was revealed by another test-fix: > https://bugs.openjdk.java.net/browse/JDK-8025638. The JDKToolLauncher have > probably never worked in Solaris 64 environments. > > JTReg tests run in JPRT: > http://sthjprt.se.oracle.com/archives/2013/10/2013-10-18-094943.farvidss.hotspot/ > verified successful. > > Cheers > /F