Re: RFR of JDK-8158881: Doc typo in src/../java/net/URI.java

[Hamlin Li]

On 2016/6/7 17:34, Chris Hegarty wrote:
On 7 Jun 2016, at 05:50, Hamlin Li <huaming...@oracle.com> wrote:
Would you please review the following simple doc patch?

bug: https://bugs.openjdk.java.net/browse/JDK-8158881
webrev: http://cr.openjdk.java.net/~mli/8158881/webrev.00/
I’m not sure why the URL’s were ever changed from java.sun.com, since they are 
not
hyperlinks. This was a bad mistake, and badly broke the documentation, for an
unseeingly related change.
Thank you Chris for taking a look.
What about the scheme, shouldn’t that be updated to ‘https' too ?
Not sure if I understand you correctly, please correct me otherwise. 
It's not necessary to update to "https", because it's only sample code, 
and it also works well with "http".

Thank you
-Hamlin
-Chris.

Thank you
-Hamlin


--- a/src/java.base/share/classes/java/net/URI.java    Tue Jun 07 10:33:38 2016 
+0800
+++ b/src/java.base/share/classes/java/net/URI.java    Mon Jun 06 21:47:44 2016 
-0700
@@ -183,7 +183,7 @@
  * &nbsp;&nbsp;&nbsp;&nbsp;(1)
  * </blockquote>
  *
- * against the base URI {@code http://java.sun.com/j2se/1.3/} is the result
+ * against the base URI {@code http://docs.oracle.com/javase/1.3/} is the 
result
  * URI
  *
  * <blockquote>
@@ -193,13 +193,13 @@
  * Resolving the relative URI
  *
  * <blockquote>
- * {@code 
../../../demo/jfc/SwingSet2/src/SwingSet2.java}&nbsp;&nbsp;&nbsp;&nbsp;(2)
+ * {@code 
../../demo/jfc/SwingSet2/src/SwingSet2.java}&nbsp;&nbsp;&nbsp;&nbsp;(2)
  * </blockquote>
  *
  * against this result yields, in turn,
  *
  * <blockquote>
- * {@code http://java.sun.com/j2se/1.3/demo/jfc/SwingSet2/src/SwingSet2.java}
+ * {@code http://docs.oracle.com/demo/jfc/SwingSet2/src/SwingSet2.java}
  * </blockquote>
  *