jmx-dev RFR: 8214545: sun/management/jmxremote/bootstrap tests hang in revokeall.exe on Windows

[Daniel Fuchs]

Hi,

On 20/05/2019 01:43, David Holmes wrote:
>> Webrev: http://cr.openjdk.java.net/~dtitov/8214545
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8214545
> The count-- is obvious as it is the loop counter, but it is far from 
> clear to me that i++ is correct. I don't fully understand the logic but 
> i is only incremented under very specific conditions. If you rewrote the 
> code to avoid the use of the continue then i would not be modified 
> except where it currently is.
> 

Looks like `i` tries to count the entries that were not
modified - so the fact that it was not incremented before
`continue` looks like an oversight. I'd say that Daniil is
right.
I believe Alan wrote that tool - he may be able to confirm ;-)

That said - if we could do the same thing in java as Alan suggests
and replace these shell scripts with java that might be a big
win!

best regards,

-- daniel