Discussion:
RFR JDK-8212629: [TEST] wrong breakpoint in test/jdk/com/sun/jdi/DeferredStepTest
(too old to reply)
Alex Menkov
2018-10-18 22:26:06 UTC
Permalink
Hi all,

please review trivial fix for
https://bugs.openjdk.java.net/browse/JDK-8212629
webrev:
http://cr.openjdk.java.net/~amenkov/deferredStep_extraBP/webrev/

The fix adds space inside comment to avoid wrong breakpoint parsing
(breakpoint lines are marked as "@{id} breakpoint", where {id} is an
integer)

--alex
Chris Plummer
2018-10-18 22:32:12 UTC
Permalink
Looks good.

Chris
Post by Alex Menkov
Hi all,
please review trivial fix for
https://bugs.openjdk.java.net/browse/JDK-8212629
http://cr.openjdk.java.net/~amenkov/deferredStep_extraBP/webrev/
The fix adds space inside comment to avoid wrong breakpoint parsing
integer)
--alex
JC Beyler
2018-10-19 02:23:16 UTC
Permalink
Looks good to me too!
Jc
Post by Chris Plummer
Looks good.
Chris
Post by Alex Menkov
Hi all,
please review trivial fix for
https://bugs.openjdk.java.net/browse/JDK-8212629
http://cr.openjdk.java.net/~amenkov/deferredStep_extraBP/webrev/
The fix adds space inside comment to avoid wrong breakpoint parsing
integer)
--alex
--
Thanks,
Jc
Loading...