Looks good! Thanks, /Staffan
> On 22 apr 2015, at 11:17, Yekaterina Kantserova > <yekaterina.kantser...@oracle.com> wrote: > > Hi, > > Could I please have a review of this fix. > > bug: https://bugs.openjdk.java.net/browse/JDK-8059047 > webrev: http://cr.openjdk.java.net/~ykantser/8059047/webrev.00/ > > This fix is a part of JEP 241: Remove the jhat Tool > (https://bugs.openjdk.java.net/browse/JDK-8059039). I suggest to put > parser/validator into common test library since the functionality can be > useful not only for SVC tools tests but even for some future GC tests. > > The old jhat packages have been moved as follows: > com.sun.tools.hat.internal.model -> jdk.test.lib.hprof.model > com.sun.tools.hat.internal.parser -> jdk.test.lib.hprof.parser > com.sun.tools.hat.internal.util -> jdk.test.lib.hprof.util > > The source has not been changed except Copyrights year. > > Thanks, > Katja