<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi all,<div><br></div><div>I am continuing the NSK_CPP_STUB removal with this next webrev.<br></div><div><div>Webrev: <a href="http://cr.openjdk.java.net/~jcbeyler/8211899/webrev.00/">http://cr.openjdk.java.net/~jcbeyler/8211899/webrev.00/</a><br></div><div>Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8211899">https://bugs.openjdk.java.net/browse/JDK-8211899</a></div></div><div><br></div><div>The change is still straight-forward though, since it is just doing the same NSK_CPP_STUB removal. However when I was looking at the changes, a lot of these changes are touching lines with spaces before/after parenthesis. I've almost never touched the spaces except if I was refactoring by hand the line at the same time. The rationale being that the lines will get fixed a few more times and are, at worse, covered by the bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8211335" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8211335</a>, which I've commited to doing. </div><div><br></div><div>Two exceptions are here where I pushed out the code into assignments due to really long lines and complex if structures:</div><div>- <a href="http://cr.openjdk.java.net/~jcbeyler/8211899/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/hotswap/HS204/hs204t003/hs204t003.cpp.udiff.html">jvmti/scenarios/hotswap/HS204/hs204t003/hs204t003.cpp</a><br></div><div>- <a href="http://cr.openjdk.java.net/~jcbeyler/8211899/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/jni_interception/JI01/ji01t001/ji01t001.cpp.udiff.html">jvmti/scenarios/jni_interception/JI01/ji01t001/ji01t001.cpp</a></div><div><br></div><div>And one exception here where a commented line was doing the out-of-if assignment so I just uncommented it and used the variable:</div><div>- <a href="http://cr.openjdk.java.net/~jcbeyler/8211899/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/hotswap/HS301/hs301t001/hs301t001.cpp.udiff.html">jvmti/scenarios/hotswap/HS301/hs301t001/hs301t001.cpp</a><br></div><div><br></div><div><div dir="ltr" class="gmail-m_-3749740427441456462gmail-m_4350138340315380472gmail-m_-5052348339937521290gmail_signature"><div dir="ltr"><div>I've tested the modified changes on my machine, all modified tests pass.<br></div><div><br></div>Let me know what you think,<div>Jc</div><div><br></div><div>Ps: 2 more of these and we can say good bye to NSK_CPP_STUB*</div><div><br></div></div></div></div></div></div></div></div></div></div>