On Fri, 5 Apr 2024 11:49:25 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> Can I please get a review of this doc-only change which proposes to clean up >> the documentation of `java.net.SocketOptions` interface? >> >> As noted in https://bugs.openjdk.org/browse/JDK-8329733, the existing >> documentation in this legacy interface isn't accurate. The commit in this >> PR, updates the documentation on several of the fields to point to the newer >> java.net.StandardSocketOptions class. While at it, I also did a general >> update of this documentation to use snippets and also reword some of it to >> be a bit more clear. >> >> I have run `make docs-image` on this change and the updated doc looks fine >> to me. > > Jaikiran Pai has updated the pull request incrementally with three additional > commits since the last revision: > > - "timeout value" instead of "timeout" > - missed SO_TIMEOUT review suggestion in previous commit > - Alan's review suggestions > So I think we need a follow-up to the change here to say that the value is > Boolean.FALSE to disable (or disabled), otherwise an Integer with the linger > timeout. I've opened https://bugs.openjdk.org/browse/JDK-8329825 to track this. ------------- PR Comment: https://git.openjdk.org/jdk/pull/18645#issuecomment-2041363132
