<div dir="ltr">Hi all,<div><br></div><div>@Serguei: no worries for the time taken, I am just two webrevs away from conquering the NSK_CPP_STUBs, I might have been a bit pushy for a review :-) I apologize!</div><div><br></div><div>@David: It's not hotspot style but a lot of those files had that style. My CL actually makes it non-conforming all around:<div><br></div><div>if ( before_we_had_this ) // space after ( + space before )</div><div><br></div><div>if (now_we_have_this )  // only space before )</div><div><br></div><div>which is kind of weird and Paul's comment was to at least try to not make it inconsistent to that point.</div><div><br></div><div>I have one webrev to go that will remove NSK_CPP_STUBs and then I'm going to fix all the spaces (via <a href="https://bugs.openjdk.java.net/browse/JDK-8211335" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8211335</a>) for these .cpp files so we won't have the issue anymore.</div><div><br></div><div>Does that make sense?</div><div><br></div><div>Thanks!</div><div>Jc</div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Oct 17, 2018 at 6:34 PM David Holmes <<a href="mailto:david.holmes@oracle.com">david.holmes@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 18/10/2018 6:01 AM, Hohensee, Paul wrote:<br>
> Re spaces :).  There’s a bunch of places where “(jvmti_env”  should be <br>
> “( jvmti_env”, but of course not all of them. I found a bunch. <br>
<br>
Why should there be a space after the '(' ? That's not hotspot-style. <br>
Neither is a space before ')'.<br>
<br>
David<br>
<br>
> Otherwise, lgtm., no need for another webrev.<br>
> <br>
> Paul<br>
> <br>
> In hs104t002.cpp<br>
> <br>
> -        if (! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti,<br>
> <br>
> -                &caps) )) {<br>
> <br>
> +        if (! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) )) {<br>
> <br>
> =><br>
> <br>
> +        if (! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) )) {<br>
> <br>
> In hs203t003.cpp<br>
> <br>
> -    if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB4(GetClassSignature,<br>
> <br>
> -                    jvmti_env, klass, &className, &generic) ) ) {<br>
> <br>
> +    if ( ! NSK_JVMTI_VERIFY (jvmti_env->GetClassSignature(klass, <br>
> &className, &generic) ) ) {<br>
> <br>
> =><br>
> <br>
> +    if ( ! NSK_JVMTI_VERIFY ( jvmti_env->GetClassSignature(klass, <br>
> &className, &generic) ) ) {<br>
> <br>
> and<br>
> <br>
> -        if (! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti, <br>
> &caps) )) {<br>
> <br>
> +        if (! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) )) {<br>
> <br>
> =><br>
> <br>
> +        if (! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) )) {<br>
> <br>
> and<br>
> <br>
> -    if ( ! NSK_JVMTI_VERIFY( NSK_CPP_STUB3(GetThreadState, jvmti, <br>
> thread, &state) )  ) {<br>
> <br>
> +    if ( ! NSK_JVMTI_VERIFY(jvmti->GetThreadState(thread, &state) )  ) {<br>
> <br>
> =><br>
> <br>
> +    if ( ! NSK_JVMTI_VERIFY( jvmti->GetThreadState(thread, &state) )  ) {<br>
> <br>
> and<br>
> <br>
> -    if ( ! NSK_JVMTI_VERIFY( NSK_CPP_STUB3(GetThreadState, jvmti, <br>
> thread, &state) )  ) {<br>
> <br>
> +    if ( ! NSK_JVMTI_VERIFY(jvmti->GetThreadState(thread, &state) )  ) {<br>
> <br>
>           nsk_printf(" Agent :: Error while getting thread state.\n");<br>
> <br>
>           nsk_jvmti_agentFailed();<br>
> <br>
>       } else {<br>
> <br>
>           if ( state & JVMTI_THREAD_STATE_SUSPENDED) {<br>
> <br>
> -            if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2( PopFrame, jvmti, <br>
> thread) ) ) {<br>
> <br>
> +            if ( ! NSK_JVMTI_VERIFY (jvmti->PopFrame(thread) ) ) {<br>
> <br>
> =><br>
> <br>
> +    if ( ! NSK_JVMTI_VERIFY( jvmti->GetThreadState(thread, &state) )  ) {<br>
> <br>
> …<br>
> <br>
> +            if ( ! NSK_JVMTI_VERIFY ( jvmti->PopFrame(thread) ) ) {<br>
> <br>
> and<br>
> <br>
> -    if ( !NSK_JVMTI_VERIFY( NSK_CPP_STUB2 ( ResumeThread, jvmti, <br>
> thread)) ) {<br>
> <br>
> +    if ( !NSK_JVMTI_VERIFY(jvmti->ResumeThread(thread)) ) {<br>
> <br>
> =><br>
> <br>
> +    if ( !NSK_JVMTI_VERIFY( jvmti->ResumeThread(thread)) ) {<br>
> <br>
> In hs203t004.cpp<br>
> <br>
> -                if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(GenerateEvents, <br>
> jvmti_env,<br>
> <br>
> -                                JVMTI_EVENT_COMPILED_METHOD_LOAD ) )) {<br>
> <br>
> +                if ( ! NSK_JVMTI_VERIFY <br>
> (jvmti_env->GenerateEvents(JVMTI_EVENT_COMPILED_METHOD_LOAD) )) {<br>
> <br>
> =><br>
> <br>
> +                if ( ! NSK_JVMTI_VERIFY ( <br>
> jvmti_env->GenerateEvents(JVMTI_EVENT_COMPILED_METHOD_LOAD) )) {<br>
> <br>
> and<br>
> <br>
> -        if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB3(GetMethodDeclaringClass,<br>
> <br>
> -                        jvmti_env, method, &threadClass) ) ) {<br>
> <br>
> +        if ( ! NSK_JVMTI_VERIFY <br>
> (jvmti_env->GetMethodDeclaringClass(method, &threadClass) ) ) {<br>
> <br>
> =><br>
> <br>
> +        if ( ! NSK_JVMTI_VERIFY ( <br>
> jvmti_env->GetMethodDeclaringClass(method, &threadClass) ) ) {<br>
> <br>
> and<br>
> <br>
> -        if (! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti, <br>
> &caps) )) {<br>
> <br>
> +        if (! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) )) {<br>
> <br>
> =><br>
> <br>
> +        if (! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) )) {<br>
> <br>
> and<br>
> <br>
> -    if (  NSK_JVMTI_VERIFY( NSK_CPP_STUB2(SuspendThread, jvmti, thread) <br>
> ) ) {<br>
> <br>
> +    if (  NSK_JVMTI_VERIFY(jvmti->SuspendThread(thread) ) ) {<br>
> <br>
> =><br>
> <br>
> +    if (  NSK_JVMTI_VERIFY( jvmti->SuspendThread(thread) ) ) {<br>
> <br>
> and<br>
> <br>
> -    if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB3(GetThreadState, jvmti,<br>
> <br>
> -                    thread, &state) ) ) {<br>
> <br>
> +    if ( ! NSK_JVMTI_VERIFY (jvmti->GetThreadState(thread, &state) ) ) {<br>
> <br>
>           NSK_COMPLAIN0("#error Agent :: while getting thread's state.\n");<br>
> <br>
>           nsk_jvmti_agentFailed();<br>
> <br>
>       } else {<br>
> <br>
>           if ( state & JVMTI_THREAD_STATE_SUSPENDED) {<br>
> <br>
> -            if ( ! NSK_JVMTI_VERIFY( NSK_CPP_STUB2(PopFrame, jvmti, <br>
> thread) ) ){<br>
> <br>
> +            if ( ! NSK_JVMTI_VERIFY(jvmti->PopFrame(thread) ) ){<br>
> <br>
> =><br>
> <br>
> +    if ( ! NSK_JVMTI_VERIFY ( jvmti->GetThreadState(thread, &state) ) ) {<br>
> <br>
> …<br>
> <br>
> +            if ( ! NSK_JVMTI_VERIFY( jvmti->PopFrame(thread) ) ){<br>
> <br>
> In hs204t001.cpp<br>
> <br>
> -        /* if( (myTestClass =NSK_CPP_STUB2(NewGlobalRef,jni_env, <br>
> klass)) == NULL) {<br>
> <br>
> +        /* if( (myTestClass =jni_env->NewGlobalRef(klass)) == NULL) {<br>
> <br>
> =><br>
> <br>
> +        /* if( (myTestClass = jni_env->NewGlobalRef(klass)) == NULL) {<br>
> <br>
> and<br>
> <br>
> -    if (!NSK_JNI_VERIFY(env, (testClass =(jclass) <br>
> NSK_CPP_STUB2(NewGlobalRef, env, klass)) != NULL))<br>
> <br>
> +    if (!NSK_JNI_VERIFY(env, (testClass =(jclass) <br>
> env->NewGlobalRef(klass)) != NULL))<br>
> <br>
>           nsk_jvmti_setFailStatus();<br>
> <br>
> -    if (!NSK_JNI_VERIFY(env, (testedThread =NSK_CPP_STUB2(NewGlobalRef, <br>
> env, thread)) != NULL))<br>
> <br>
> +    if (!NSK_JNI_VERIFY(env, (testedThread =env->NewGlobalRef(thread)) <br>
> != NULL))<br>
> <br>
> =><br>
> <br>
> +    if (!NSK_JNI_VERIFY(env, (testClass = (jclass) <br>
> env->NewGlobalRef(klass)) != NULL))<br>
> <br>
>           nsk_jvmti_setFailStatus();<br>
> <br>
> …<br>
> <br>
> +    if (!NSK_JNI_VERIFY(env, (testedThread = env->NewGlobalRef(thread)) <br>
> != NULL))<br>
> <br>
> and<br>
> <br>
> -            if (!NSK_JVMTI_VERIFY(  NSK_CPP_STUB2(SuspendThread, jvmti, <br>
> thread))) {<br>
> <br>
> +            if (!NSK_JVMTI_VERIFY(  jvmti->SuspendThread(thread))) {<br>
> <br>
> =><br>
> <br>
> +            if (!NSK_JVMTI_VERIFY(jvmti->SuspendThread(thread))) {<br>
> <br>
> In hs204t003.cpp<br>
> <br>
> -    if (! NSK_JVMTI_VERIFY( NSK_CPP_STUB3(GetThreadState, jvmti, <br>
> thread, &state)) ){<br>
> <br>
> +    if (! NSK_JVMTI_VERIFY(jvmti->GetThreadState(thread, &state)) ){<br>
> <br>
>           NSK_DISPLAY0(" Agent :: Error getting thread state.\n");<br>
> <br>
>           nsk_jvmti_agentFailed();<br>
> <br>
>       } else {<br>
> <br>
>           if ( state & JVMTI_THREAD_STATE_SUSPENDED) {<br>
> <br>
>               NSK_DISPLAY0(" Agent :: Thread state = JVMTI_THREAD_STATE_SUSPENDED.\n");<br>
> <br>
> -            if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(PopFrame, jvmti, <br>
> thread) ) ) {<br>
> <br>
> +            if ( ! NSK_JVMTI_VERIFY (jvmti->PopFrame(thread) ) ) {<br>
> <br>
>                   NSK_DISPLAY0("#error Agent :: Jvmti failed to do popFrame.\n");<br>
> <br>
>                   nsk_jvmti_agentFailed();<br>
> <br>
>               } else {<br>
> <br>
> -                if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(ResumeThread, <br>
> jvmti, thread)) ) {<br>
> <br>
> +                if ( ! NSK_JVMTI_VERIFY (jvmti->ResumeThread(thread)) ) {<br>
> <br>
> =><br>
> <br>
> +    if (! NSK_JVMTI_VERIFY( jvmti->GetThreadState(thread, &state)) ){<br>
> <br>
> …<br>
> <br>
> +            if ( ! NSK_JVMTI_VERIFY ( jvmti->PopFrame(thread) ) ) {<br>
> <br>
> …<br>
> <br>
> +                if ( ! NSK_JVMTI_VERIFY ( jvmti->ResumeThread(thread)) ) {<br>
> <br>
> In hs301t001.cpp<br>
> <br>
> -        if (! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti, <br>
> &caps) )) {<br>
> <br>
> +        if (! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) )) {<br>
> <br>
> =><br>
> <br>
> +        if (! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) )) {<br>
> <br>
> In hs301t002.cpp<br>
> <br>
> -        if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti, <br>
> &caps) ) ) {<br>
> <br>
> +        if ( ! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) ) ) {<br>
> <br>
> =><br>
> <br>
> +        if ( ! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) ) ) {<br>
> <br>
> In hs301t003.cpp<br>
> <br>
> -  if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB4(GetClassSignature,<br>
> <br>
> -                  jvmti_env, klass, &className, &generic) ) ) {<br>
> <br>
> +  if ( ! NSK_JVMTI_VERIFY (jvmti_env->GetClassSignature(klass, <br>
> &className, &generic) ) ) {<br>
> <br>
> =><br>
> <br>
> +  if ( ! NSK_JVMTI_VERIFY ( jvmti_env->GetClassSignature(klass, <br>
> &className, &generic) ) ) {<br>
> <br>
> and<br>
> <br>
> -        if (! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities, jvmti, <br>
> &caps) ))  {<br>
> <br>
> +        if (! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) ))  {<br>
> <br>
> =><br>
> <br>
> +        if (! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) ))  {<br>
> <br>
> and<br>
> <br>
> -        if ( ! NSK_JVMTI_VERIFY ( NSK_CPP_STUB2(AddCapabilities,<br>
> <br>
> -                        jvmti, &caps) ))  {<br>
> <br>
> +        if ( ! NSK_JVMTI_VERIFY (jvmti->AddCapabilities(&caps) ))  {<br>
> <br>
> =><br>
> <br>
> +        if ( ! NSK_JVMTI_VERIFY ( jvmti->AddCapabilities(&caps) ))  {<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: *Tuesday, October 16, 2018 at 4:24 PM<br>
> *To: *"<a href="mailto:alexey.menkov@oracle.com" target="_blank">alexey.menkov@oracle.com</a>" <<a href="mailto:alexey.menkov@oracle.com" target="_blank">alexey.menkov@oracle.com</a>><br>
> *Cc: *"<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: *Re: RFR (L) 8211899: Remove the NSK_CPP_STUB macros from <br>
> vmTestbase for jvmti/scenarios/[E-M]<br>
> <br>
> Hi all,<br>
> <br>
> How about on a Tuesday? :)<br>
> <br>
> Sneak peak and motivational sentence: after this goes in, we have this <br>
> one <<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>> which <br>
> removes the NSK_CPP_STUBs from the tests entirely! :)<br>
> <br>
> Jc<br>
> <br>
> On Fri, Oct 12, 2018 at 5:37 PM JC Beyler <<a href="mailto:jcbeyler@google.com" target="_blank">jcbeyler@google.com</a> <br>
> <mailto:<a href="mailto:jcbeyler@google.com" target="_blank">jcbeyler@google.com</a>>> wrote:<br>
> <br>
>     Hi all,<br>
> <br>
>     Any chance for a second reviewer on a Friday? :)<br>
> <br>
>     Jc<br>
> <br>
>     On Thu, Oct 11, 2018 at 11:03 AM Alex Menkov<br>
>     <<a href="mailto:alexey.menkov@oracle.com" target="_blank">alexey.menkov@oracle.com</a> <mailto:<a href="mailto:alexey.menkov@oracle.com" target="_blank">alexey.menkov@oracle.com</a>>> wrote:<br>
> <br>
>         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<br>
>         this<br>
>          > review out and forked that fix into this:<br>
>          > <a href="https://bugs.openjdk.java.net/browse/JDK-8211905" rel="noreferrer" 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<br>
>         this of course.<br>
>          ><br>
>          > My rationale for not doing it here directly is always the<br>
>         same: keeping<br>
>          > the changes to the "spirit" of what the change is trying to<br>
>         do. Those<br>
>          > extra casts were a bit out-of-scope and so I just forked the<br>
>         fix in 8211905.<br>
>          ><br>
>          > Thanks!<br>
>          > Jc<br>
>          ><br>
>          > On Wed, Oct 10, 2018 at 5:40 PM Alex Menkov<br>
>         <<a href="mailto:alexey.menkov@oracle.com" target="_blank">alexey.menkov@oracle.com</a> <mailto:<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><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>
>          >   <br>
>           test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/events/EM06/em06t001/em06t001.cpp:<br>
>          >     -    jclassName = (jstring) (jstring) (jstring) (jstring)<br>
>         (jstring)<br>
>          >     (jstring) (jstring) (jstring) (jstring)<br>
>         NSK_CPP_STUB3(CallObjectMethod,<br>
>          >     jni_env, klass,<br>
>          >     -                        methodID);<br>
>          >     +    jclassName = (jstring) (jstring) (jstring) (jstring)<br>
>         (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<br>
>         next webrev.<br>
>          >      > Webrev:<br>
>         <a href="http://cr.openjdk.java.net/~jcbeyler/8211899/webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~jcbeyler/8211899/webrev.00/</a><br>
>         <<a href="http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/</a>><br>
>          >     <<a href="http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/</a>><br>
>          >      ><br>
>         <<a href="http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/%7Ejcbeyler/8211899/webrev.00/</a>><br>
>          >      > Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8211899" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8211899</a><br>
>          >      ><br>
>          >      > The change is still straight-forward though, since it<br>
>         is just<br>
>          >     doing the<br>
>          >      > same NSK_CPP_STUB removal. However when I was looking<br>
>         at the<br>
>          >     changes, a<br>
>          >      > lot of these changes are touching lines with spaces<br>
>         before/after<br>
>          >      > parenthesis. I've almost never touched the spaces<br>
>         except if I was<br>
>          >      > refactoring by hand the line at the same time. The<br>
>         rationale<br>
>          >     being that<br>
>          >      > the lines will get fixed a few more times and are, at<br>
>         worse,<br>
>          >     covered by<br>
>          >      > the bug:<br>
>         <a href="https://bugs.openjdk.java.net/browse/JDK-8211335" rel="noreferrer" 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>
>          >   <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" rel="noreferrer" 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>
>          >      > -<br>
>         jvmti/scenarios/jni_interception/JI01/ji01t001/ji01t001.cpp<br>
>          >      ><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" rel="noreferrer" 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<br>
>         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>
>          >   <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" rel="noreferrer" 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<br>
>         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<br>
>         NSK_CPP_STUB*<br>
>          >      ><br>
>          ><br>
>          ><br>
>          ><br>
>          > --<br>
>          ><br>
>          > Thanks,<br>
>          > Jc<br>
> <br>
> <br>
>     -- <br>
> <br>
>     Thanks,<br>
> <br>
>     Jc<br>
> <br>
> <br>
> -- <br>
> <br>
> Thanks,<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>