RFR [9] 8139179: URLStreamHandler* should link to URL ctor that specifies how factories/providers are located

Alan Bateman Alan.Bateman at oracle.com
Fri Oct 9 14:29:29 UTC 2015


This looks okay to me.

On 09/10/2015 14:52, Chris Hegarty wrote:
> It was pointed out that the updated URL spec that describes how
> URL protocol handlers are located isn't prominent in the
> avadoc. In particular it was noted that it's not linked from 
> URLStreamHandlerFactory or URLStreamHandlerProvider.
>
> Adding such links will make it clear how these classes tie
> together.
>
> diff --git 
> a/src/java.base/share/classes/java/net/URLStreamHandlerFactory.java 
> b/src/java.base/share/classes/java/net/URLStreamHandlerFactory.java
> --- a/src/java.base/share/classes/java/net/URLStreamHandlerFactory.java
> +++ b/src/java.base/share/classes/java/net/URLStreamHandlerFactory.java
> @@ -28,9 +28,9 @@
>  /**
>   * This interface defines a factory for {@code URL} stream
>   * protocol handlers.
> - * <p>
> - * It is used by the {@code URL} class to create a
> - * {@code URLStreamHandler} for a specific protocol.
> + *
> + * <p> A URL stream handler factory is used as specified in the
> + * {@linkplain java.net.URL#URL(String,String,int,String) URL 
> constructor}.
>   *
>   * @author  Arthur van Hoff
>   * @see     java.net.URL
> diff --git 
> a/src/java.base/share/classes/java/net/spi/URLStreamHandlerProvider.java 
> b/src/java.base/share/classes/java/net/spi/URLStreamHandlerProvider.java
> --- 
> a/src/java.base/share/classes/java/net/spi/URLStreamHandlerProvider.java
> +++ 
> b/src/java.base/share/classes/java/net/spi/URLStreamHandlerProvider.java
> @@ -41,6 +41,9 @@
>   * fully-qualified concrete URL stream handler provider class names, 
> one per
>   * line.
>   *
> + * <p> URL stream handler providers are located at runtime, as 
> specified in the
> + * {@linkplain java.net.URL#URL(String,String,int,String) URL 
> constructor}.
> + *
>   * @since 1.9
>   */
>  public abstract class URLStreamHandlerProvider
>
> -Chris.



More information about the net-dev mailing list