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