RFR (S): 8217773: Test langtools/jdk/javadoc/doclet/testLinkOption/TestRedirectLinks.java fails after JDK-8217034

[Jonathan Gibbons]

Thanks for the suggestion. If that is sufficient for now, that may be 
the best solution.

Maybe there's a future-RFE to exclude specific references from the link 
checker.

-- Jon

On 1/25/19 8:38 AM, Langer, Christoph wrote:
>
> Hi Jon,
>
> maybe the link check just has to be skipped for the 
> URISyntaxException. The problem seems to be that the (hardcoded) link 
> at this place links to a location that does not exist – but this is 
> checked. So, looking forward to your fix ��
>
> Thanks & Best regards
>
> Christoph
>
> *From:*Jonathan Gibbons <jonathan.gibbons at oracle.com>
> *Sent:* Freitag, 25. Januar 2019 17:30
> *To:* Langer, Christoph <christoph.langer at sap.com>; 
> javadoc-dev at openjdk.java.net
> *Cc:* Zeller, Arno <arno.zeller at sap.com>
> *Subject:* Re: RFR (S): 8217773: Test 
> langtools/jdk/javadoc/doclet/testLinkOption/TestRedirectLinks.java 
> fails after JDK-8217034
>
> Christoph,
>
> Thanks for reporting the problem.
>
> The issue may be real, but at first glance, your fix looks incorrect. 
> The test is specifically about testing a "<a href...>" and so it is 
> incorrect to replace it to the more conventional "@see ..." tag.
>
> I will investigate in more detail later today.
>
> -- Jon
>
> On 1/25/19 12:10 AM, Langer, Christoph wrote:
>
>     Hi,
>
>     it seems that after JDK-8217034, the test
>     langtools/jdk/javadoc/doclet/testLinkOption/TestRedirectLinks.java
>     is broken. The fix looks trivial, please review:
>
>     Bug: https://bugs.openjdk.java.net/browse/JDK-8217773
>     <https://bugs.openjdk.java.net/browse/JDK-8217773>
>
>     Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8217773.0/
>     <http://cr.openjdk.java.net/~clanger/webrevs/8217773.0/>
>
>     Thanks
>
>     Christoph
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/javadoc-dev/attachments/20190125/59a856dd/attachment.html>