<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Hi,</div><div class=""><br class=""></div><div class="">please review the following fix for the test bug.</div><div class=""><br class=""></div><div class="">Issue: tests incorrectly set inlining state for methods, that belong to any of Internal subclass of both pool.sub.Klass and pool.subpack.KlassDup. </div><div class="">This happen because test have an assumption that any of method callers will match only *.* directive pattern. But they could match patterns like “*Internal*”, because</div><div class="">a typical method caller in this case could be <span style="color: rgb(51, 51, 51); font-family: sans-serif; font-size: 14px; widows: 1; background-color: rgb(245, 245, 245);" class=""> </span><span class="">pool/sub/Klass$Internal::lambda$getAllMethods$0. </span></div><div class=""><span class=""><br class=""></span></div><div class="">Fix: Make method callers (lambdas) do not contain any names used in the test, such as Internal, or Klass. That’s why all executable and callable creation was moved to a new SubMethodHolder class.</div><div class=""><br class=""></div><div class="">bug id: <a href="https://bugs.openjdk.java.net/browse/JDK-8145800" class="">https://bugs.openjdk.java.net/browse/JDK-8145800</a></div><div class="">webrev: <a href="http://cr.openjdk.java.net/~ppunegov/8145800/webrev.00/" class="">http://cr.openjdk.java.net/~ppunegov/8145800/webrev.00/</a></div><div class=""><br class=""></div><div class="">
— Thanks,<br class="">Pavel Punegov
</div>
<br class=""></body></html>