RFR: 8256364: vmTestbase/nsk/jvmti/scenarios/capability/CM01/cm01t002 failed with "assert(handle != __null) failed: JNI handle should not be null"

Chris Plummer cjplummer at openjdk.java.net
Thu Nov 19 22:29:00 UTC 2020


On Thu, 19 Nov 2020 22:13:35 GMT, Leonid Mesnik <lmesnik at openjdk.org> 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


More information about the serviceability-dev mailing list