Hi Yasumasa, +1
Thanks, Serguei On 7/17/19 09:11, Chris Plummer wrote:
Looks good. Chris On 7/17/19 7:37 AM, Yasumasa Suenaga wrote:Hi all, Please review this change: JBS: https://bugs.openjdk.java.net/browse/JDK-8227815 webrev: http://cr.openjdk.java.net/~ysuenaga/JDK-8227815/webrev.00/ After JDK-8225690, minimal VM could not be built.Minimal VM does not include Attach Listener implementation. So I exclude it in os.cpp with INCLUDE_SERVICES macro.Thanks, Yasumasa