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

[Alex Menkov]

Hi Jc, Gary, Serguei,

Just for the record, updated webrev:
http://cr.openjdk.java.net/~amenkov/deferredStep_endOfCycle/webrev.01

(spaces are added, "steps" in the comment is replaced with "stops")

--alex

On 10/22/2018 13:10, serguei.spitsyn at oracle.com wrote:
> 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
>