Integrated: 8267708: Remove references to com.sun.tools.javadoc.**
Pavel Rappo
prappo at openjdk.java.net
Wed May 26 11:30:16 UTC 2021
On Tue, 25 May 2021 17:32:11 GMT, Pavel Rappo <prappo at 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 <prappo at 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
More information about the compiler-dev
mailing list