On Thu, 30 Apr 2026 14:04:38 GMT, Jaikiran Pai <[email protected]> wrote:
>> Can I please get a review of this doc-only change which updates the >> specification of `Socket.isInputShutdown()` and `Socket.isOutputShutdown()` >> to clarify how it behaves when the `Socket` has been closed? >> >> This addresses https://bugs.openjdk.org/browse/JDK-8310138. The commit in >> this PR merely specifies the current implementation. I'll create a CSR >> shortly. >> >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Jaikiran Pai has updated the pull request incrementally with one additional > commit since the last revision: > > improve the text src/java.base/share/classes/java/net/Socket.java line 1756: > 1754: * independent of each other. Irrespective of whether or not the > socket > 1755: * is {@linkplain #close() closed}, this method will return {@code > true} only > 1756: * if a prior call to {@linkplain #shutdownInput() shutdownInput()} Since the link text is the method name (even including the `()`), would `@link` instead of `@linkplain` be more appropriate, to have it formatted as code? Suggestion: * if a prior call to {@link #shutdownInput()} Same applies to `isOutputShutdown`. (This is just a suggestion; feel free to ignore.) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31000#discussion_r3168648210
