On Wed, 1 Jun 2022 06:26:23 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> Alan Bateman has updated the pull request with a new target base due to a 
>> merge or a rebase. The pull request now contains 11 commits:
>> 
>>  - Fixed another typo in comment
>>  - Merge
>>  - Fix typos in comments
>>  - Allowing linking without intrinsic stub, contributed-by: rehn
>>  - Continuation clinit should fail if no continuations support
>>  - Fix merge issue with test
>>  - Change VMContinuations to be experimental option, ensure JNI GetEnv 
>> returns EVERSION
>>  - Update
>>  - Expend test coverage
>>  - Merge
>>  - ... and 1 more: 
>> https://git.openjdk.java.net/jdk/compare/3deb58a8...a5c06cb6
>
> src/java.base/share/classes/java/lang/Thread.java line 742:
> 
>> 740:      * @param name thread name, can be null
>> 741:      * @param characteristics thread characteristics
>> 742:      * @param bound true when bound to an OS thread
> 
> Nit: s/OS/VM/ ?

I think it would be confusing to use "VM" here. The intention is that "true" 
means the virtual thread will be bound to an OS thread once it has started. 
Yes, technically it's whatever the VM implements but I think that is too much 
to try to explain in the parameter description.

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

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

Reply via email to