On Tue, 25 May 2021 17:32:11 GMT, Pavel Rappo <pra...@openjdk.org> wrote:
> Although the `com.sun.tools.javadoc.**` packages were removed in JDK-8215584, > some references to them remain, mainly in the form of links in doc comments. > Some of those links refer to classes that never existed, which I reckon is > due to a typo; for example, this reference > > com.sun.tools.javadoc.ClassDocImpl#findField > > should've looked like this > > com.sun.tools.javadoc.main.ClassDocImpl#findField > > Grep showed that some tests also use `com.sun.tools.javadoc` and some other > extinct packages. However, it is unclear if those "references" should also be > removed: > > 1. > https://github.com/openjdk/jdk/blob/739769c8fc4b496f08a92225a12d07414537b6c0/test/langtools/tools/javac/NoStringToLower.java#L60-L74 > 2. > https://github.com/openjdk/jdk/blob/739769c8fc4b496f08a92225a12d07414537b6c0/test/langtools/tools/javac/T8003967/DetectMutableStaticFields.java#L73-L85 This pull request has now been integrated. Changeset: 4343997a Author: Pavel Rappo <pra...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/4343997a1a2b38581488932f6a4971ce330bd467 Stats: 24 lines in 6 files changed: 1 ins; 13 del; 10 mod 8267708: Remove references to com.sun.tools.javadoc.** This commit changes BaseOptions.java to fix the NoStringToLower test that failed after jdk.javadoc has been added to the list of package prefixes that NoStringToLower scans. Reviewed-by: jjg ------------- PR: https://git.openjdk.java.net/jdk/pull/4192
