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

[Chris Hegarty]

On 8 Jun 2016, at 03:36, Hamlin Li <huaming.li at oracle.com> wrote:
> 
> On 2016/6/8 4:27, Bradford Wetmore wrote:
>> May I suggest using example.com [1] and an arbitrary path instead on these two examples, so that there's no chance of it being mistaken for a valid URL.  This example uses old hostnames that might change yet again (docs.oracle.com) and an ancient JDK platform (1.3), let's just remove that confusion now.
> Thank you Brad.
> Sure, your suggestion sounds reasonable, I should have made it more clear that the changed URLs in doc are not valid URLs, they just sample.
> Please check webrev: http://cr.openjdk.java.net/~mli/8158881/webrev.01/

Thank you Hamlin ( and Brad ), this looks good.

-Chris.

> Thank you
> -Hamlin
>> 
>> Thanks,
>> 
>> Brad
>> 
>> [1] RFC 2606:  https://tools.ietf.org/html/rfc2606
>> 
>> On 6/7/2016 2:34 AM, Chris Hegarty wrote:
>>> 
>>>> On 7 Jun 2016, at 05:50, Hamlin Li <huaming.li at 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>
>>>> *
>>>> 
>>> 
>