<div dir="ltr"><div dir="ltr">Hi all,<div><br></div><div>I pushed the webrev into the submit repo and it failed to build due to internal tests that have references to the NSK_CPP_STUB. Here is the new webrev:</div><div><br></div><div>Webrev: <a href="http://cr.openjdk.java.net/~jcbeyler/8212148/webrev.01/">http://cr.openjdk.java.net/~jcbeyler/8212148/webrev.01/</a></div><div>Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8212148" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8212148</a></div><br class="gmail-Apple-interchange-newline"></div><div>The only difference is I reverted the changes of the nsk_tools.h file which have the NSK_CPP_STUB. That file now contains the only references to the NSK_CPP_STUB macros until internally there are no references to them. Then we can remove the macros from the header file.</div><div><br></div><div>Could I please get a new set of LGTM?</div><div><br></div><div>(I've re-submitted this to the submit repo and am awaiting results)</div><div><br></div><div>Thanks,</div><div>Jc</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 19, 2018 at 12:02 PM Alex Menkov <<a href="mailto:alexey.menkov@oracle.com">alexey.menkov@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1<br>
<br>
--alex<br>
<br>
On 10/19/2018 11:07, Hohensee, Paul wrote:<br>
> Looks good.<br>
> <br>
> Paul<br>
> <br>
> *From: *serviceability-dev <<a href="mailto:serviceability-dev-bounces@openjdk.java.net" target="_blank">serviceability-dev-bounces@openjdk.java.net</a>> <br>
> on behalf of JC Beyler <<a href="mailto:jcbeyler@google.com" target="_blank">jcbeyler@google.com</a>><br>
> *Date: *Thursday, October 18, 2018 at 9:44 PM<br>
> *To: *"<a href="mailto:serviceability-dev@openjdk.java.net" target="_blank">serviceability-dev@openjdk.java.net</a>" <br>
> <<a href="mailto:serviceability-dev@openjdk.java.net" target="_blank">serviceability-dev@openjdk.java.net</a>><br>
> *Subject: *RFR (M) 8212148: Remove remaining NSK_CPP_STUBs<br>
> <br>
> Hi all,<br>
> <br>
> Here is the final webrev to remove the NSK_CPP_STUB! Please review the <br>
> end of NSK_CPP_STUB.<br>
> <br>
> Webrev: <a href="http://cr.openjdk.java.net/~jcbeyler/8212148/webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~jcbeyler/8212148/webrev.00/</a> <br>
> <<a href="http://cr.openjdk.java.net/%7Ejcbeyler/8212148/webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/%7Ejcbeyler/8212148/webrev.00/</a>><br>
> <br>
> Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8212148" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8212148</a><br>
> <br>
> I've tested the whole vmTestbase since this changed a few shared files. <br>
> The whole suite passed on my dev machine.<br>
> <br>
> Let me know what you think,<br>
> <br>
> Jc<br>
> <br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><br></div>Thanks,<div>Jc</div></div></div>