+1
But could you please remove old "pipe.println("docontinue")" statement
instead of commenting it out (no new webrev is required)
--alex
On 02/20/2019 09:58, Chris Plummer wrote:
Looks good.
Chris
On 2/20/19 7:57 AM, Gary Adams wrote:
The issuspended002 has been on the ProblemList since the tests were
moved to
the open repos. The proposed changeset applies the same fix that was
used in issuspended001.
The current test fails when the debuggee replies with the "docontinue"
response,
while it is still holding a lock in a syncrhonized block. When the
debugger then
suspends the vm the debuggee test thread fails to proceed to the
expected breakpoint.
Testing in progress.
Issue: https://bugs.openjdk.java.net/browse/JDK-4903717
Webrev: http://cr.openjdk.java.net/~gadams/4903717/webrev.00/
