Hi Alan,
Thank you for looking at it! When JEP was created the work has been
divided in product and test parts according to the JEP guidelines.
That's the reason I can come up with.
To test part belongs even
https://bugs.openjdk.java.net/browse/JDK-8059047. That's would be great
if you can take a look at it as well.
// Katja
On 04/22/2015 01:09 PM, Alan Bateman wrote:
On 22/04/2015 11:56, Yekaterina Kantserova wrote:
Hi,
Could I please have a review of this fix.
bug: https://bugs.openjdk.java.net/browse/JDK-8076524
webrev: http://cr.openjdk.java.net/~ykantser/8076524/webrev.00
This fix is a part of JEP 241: Remove the jhat Tool
(https://bugs.openjdk.java.net/browse/JDK-8059039).
This looks okay to me although I would have expected these tests to be
removed as part of the change to remove jhat (no issue with it being
split into multiple changes, just wondering why it is being done this
way).
-Alan
