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 Marked as reviewed by alanb (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/18645#pullrequestreview-1984466385
