Looks good! Sorry for taking so long to review. ./test/lib/testlibrary/jdk/testlibrary/Utils.java L78: nit: missing spaces around = and <
/Staffan On 08 Nov 2013, at 15:48, Mattias Tobiasson <mattias.tobias...@oracle.com> wrote: > Hi, > Could you please review this fix. > The tests for java/lang/instrument/PremainClass/PremainAgent has been ported > from script to java. > The purpose is to make the tests more stable and easier to maintain. > > Thanks, > Mattias > > Webrev: > http://cr.openjdk.java.net/~ykantser/8023138/webrev.00/ > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8023138