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

[Alan Bateman]

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.