LGTM
Thanks,
Serguei
On 10/22/18 14:38, Alex Menkov wrote:
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, [email protected] 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
