RFR (L) 8214417: Add space after/before {} in vmTestbase/nsk/jvmti/[A-I] tests

[serguei.spitsyn at oracle.com]

On 11/28/18 15:43, Alex Menkov wrote:
> Hi Jc,
>
> In the JDK-8212771 review thread cleanup for "{}" was requested for 
> statements like
> test/hotspot/jtreg/vmTestbase/nsk/jvmti/unit/MethodBind/JvmtiTest/JvmtiTest.cpp, 
>
>
> +#define JVMTI_ERROR_CHECK(str,res) if (res != JVMTI_ERROR_NONE) { 
> printf(str); printf("%d\n",res); return res;}
>
> +#define JVMTI_ERROR_CHECK_EXPECTED_ERROR(str,res,err) if (res != err) 
> { printf(str); printf("unexpected error %d\n",res); return res;}
>
> I.e. something like ";}" -> "; }"
>
>
> I don't think changes like
>
> -static jvmtiEvent testEvents[] = {JVMTI_EVENT_THREAD_START, 
> JVMTI_EVENT_THREAD_END};
> +static jvmtiEvent testEvents[] = { JVMTI_EVENT_THREAD_START, 
> JVMTI_EVENT_THREAD_END };
>
> are required.

It is better to have it - rules are rules.

Thanks,
Serguei

>
> --alex
>
>
> On 11/28/2018 11:20, JC Beyler wrote:
>> Hi all,
>>
>> When working on a previous clean-up (JDK-8212771), I was asked to 
>> clean-up also spaces around {}.
>>
>> Here is the first batch out of 2 to fix these cases. Let me know what 
>> you think.
>>
>> Webrev: http://cr.openjdk.java.net/~jcbeyler/8214417/webrev.00/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8214417
>>
>> Thanks for your reviews :-),
>> Jc