On Tue, 11 Nov 2025 16:03:11 GMT, Pavel Rappo <[email protected]> wrote:
>> This is docs only change to a add a new section to the Thread class
>> description on Thread Interruption. The new section provides guidance on
>> handling InterruptedException. A subset of that guidance is also proposed
>> for the InterruptedException class description.
>
> src/java.base/share/classes/java/lang/Thread.java line 197:
>
>> 195: * java.nio.channels.InterruptibleChannel} is interrupted then it
>> causes the channel to be
>> 196: * closed, and the blocking I/O operation to throw {@link
>> java.nio.channels.ClosedByInterruptException}
>> 197: * with the thread's interrupted status set. If a thread is blocked in
>> a {@linkplain
>
> Suggestion:
>
> * with the thread's interrupted status set. If a thread that is blocked in a
> {@linkplain
>
> Otherwise that sentence reads really weird.
You are right that this could be improved but rather than inserting "that", it
can be shortened to "If a thread blocked in a selection operation is
interrupted".
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28216#discussion_r2515083266
