RFR (M) 8212884: Remove the assignments in ifs for vmTestbase/[a-s]

JC Beyler jcbeyler at google.com
Wed Oct 24 20:00:24 UTC 2018


Hi Chris, thanks for the review!

I created https://bugs.openjdk.java.net/browse/JDK-8212939 for the
remaining spaces missing after an  if/while/for/switch and between ){ in
the case of an if:
jint  Agent_Initialize(JavaVM *vm, char *options, void *reserved){

Thanks,
Jc

On Wed, Oct 24, 2018 at 10:40 AM Chris Plummer <chris.plummer at oracle.com>
wrote:

> Hi JC,
>
> Overall looks good. Just a couple of minor things. I don't need to see
> another webrev.
>
> In Abort.cpp, Callbacks.cpp, ConcreteKlassFilter.cpp,
> HeapFilter.cpp.frames, NonConcreteKlassFilter.cpp, and
> threadend002.cpp.frames you need to add a space after the "if":
>
>  130   if(!NSK_VERIFY(jvmti != NULL)) {
>
> In em01t002.cpp, indent of 2nd line isn't correct:
>
>   74     methodID = jni_env->GetMethodID(
>   75                 klass, "loadClass",
> "(Ljava/lang/String;)Ljava/lang/Class;");
>
> thanks,
>
> Chris
>
> On 10/24/18 9:40 AM, JC Beyler wrote:
>
> Hi all,
>
> Here is the first webrev to extract assignments from if test clauses. It
> should be relatively straightforward to review.
>
> I tested all the tests changed and they pass on my dev machine.
>
> Most changes were done via the script I put in the bug, I had to do a few
> clean-ups and corner cases as well.
>
> Webrev: http://cr.openjdk.java.net/~jcbeyler/8212884/webrev.00/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8212884
>
> Thanks,
> Jc
>
>
>

-- 

Thanks,
Jc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20181024/d30dd5aa/attachment.html>


More information about the serviceability-dev mailing list