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 <[email protected]>: > > 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
