Hi Alex,

The fix looks good.
+1 for the Jc's suggestion to add spacing to the line:

 176         for (int i=0; i<50; i++) {

Thanks,
Serguei


On 10/18/18 15:42, Alex Menkov wrote:
Hi all,

Please review small test fix for
https://bugs.openjdk.java.net/browse/JDK-8212665
webrev:
http://cr.openjdk.java.net/~amenkov/deferredStep_endOfCycle/webrev/

The test verifies that after "next" command debugger always stops in the position within loops. Each thread has maximum 40 stops (10 iterations, 4 lines each), but with 50 "next" command it's possible that one of the thread exits the loop. The fix increments loops to make 15 iterations, so each thread has more than 50 stops.

--alex

Reply via email to