<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi <div class=""><br class=""></div><div class="">Could you please review following fix. I think that fix could be considered as trivial. So I would like to push it once I get Review to avoid failures in tier3.</div><div class="">bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8209920" class="">https://bugs.openjdk.java.net/browse/JDK-8209920</a></div><div class="">diff:</div><div class=""><br class=""></div><div class=""><div class=""><font face="Menlo" class="">diff -r 9d01ad46daef test/hotspot/jtreg/runtime/logging/RedefineClasses.java</font></div><div class=""><font face="Menlo" class="">--- a/test/hotspot/jtreg/runtime/logging/RedefineClasses.java Fri Aug 24 15:49:21 2018 -0400</font></div><div class=""><font face="Menlo" class="">+++ b/test/hotspot/jtreg/runtime/logging/RedefineClasses.java Fri Aug 24 13:23:17 2018 -0700</font></div><div class=""><font face="Menlo" class="">@@ -26,6 +26,7 @@</font></div><div class=""><font face="Menlo" class=""> * @bug 8197901 8209758</font></div><div class=""><font face="Menlo" class=""> * @summary Redefine classes with enabling logging to verify Klass:external_name() during GC.</font></div><div class=""><font face="Menlo" class=""> * @comment This test is simplified version of runtime/RedefineTests/RedefineRunningMethods.java.</font></div><div class=""><font face="Menlo" class="">+ * @requires vm.opt.final.ClassUnloading</font></div><div class=""><font face="Menlo" class=""> * @library /test/lib</font></div><div class=""><font face="Menlo" class=""> * @modules java.compiler</font></div><div class=""><font face="Menlo" class=""> * java.instrument</font></div></div><div class=""><font face="Menlo" class=""><br class=""></font></div><div class=""><font face="Menlo" class=""><br class=""></font></div><div class="">Leonid</div></body></html>