RFR [13] 8153508: ContentHandler API contains link to private contentPathProp

[Vyom Tiwari]

Hi Chris,

Change looks good to me.
Thanks,
Vyom

On Tue, Mar 19, 2019 at 4:12 PM Chris Hegarty <chris.hegarty at oracle.com>
wrote:

> This review request is to resolve a documentation regression in Java 9,
> resulting from part of the changes for 8132478, where {@value
> java.net.URLConnection#contentPathProp} was inadvertently replaced with
> {@link java.net.URLConnection#contentPathProp}. The former will inline
> with the constant field value, where the latter will link to the
> ( private ) field.
>
> The fix is to just revert the incorrect change.
>
> --- a/src/java.base/share/classes/java/net/ContentHandler.java
> +++ b/src/java.base/share/classes/java/net/ContentHandler.java
> @@ -47,7 +47,7 @@
>   * If no content handler could be {@linkplain URLConnection#getContent()
> found},
>   * URLConnection will look for a content handler in a user-definable set
> of places.
>   * Users can define a vertical-bar delimited set of class prefixes
> - * to search through by defining the <i>{@link
> java.net.URLConnection#contentPathProp}</i>
> + * to search through by defining the <i>{@value
> java.net.URLConnection#contentPathProp}</i>
>   * property. The class name must be of the form:
>   * <blockquote>
>   *     <i>{package-prefix}.{major}.{minor}</i>
>
> -Chris.



-- 
Thanks,
Vyom
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/net-dev/attachments/20190319/f6eda350/attachment.html>