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