Looks good to me.
/Erik
On 2015-04-23 05:15, Alan Bateman wrote:
On 23/04/2015 13:06, Staffan Larsen wrote:
Please review this change to remove the jhat tool. I will not push
this change until the tests have been removed (see a different review
thread).
JEP: https://bugs.openjdk.java.net/browse/JDK-8059039
bug: https://bugs.openjdk.java.net/browse/JDK-8076473
webrev root: http://cr.openjdk.java.net/~sla/8076473/webrev.root.00/
<http://cr.openjdk.java.net/%7Esla/8076473/webrev.root.00/>
webrev jdk: http://cr.openjdk.java.net/~sla/8076473/webrev.jdk.00/
<http://cr.openjdk.java.net/%7Esla/8076473/webrev.jdk.00/>
This looks good to me, and I assume Katja's patch from yesterday got
all the tests.
-Alan.
