+1

--alex

On 10/19/2018 11:07, Hohensee, Paul wrote:
Looks good.

Paul

*From: *serviceability-dev <serviceability-dev-boun...@openjdk.java.net> on behalf of JC Beyler <jcbey...@google.com>
*Date: *Thursday, October 18, 2018 at 9:44 PM
*To: *"serviceability-dev@openjdk.java.net" <serviceability-dev@openjdk.java.net>
*Subject: *RFR (M) 8212148: Remove remaining NSK_CPP_STUBs

Hi all,

Here is the final webrev to remove the NSK_CPP_STUB! Please review the end of NSK_CPP_STUB.

Webrev: http://cr.openjdk.java.net/~jcbeyler/8212148/webrev.00/ <http://cr.openjdk.java.net/%7Ejcbeyler/8212148/webrev.00/>

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

I've tested the whole vmTestbase since this changed a few shared files. The whole suite passed on my dev machine.

Let me know what you think,

Jc

Reply via email to