Hi Chris,
Change looks good to me.
Thanks,
Vyom
On Tue, Mar 19, 2019 at 4:12 PM Chris Hegarty <chris.hega...@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
