On Thu, 19 Nov 2020 22:13:35 GMT, Leonid Mesnik <[email protected]> wrote:

>> - fixed java part of the test to handle spurious wakeups in the debuggee 
>> thread;
>> - fixed native part of the test to detect and report the case when debuggee 
>> thread not found.
>
> Looks good.

Why does this PR show 5 commits, including one that looks like it was for a 
previously fixed CR? It looks maybe you reused a branch. In the end it looks 
like skara did the right thing, and the diff looks right, but this seems to be 
something it is best to avoid with a PR. A rebase and a forced push might 
resolve this.

-------------

PR: https://git.openjdk.java.net/jdk/pull/1330

Reply via email to