RFR: 8351419: java.net.http: Cleanup links in HttpResponse and module-info API doc comments

Michael McMahon michaelm at openjdk.org
Fri Mar 7 16:46:53 UTC 2025


On Fri, 7 Mar 2025 15:53:22 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

> This is a trivial doc cleanup. HttpResponse and module-info have a few links that should use {@linkplain } instead of {@link } or <href>. Also I noticed one link target mismatch, and some clarification missing in the send/receive buffer properties.

Looks fine. Minor formatting nit aside

src/java.net.http/share/classes/java/net/http/HttpResponse.java line 432:

> 430:          * BodySubscribers.fromLineSubscriber(subscriber, finisher, charset, lineSeparator)},
> 431:          * with the given {@code subscriber}, {@code finisher} function, and line separator.
> 432:          * The {@linkplain  Charset charset} used to decode the response body bytes is

Suggestion:

         * The {@linkplain Charset charset} used to decode the response body bytes is

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

Marked as reviewed by michaelm (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/23945#pullrequestreview-2667803078
PR Review Comment: https://git.openjdk.org/jdk/pull/23945#discussion_r1985378139


More information about the net-dev mailing list