RFR: XXS: JDK-8213910: Invalid HTML in java.net.http.HttpClient

Mandy Chung mandy.chung at oracle.com
Thu Nov 15 00:04:02 UTC 2018


Looks fine.

Mandy

On 11/14/18 4:01 PM, Jonathan Gibbons wrote:
> Please review the removal of two extraneous </a> in HttpClient.java.
>
> $ hg diff -R open
> diff -r 40098289d580 
> src/java.net.http/share/classes/java/net/http/HttpClient.java
> --- a/src/java.net.http/share/classes/java/net/http/HttpClient.java 
> Wed Nov 14 12:25:15 2018 -0800
> +++ b/src/java.net.http/share/classes/java/net/http/HttpClient.java 
> Wed Nov 14 15:56:04 2018 -0800
> @@ -104,7 +104,7 @@
>   *        .thenApply(HttpResponse::body)
>   *        .thenAccept(System.out::println); }</pre>
>   *
> - * <p> <a id="securitychecks"></a><b>Security checks</b></a>
> + * <p> <a id="securitychecks"><b>Security checks</b></a>
>   *
>   * <p> If a security manager is present then security checks are 
> performed by
>   * the HTTP Client's sending methods. An appropriate {@link 
> URLPermission} is
> @@ -327,7 +327,7 @@
>           * then newly built clients will use the {@linkplain
>           * ProxySelector#getDefault() default proxy selector}, which 
> is usually
>           * adequate for client applications. The default proxy 
> selector supports
> -         * a set of system properties</a> related to
> +         * a set of system properties related to
>           * <a 
> href="{@docRoot}/java.base/java/net/doc-files/net-properties.html#Proxies">
>           * proxy settings</a>. This default behavior can be disabled by
>           * supplying an explicit proxy selector, such as {@link 
> #NO_PROXY} or
>
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8213910
>
> -- Jon
>



More information about the core-libs-dev mailing list