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.gibb...@oracle.com>
*Sent:* Freitag, 25. Januar 2019 17:30
*To:* Langer, Christoph <christoph.lan...@sap.com>; javadoc-dev@openjdk.java.net
*Cc:* Zeller, Arno <arno.zel...@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

Reply via email to