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