<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
pre
        {mso-style-priority:99;
        mso-style-link:"HTML Preformatted Char";
        margin:0in;
        margin-bottom:.0001pt;
        font-size:10.0pt;
        font-family:"Courier New";}
p.msonormal0, li.msonormal0, div.msonormal0
        {mso-style-name:msonormal;
        mso-margin-top-alt:auto;
        margin-right:0in;
        mso-margin-bottom-alt:auto;
        margin-left:0in;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
span.EmailStyle18
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.HTMLPreformattedChar
        {mso-style-name:"HTML Preformatted Char";
        mso-style-priority:99;
        mso-style-link:"HTML Preformatted";
        font-family:"Courier New";}
span.removed
        {mso-style-name:removed;}
span.new
        {mso-style-name:new;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style>
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal">Re spaces :).  There’s a bunch of places where “(jvmti_env”  should be “( jvmti_env”, but of course not all of them. I found a bunch. Otherwise, lgtm., no need for another webrev.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Paul<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">In hs104t002.cpp<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Courier New"">-        if (! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Courier New"">-                &caps) )) {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Courier New"">+        if (! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) )) {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Courier New""><o:p> </o:p></span></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Courier New""><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Courier New"">+        if (! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) )) {<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Courier New""><o:p> </o:p></span></p>
<p class="MsoNormal">In hs203t003.cpp<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-    if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB4(GetClassSignature,</span><o:p></o:p></pre>
<pre><span class="removed">-                    jvmti_env, klass, &className, &generic) ) ) {</span><o:p></o:p></pre>
<pre><span class="new">+    if ( ! NSK_JVMTI_VERIFY (jvmti_env->GetClassSignature(klass, &className, &generic) ) ) {<o:p></o:p></span></pre>
<pre><o:p> </o:p></pre>
<pre><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">=><o:p></o:p></span></pre>
<pre><o:p> </o:p></pre>
<pre><span class="new">+    if ( ! NSK_JVMTI_VERIFY ( jvmti_env->GetClassSignature(klass, &className, &generic) ) ) {<o:p></o:p></span></pre>
<pre><o:p> </o:p></pre>
<pre><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">and<o:p></o:p></span></pre>
<pre><o:p> </o:p></pre>
<pre><span class="removed">-        if (! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti, &caps) )) {</span><o:p></o:p></pre>
<pre><span class="new">+        if (! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) )) {</span><o:p></o:p></pre>
<pre><o:p> </o:p></pre>
<pre><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">=><o:p></o:p></span></pre>
<pre><o:p> </o:p></pre>
<pre><span class="new">+        if (! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) )) {</span><o:p></o:p></pre>
<pre><o:p> </o:p></pre>
<pre><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">and<o:p></o:p></span></pre>
<pre><o:p> </o:p></pre>
<pre><span class="removed">-    if ( ! NSK_JVMTI_VERIFY( NSK_CPP_STUB3(GetThreadState, jvmti, thread, &state) )  ) {</span><o:p></o:p></pre>
<pre><span class="new">+    if ( ! NSK_JVMTI_VERIFY(jvmti->GetThreadState(thread, &state) )  ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+    if ( ! NSK_JVMTI_VERIFY( jvmti->GetThreadState(thread, &state) )  ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">and<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-    if ( ! NSK_JVMTI_VERIFY( NSK_CPP_STUB3(GetThreadState, jvmti, thread, &state) )  ) {</span><o:p></o:p></pre>
<pre><span class="new">+    if ( ! NSK_JVMTI_VERIFY(jvmti->GetThreadState(thread, &state) )  ) {</span><o:p></o:p></pre>
<pre>         nsk_printf(" Agent :: Error while getting thread state.\n");<o:p></o:p></pre>
<pre>         nsk_jvmti_agentFailed();<o:p></o:p></pre>
<pre>     } else {<o:p></o:p></pre>
<pre>         if ( state & JVMTI_THREAD_STATE_SUSPENDED) {<o:p></o:p></pre>
<pre><span class="removed">-            if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2( PopFrame, jvmti, thread) ) ) {</span><o:p></o:p></pre>
<pre><span class="new">+            if ( ! NSK_JVMTI_VERIFY (jvmti->PopFrame(thread) ) ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+    if ( ! NSK_JVMTI_VERIFY( jvmti->GetThreadState(thread, &state) )  ) {<o:p></o:p></span></pre>
<pre>…<o:p></o:p></pre>
<p class="MsoNormal"><span class="new"><span style="font-size:10.0pt;font-family:"Courier New"">+            if ( ! NSK_JVMTI_VERIFY ( jvmti->PopFrame(thread) ) ) {</span></span><span style="font-size:10.0pt;font-family:"Courier New""><o:p></o:p></span></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">and<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-    if ( !NSK_JVMTI_VERIFY( NSK_CPP_STUB2 ( ResumeThread, jvmti, thread)) ) {</span><o:p></o:p></pre>
<pre><span class="new">+    if ( !NSK_JVMTI_VERIFY(jvmti->ResumeThread(thread)) ) {<o:p></o:p></span></pre>
<pre><o:p> </o:p></pre>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+    if ( !NSK_JVMTI_VERIFY( jvmti->ResumeThread(thread)) ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">In hs203t004.cpp<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-                if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(GenerateEvents, jvmti_env,</span><o:p></o:p></pre>
<pre><span class="removed">-                                JVMTI_EVENT_COMPILED_METHOD_LOAD ) )) {</span><o:p></o:p></pre>
<pre><span class="new">+                if ( ! NSK_JVMTI_VERIFY (jvmti_env->GenerateEvents(JVMTI_EVENT_COMPILED_METHOD_LOAD) )) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+                if ( ! NSK_JVMTI_VERIFY ( jvmti_env->GenerateEvents(JVMTI_EVENT_COMPILED_METHOD_LOAD) )) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">and<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-        if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB3(GetMethodDeclaringClass,</span><o:p></o:p></pre>
<pre><span class="removed">-                        jvmti_env, method, &threadClass) ) ) {</span><o:p></o:p></pre>
<pre><span class="new">+        if ( ! NSK_JVMTI_VERIFY (jvmti_env->GetMethodDeclaringClass(method, &threadClass) ) ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+        if ( ! NSK_JVMTI_VERIFY ( jvmti_env->GetMethodDeclaringClass(method, &threadClass) ) ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">and<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-        if (! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti, &caps) )) {</span><o:p></o:p></pre>
<pre><span class="new">+        if (! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) )) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+        if (! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) )) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">and<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-    if (  NSK_JVMTI_VERIFY( NSK_CPP_STUB2(SuspendThread, jvmti, thread) ) ) {</span><o:p></o:p></pre>
<pre><span class="new">+    if (  NSK_JVMTI_VERIFY(jvmti->SuspendThread(thread) ) ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+    if (  NSK_JVMTI_VERIFY( jvmti->SuspendThread(thread) ) ) {<o:p></o:p></span></pre>
<pre><o:p> </o:p></pre>
<pre><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">and<o:p></o:p></span></pre>
<pre><o:p> </o:p></pre>
<pre><span class="removed">-    if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB3(GetThreadState, jvmti,</span><o:p></o:p></pre>
<pre><span class="removed">-                    thread, &state) ) ) {</span><o:p></o:p></pre>
<pre><span class="new">+    if ( ! NSK_JVMTI_VERIFY (jvmti->GetThreadState(thread, &state) ) ) {</span><o:p></o:p></pre>
<pre>         NSK_COMPLAIN0("#error Agent :: while getting thread's state.\n");<o:p></o:p></pre>
<pre>         nsk_jvmti_agentFailed();<o:p></o:p></pre>
<pre>     } else {<o:p></o:p></pre>
<pre>         if ( state & JVMTI_THREAD_STATE_SUSPENDED) {<o:p></o:p></pre>
<pre><span class="removed">-            if ( ! NSK_JVMTI_VERIFY( NSK_CPP_STUB2(PopFrame, jvmti, thread) ) ){</span><o:p></o:p></pre>
<pre><span class="new">+            if ( ! NSK_JVMTI_VERIFY(jvmti->PopFrame(thread) ) ){</span><o:p></o:p></pre>
<pre><o:p> </o:p></pre>
<pre><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">=><o:p></o:p></span></pre>
<pre><o:p> </o:p></pre>
<pre><span class="new">+    if ( ! NSK_JVMTI_VERIFY ( jvmti->GetThreadState(thread, &state) ) ) {</span><o:p></o:p></pre>
<pre>…<o:p></o:p></pre>
<pre><span class="new">+            if ( ! NSK_JVMTI_VERIFY( jvmti->PopFrame(thread) ) ){</span><o:p></o:p></pre>
<pre><o:p> </o:p></pre>
<p class="MsoNormal">In hs204t001.cpp<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-        /* if( (myTestClass =NSK_CPP_STUB2(NewGlobalRef,jni_env, klass)) == NULL) {</span><o:p></o:p></pre>
<pre><span class="new">+        /* if( (myTestClass =jni_env->NewGlobalRef(klass)) == NULL) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+        /* if( (myTestClass = jni_env->NewGlobalRef(klass)) == NULL) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">and<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-    if (!NSK_JNI_VERIFY(env, (testClass =(jclass) NSK_CPP_STUB2(NewGlobalRef, env, klass)) != NULL))</span><o:p></o:p></pre>
<pre><span class="new">+    if (!NSK_JNI_VERIFY(env, (testClass =(jclass) env->NewGlobalRef(klass)) != NULL))</span><o:p></o:p></pre>
<pre>         nsk_jvmti_setFailStatus();<o:p></o:p></pre>
<pre><span class="removed">-    if (!NSK_JNI_VERIFY(env, (testedThread =NSK_CPP_STUB2(NewGlobalRef, env, thread)) != NULL))</span><o:p></o:p></pre>
<pre><span class="new">+    if (!NSK_JNI_VERIFY(env, (testedThread =env->NewGlobalRef(thread)) != NULL))</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+    if (!NSK_JNI_VERIFY(env, (testClass = (jclass) env->NewGlobalRef(klass)) != NULL))</span><o:p></o:p></pre>
<pre>         nsk_jvmti_setFailStatus();<o:p></o:p></pre>
<pre><span class="removed">…<o:p></o:p></span></pre>
<pre><span class="new">+    if (!NSK_JNI_VERIFY(env, (testedThread = env->NewGlobalRef(thread)) != NULL))</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">and<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-            if (!NSK_JVMTI_VERIFY(  NSK_CPP_STUB2(SuspendThread, jvmti, thread))) {</span><o:p></o:p></pre>
<pre><span class="new">+            if (!NSK_JVMTI_VERIFY(  jvmti->SuspendThread(thread))) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+            if (!NSK_JVMTI_VERIFY(jvmti->SuspendThread(thread))) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">In hs204t003.cpp<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-    if (! NSK_JVMTI_VERIFY( NSK_CPP_STUB3(GetThreadState, jvmti, thread, &state)) ){</span><o:p></o:p></pre>
<pre><span class="new">+    if (! NSK_JVMTI_VERIFY(jvmti->GetThreadState(thread, &state)) ){</span><o:p></o:p></pre>
<pre>         NSK_DISPLAY0(" Agent :: Error getting thread state.\n");<o:p></o:p></pre>
<pre>         nsk_jvmti_agentFailed();<o:p></o:p></pre>
<pre>     } else {<o:p></o:p></pre>
<pre>         if ( state & JVMTI_THREAD_STATE_SUSPENDED) {<o:p></o:p></pre>
<pre>             NSK_DISPLAY0(" Agent :: Thread state = JVMTI_THREAD_STATE_SUSPENDED.\n");<o:p></o:p></pre>
<pre><span class="removed">-            if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(PopFrame, jvmti, thread) ) ) {</span><o:p></o:p></pre>
<pre><span class="new">+            if ( ! NSK_JVMTI_VERIFY (jvmti->PopFrame(thread) ) ) {</span><o:p></o:p></pre>
<pre>                 NSK_DISPLAY0("#error Agent :: Jvmti failed to do popFrame.\n");<o:p></o:p></pre>
<pre>                 nsk_jvmti_agentFailed();<o:p></o:p></pre>
<pre>             } else {<o:p></o:p></pre>
<pre><span class="removed">-                if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(ResumeThread, jvmti, thread)) ) {</span><o:p></o:p></pre>
<pre><span class="new">+                if ( ! NSK_JVMTI_VERIFY (jvmti->ResumeThread(thread)) ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+    if (! NSK_JVMTI_VERIFY( jvmti->GetThreadState(thread, &state)) ){<o:p></o:p></span></pre>
<pre>…<o:p></o:p></pre>
<pre><span class="new">+            if ( ! NSK_JVMTI_VERIFY ( jvmti->PopFrame(thread) ) ) {<o:p></o:p></span></pre>
<pre>…<o:p></o:p></pre>
<pre><span class="new">+                if ( ! NSK_JVMTI_VERIFY ( jvmti->ResumeThread(thread)) ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">In hs301t001.cpp<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-        if (! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti, &caps) )) {</span><o:p></o:p></pre>
<pre><span class="new">+        if (! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) )) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+        if (! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) )) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">In hs301t002.cpp<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-        if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti, &caps) ) ) {</span><o:p></o:p></pre>
<pre><span class="new">+        if ( ! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) ) ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+        if ( ! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) ) ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">In hs301t003.cpp<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-  if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB4(GetClassSignature,</span><o:p></o:p></pre>
<pre><span class="removed">-                  jvmti_env, klass, &className, &generic) ) ) {</span><o:p></o:p></pre>
<pre><span class="new">+  if ( ! NSK_JVMTI_VERIFY (jvmti_env->GetClassSignature(klass, &className, &generic) ) ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+  if ( ! NSK_JVMTI_VERIFY ( jvmti_env->GetClassSignature(klass, &className, &generic) ) ) {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">and<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-        if (! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti, &caps) ))  {</span><o:p></o:p></pre>
<pre><span class="new">+        if (! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) ))  {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+        if (! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) ))  {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">and<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="removed">-        if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities,</span><o:p></o:p></pre>
<pre><span class="removed">-                        jvmti, &caps) ))  {</span><o:p></o:p></pre>
<pre><span class="new">+        if ( ! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) ))  {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">=><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<pre><span class="new">+        if ( ! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) ))  {</span><o:p></o:p></pre>
<p class="MsoNormal"><o:p> </o:p></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:12.0pt;color:black">From: </span></b><span style="font-size:12.0pt;color:black">serviceability-dev <serviceability-dev-bounces@openjdk.java.net> on behalf of JC Beyler <jcbeyler@google.com><br>
<b>Date: </b>Tuesday, October 16, 2018 at 4:24 PM<br>
<b>To: </b>"alexey.menkov@oracle.com" <alexey.menkov@oracle.com><br>
<b>Cc: </b>"serviceability-dev@openjdk.java.net" <serviceability-dev@openjdk.java.net><br>
<b>Subject: </b>Re: RFR (L) 8211899: Remove the NSK_CPP_STUB macros from vmTestbase for jvmti/scenarios/[E-M]<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Hi all, <o:p></o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">How about on a Tuesday? :)<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Sneak peak and motivational sentence: after this goes in, we have
<a href="http://cr.openjdk.java.net/~jcbeyler/8212148/webrev.00/">this one</a> which removes the NSK_CPP_STUBs from the tests entirely! :)<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Jc<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal">On Fri, Oct 12, 2018 at 5:37 PM JC Beyler <<a href="mailto:jcbeyler@google.com" target="_blank">jcbeyler@google.com</a>> wrote:<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<div>
<p class="MsoNormal">Hi all, <o:p></o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Any chance for a second reviewer on a Friday? :) <o:p></o:p></p>
<div>
<p class="MsoNormal">Jc<o:p></o:p></p>
</div>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal">On Thu, Oct 11, 2018 at 11:03 AM Alex Menkov <<a href="mailto:alexey.menkov@oracle.com" target="_blank">alexey.menkov@oracle.com</a>> wrote:<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal">got it.<br>
<br>
LGTM.<br>
<br>
--alex<br>
<br>
On 10/10/2018 19:03, JC Beyler wrote:<br>
> Hi Alex,<br>
> <br>
> Thanks for the review! Yes I had seen that too before sending this <br>
> review out and forked that fix into this:<br>
> <a href="https://bugs.openjdk.java.net/browse/JDK-8211905" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8211905</a><br>
> <br>
> Which now is merged and I've updated my webrev to reflect this of course.<br>
> <br>
> My rationale for not doing it here directly is always the same: keeping <br>
> the changes to the "spirit" of what the change is trying to do. Those <br>
> extra casts were a bit out-of-scope and so I just forked the fix in 8211905.<br>
> <br>
> Thanks!<br>
> Jc<br>
> <br>
> On Wed, Oct 10, 2018 at 5:40 PM Alex Menkov <<a href="mailto:alexey.menkov@oracle.com" target="_blank">alexey.menkov@oracle.com</a>
<br>
> <mailto:<a href="mailto:alexey.menkov@oracle.com" target="_blank">alexey.menkov@oracle.com</a>>> wrote:<br>
> <br>
>     Hi Jc,<br>
> <br>
>     Overall looks good.<br>
> <br>
>     one minor note:<br>
> <br>
>     test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/events/EM06/em06t001/em06t001.cpp:<br>
>     -    jclassName = (jstring) (jstring) (jstring) (jstring) (jstring)<br>
>     (jstring) (jstring) (jstring) (jstring) NSK_CPP_STUB3(CallObjectMethod,<br>
>     jni_env, klass,<br>
>     -                        methodID);<br>
>     +    jclassName = (jstring) (jstring) (jstring) (jstring) (jstring)<br>
>     (jstring) (jstring) (jstring) (jstring)<br>
>     jni_env->CallObjectMethod(klass,<br>
>     methodID);<br>
> <br>
>     Please drop multi "(jstring)"<br>
> <br>
>     --alex<br>
> <br>
>     On 10/08/2018 21:21, JC Beyler wrote:<br>
>      > Hi all,<br>
>      ><br>
>      > I am continuing the NSK_CPP_STUB removal with this next webrev.<br>
>      > Webrev: <a href="http://cr.openjdk.java.net/~jcbeyler/8211899/webrev.00/" target="_blank">
http://cr.openjdk.java.net/~jcbeyler/8211899/webrev.00/</a><br>
>     <<a href="http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/" target="_blank">http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/</a>><br>
>      > <<a href="http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/" target="_blank">http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/</a>><br>
>      > Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8211899" target="_blank">
https://bugs.openjdk.java.net/browse/JDK-8211899</a><br>
>      ><br>
>      > The change is still straight-forward though, since it is just<br>
>     doing the<br>
>      > same NSK_CPP_STUB removal. However when I was looking at the<br>
>     changes, a<br>
>      > lot of these changes are touching lines with spaces before/after<br>
>      > parenthesis. I've almost never touched the spaces except if I was<br>
>      > refactoring by hand the line at the same time. The rationale<br>
>     being that<br>
>      > the lines will get fixed a few more times and are, at worse,<br>
>     covered by<br>
>      > the bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8211335" target="_blank">
https://bugs.openjdk.java.net/browse/JDK-8211335</a>, which<br>
>     I've<br>
>      > commited to doing.<br>
>      ><br>
>      > Two exceptions are here where I pushed out the code into<br>
>     assignments due<br>
>      > to really long lines and complex if structures:<br>
>      > - jvmti/scenarios/hotswap/HS204/hs204t003/hs204t003.cpp<br>
>      ><br>
>     <<a href="http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/hotswap/HS204/hs204t003/hs204t003.cpp.udiff.html" target="_blank">http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/hotswap/HS204/hs204t003/hs204t003.cpp.udiff.html</a>><br>
>      > - jvmti/scenarios/jni_interception/JI01/ji01t001/ji01t001.cpp<br>
>      ><br>
>     <<a href="http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/jni_interception/JI01/ji01t001/ji01t001.cpp.udiff.html" target="_blank">http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/jni_interception/JI01/ji01t001/ji01t001.cpp.udiff.html</a>><br>
>      ><br>
>      > And one exception here where a commented line was doing the<br>
>     out-of-if<br>
>      > assignment so I just uncommented it and used the variable:<br>
>      > - jvmti/scenarios/hotswap/HS301/hs301t001/hs301t001.cpp<br>
>      ><br>
>     <<a href="http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/hotswap/HS301/hs301t001/hs301t001.cpp.udiff.html" target="_blank">http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/hotswap/HS301/hs301t001/hs301t001.cpp.udiff.html</a>><br>
>      ><br>
>      > I've tested the modified changes on my machine, all modified<br>
>     tests pass.<br>
>      ><br>
>      > Let me know what you think,<br>
>      > Jc<br>
>      ><br>
>      > Ps: 2 more of these and we can say good bye to NSK_CPP_STUB*<br>
>      ><br>
> <br>
> <br>
> <br>
> -- <br>
> <br>
> Thanks,<br>
> Jc<o:p></o:p></p>
</blockquote>
</div>
<p class="MsoNormal"><br clear="all">
<o:p></o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<p class="MsoNormal">-- <o:p></o:p></p>
<div>
<div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<p class="MsoNormal">Thanks, <o:p></o:p></p>
<div>
<p class="MsoNormal">Jc<o:p></o:p></p>
</div>
</div>
</div>
</blockquote>
</div>
<p class="MsoNormal"><br clear="all">
<o:p></o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<p class="MsoNormal">-- <o:p></o:p></p>
<div>
<div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<p class="MsoNormal">Thanks, <o:p></o:p></p>
<div>
<p class="MsoNormal">Jc<o:p></o:p></p>
</div>
</div>
</div>
</div>
</body>
</html>