On Mon, 27 Nov 2023 20:31:23 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> This is a docs only change to j.l.management.ThreadInfo::isInNative. >> >> The method currently specifies that it tests if the thread is "executing >> native code via the Java Native Interface (JNI)". It would be clearer to >> say that it tests if the thread is executing a native method, and expand it >> to include native code invoked using the new foreign linker APIs. For now, >> I've left out the detail of a downcall handle created with the "critical" >> linker option. >> >> The existing javadoc has wording to try to clarify that "native code" does >> not include runtime or compiled code. I went through a few iterations to >> work this in but all attempts just invite more questions and nit picking. To >> keep it simple, this sentence is dropped so that the method description is >> focused on the cases where the method returns true. > > Alan Bateman has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains five additional > commits since the last revision: > > - Change link to downcallHandle > - Merge > - Simplify wording > - Merge > - Initial commit Marked as reviewed by mchung (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/16791#pullrequestreview-1751563823