> 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. What about the scheme, shouldn’t that be updated to ‘https' too ? -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 @@ > * (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} (2) > + * {@code > ../../demo/jfc/SwingSet2/src/SwingSet2.java} (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> > * >