Re: КАК JDK-8212665: com/sun/jdi/DeferredStepTest.java: jj1 (line 57) - unexpected. lastLine=52, minLine=52, maxLine=55

[serguei.spitsyn at oracle.com]

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