RFR: 8370568: Refer to Thread.interrupted as "interrupted status" consistently [v2]

Pavel Rappo prappo at openjdk.org
Tue Oct 28 10:06:36 UTC 2025


> Throughout documentation and source code, the `Thread.interrupted` flag is referred to as either "interrupt**ed** status" or "interrupt status". It might be good to be consistent.
> 
> Historically, it seems to have initially been "interrupted status". This is how the flag is called in `java.lang.Thread` and the "Java Concurrency in Practice" book. ("The Java Programming Language" calls it "interrupted **state**".) However, over the years "interrupt status" appeared in documentation and source code through networking and NIO classes.

Pavel Rappo has updated the pull request incrementally with three additional commits since the last revision:

 - Update copyright years
   
   Note: any commit hashes below might be outdated due to subsequent
   history rewriting (e.g. git rebase).
   
    + update make/langtools/tools/javacserver/server/CompilerThreadPool.java due to a10f8b4304d
    + update src/java.base/share/classes/java/lang/Object.java due to a10f8b4304d
    + update src/java.base/share/classes/java/net/DatagramSocket.java due to a6a23d6fdaf
    + update src/java.base/share/classes/java/net/ServerSocket.java due to a6a23d6fdaf
    + update src/java.base/share/classes/java/nio/channels/DatagramChannel.java due to a10f8b4304d
    + update src/java.base/share/classes/java/nio/channels/FileChannel.java due to a10f8b4304d
    + update src/java.base/share/classes/java/nio/channels/InterruptibleChannel.java due to a10f8b4304d
    + update src/java.base/share/classes/java/nio/channels/ReadableByteChannel.java due to a10f8b4304d
    + update src/java.base/share/classes/java/nio/channels/ScatteringByteChannel.java due to a10f8b4304d
    + update src/java.base/share/classes/java/nio/channels/Selector.java due to a10f8b4304d
    + update src/java.base/share/classes/java/nio/channels/ServerSocketChannel.java due to a10f8b4304d
    + update src/java.base/share/classes/java/nio/channels/SocketChannel.java due to a10f8b4304d
    + update src/java.base/share/classes/sun/nio/ch/Interruptible.java due to a10f8b4304d
    + update src/java.base/share/classes/sun/security/ssl/StatusResponseManager.java due to a10f8b4304d
    + update src/java.desktop/share/classes/java/awt/Robot.java due to a10f8b4304d
    + update src/java.xml/share/classes/com/sun/org/apache/xerces/internal/parsers/DOMParserImpl.java due to a10f8b4304d
    + update src/jdk.sctp/share/classes/com/sun/nio/sctp/SctpChannel.java due to a10f8b4304d
    + update src/jdk.sctp/share/classes/com/sun/nio/sctp/SctpMultiChannel.java due to a10f8b4304d
    + update src/jdk.sctp/share/classes/com/sun/nio/sctp/SctpServerChannel.java due to a10f8b4304d
    + update test/hotspot/jtreg/serviceability/jvmti/vthread/GetThreadState/GetThreadStateTest.java due to a10f8b4304d
    + update test/hotspot/jtreg/vmTestbase/nsk/jvmti/InterruptThread/intrpthrd001/TestDescription.java due to a10f8b4304d
    + update test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/bcinstr/BI04/bi04t002/newclass02/java.base/java/lang/Object.java due to a10f8b4304d
    + update test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/sampling/SP01/sp01t002/TestDescription.java due to a10f8b4304d
    + update test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/sampling/SP01/sp01t003/TestDescription.java due to a10f8b4304d
    + update test/hotspot/jtreg/vmTestbase/nsk/share/gc/AllDiag.java due to a10f8b4304d
    + update test/hotspot/jtreg/vmTestbase/nsk/share/gc/FinDiag.java due to a10f8b4304d
    + update test/hotspot/jtreg/vmTestbase/nsk/share/runner/MemDiag.java due to a10f8b4304d
    + update test/jdk/java/lang/Thread/JoinWithDuration.java due to a10f8b4304d
    + update test/jdk/java/lang/Thread/SleepWithDuration.java due to a10f8b4304d
    + update test/jdk/java/lang/Thread/virtual/CustomScheduler.java due to a6a23d6fdaf
    + update test/jdk/java/nio/channels/Channels/SocketChannelStreams.java due to a10f8b4304d
    + update test/jdk/java/nio/channels/FileChannel/CloseDuringTransfer.java due to a10f8b4304d
    + update test/jdk/java/nio/channels/FileChannel/ClosedByInterrupt.java due to a10f8b4304d
    + update test/jdk/java/nio/channels/Pipe/PipeInterrupt.java due to a10f8b4304d
    + update test/jdk/java/nio/channels/Selector/LotsOfInterrupts.java due to a10f8b4304d
    + update test/jdk/java/nio/channels/Selector/SelectWithConsumer.java due to a10f8b4304d
    + update test/jdk/java/nio/channels/Selector/WakeupAfterClose.java due to a10f8b4304d
    + update test/jdk/java/nio/channels/SocketChannel/AdaptorStreams.java due to a10f8b4304d
    + update test/jdk/java/nio/file/Files/CallWithInterruptSet.java due to a10f8b4304d
    + update test/jdk/java/nio/file/Files/InterruptCopy.java due to a10f8b4304d
    + update test/jdk/java/util/concurrent/CompletableFuture/LostInterrupt.java due to a10f8b4304d
    + update test/jdk/java/util/concurrent/CompletableFuture/SwallowedInterruptedException.java due to a10f8b4304d
    + update test/jdk/java/util/concurrent/ExecutorService/CloseTest.java due to a10f8b4304d
    + update test/jdk/java/util/concurrent/ExecutorService/InvokeTest.java due to a10f8b4304d
    + update test/jdk/java/util/zip/InterruptibleZip.java due to a10f8b4304d
 - Reword for clarity as suggested
 - Drop to ease upcoming merge from loom repo

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/27972/files
  - new: https://git.openjdk.org/jdk/pull/27972/files/a10f8b43..89dbafe0

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=27972&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=27972&range=00-01

  Stats: 67 lines in 47 files changed: 4 ins; 0 del; 63 mod
  Patch: https://git.openjdk.org/jdk/pull/27972.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/27972/head:pull/27972

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


More information about the hotspot-gc-dev mailing list