> This change fixes a whole slew of shortcomings in the redirection of relative > links in doc comments. The basic idea is that relative links are authored to > work in their "native primary" environment (e.g. the package summary page > for a package or the class page for a class and its members), and have to be > rewritten when used in other contexts such as "use" or index pages. > > A list of omissions that are fixed in this change: > > - Relative links in class or member comments were not redirected when > inherited by other classes > - Relative links in package comments were not rewritten when displayed in > other package summaries as "related packages" > - Fragment links used in foreign contexts were not completed with the file > name > - Relative links in module comments were not redirected at all > > While fixing above issues I also made sure link rewriting is kept to a > minimum, avoiding it as much as possible for elements that live in the same > package. > > Furthermore, the test for redirected relative links was a bit out of order. > The `javadoc` command issued by the test returned `ERROR` because one of the > source files contained non-valid HTML (an anchor with a `name` attribute to > test whether that attribute would be modified). Because of this, the > `checkLinks()` method was never invoked, which is a problem for a test that > is supposed to make sure generated links are valid. I changed the test to use > the valid `id` attribute instead of `name` and made sure `checkLinks()` is > executed again. > > I also added checks for the newly supported cases. I added a whole new test > for modules since retrofitting the existing test to cover modules would not > have been practical.
Hannes Wallnöfer has updated the pull request incrementally with one additional commit since the last revision: JDK-8262886: Fix copyright headers ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/4459/files - new: https://git.openjdk.java.net/jdk/pull/4459/files/89945cce..0a597560 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4459&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4459&range=00-01 Stats: 4 lines in 4 files changed: 0 ins; 0 del; 4 mod Patch: https://git.openjdk.java.net/jdk/pull/4459.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/4459/head:pull/4459 PR: https://git.openjdk.java.net/jdk/pull/4459
