On Tue, 9 Aug 2022 13:56:44 GMT, Alan Bateman <al...@openjdk.org> wrote:

>> Thread.State was added (in Java 5) for monitoring and management purposes, 
>> it's not an API that most libraries or programs would use. It could be 
>> introduced in the class description and referenced from other APIs that 
>> change/test the state but it would encourage use beyond the original intent.
>
> What you have is okay although I would have preferred if the sentence on the 
> join method was in the previous paragraph rather as it fits with termination.

Plus one on the join() bit. You could also argue for mentioning isAlive() here 
too.

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

PR: https://git.openjdk.org/jdk/pull/9437

Reply via email to