Thumbs up!

Dan



On 2/12/17 5:19 PM, David Holmes wrote:
Planning to push under trivial rules so just need one Reviewer please.

Bug: https://bugs.openjdk.java.net/browse/JDK-8174798

Stumbled on this by accident. This test should no longer be ignored, but somehow the removal of the @ignore was lost in a Merge.

webrev: http://cr.openjdk.java.net/~dholmes/8174798/webrev/

Patch below.

Thanks,
David

--- old/test/serviceability/sa/TestCpoolForInvokeDynamic.java 2017-02-12 19:08:58.969198177 -0500 +++ new/test/serviceability/sa/TestCpoolForInvokeDynamic.java 2017-02-12 19:08:57.005084892 -0500
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2016, 2017, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -51,7 +51,6 @@
  *          jdk.hotspot.agent/sun.jvm.hotspot.oops
  *          jdk.hotspot.agent/sun.jvm.hotspot.debugger
  *          jdk.hotspot.agent/sun.jvm.hotspot.ui.classbrowser
- * @ignore 8169232
  * @run main/othervm TestCpoolForInvokeDynamic
  */


Reply via email to