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

[Hannes Wallnöfer]

Sorry, 

Sent this without seeing Jon’s more thorough replies, which showed as a separate thread in my mail reader. Please ignore.

Hannes

> Am 28.01.2019 um 13:26 schrieb Hannes Wallnöfer <hannes.wallnoefer at oracle.com>:
> 
> Hi Christoph,
> 
> Looks good to me and fixes the problem. I’m not sure whether the purpose of the original link was to have a verbatim <a> HTML tag, but that seems to be covered by the other links in that test.
> 
> Hannes
> 
>> Am 25.01.2019 um 11:12 schrieb Langer, Christoph <christoph.langer at sap.com>:
>> 
>> 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
>> Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8217773.0/
>> 
>> BTW: Looking at the generated Javadoc for method pkg.B::externalLink, one can see that the output for the “see also” section does not look optimal. The line of “java.net.URISyntaxException” starts with a comma which can’t be avoided. Don’t know if this merits a JBS issue or it is a corner case which is hardly ever used…
>> 
>> Thanks
>> Christoph
>