RFR: 8284027: vmTestbase/nsk/jvmti/GetAllThreads/allthr001/ is failing [v2]

Alex Menkov amenkov at openjdk.java.net
Wed May 4 03:40:04 UTC 2022


On Wed, 4 May 2022 00:54:14 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Alex Menkov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   feedback
>
> test/hotspot/jtreg/vmTestbase/nsk/jvmti/GetAllThreads/allthr001/allthr001.cpp line 184:
> 
>> 182:     jthread *threads;
>> 183:     int i, j;
>> 184:     bool found;
> 
> Nit: this isn't old style C code, we don't have to declare all variables up front. New variables should be declared where needed.

ok :)
fixed.

-------------

PR: https://git.openjdk.java.net/jdk/pull/8512


More information about the serviceability-dev mailing list