Integrated: 8338834: Remove unused import declarations in java.compiler

[Pavel Rappo]

On Thu, 22 Aug 2024 16:12:27 GMT, Pavel Rappo <prappo at openjdk.org> wrote:

> Please review this simple cleanup. I tested it first by usual means, and then additionally compared documentation before and after the change.
> 
> How did I end up with this cleanup? I finally got tired of an eyesore which greyed-out unused imports in my IDE are. So I removed them as per IDE suggestion and then had some fun chasing bugs. Who would've thought that removing imports can be so painful?
> 
> During my bug chase, I created the following issues, from which you could guess the kind of bugs I ran into:
> 
>   * https://youtrack.jetbrains.com/issue/IDEA-358082/Import-of-a-symbol-referred-to-by-snippet-target-is-considered-unused
>   * https://youtrack.jetbrains.com/issue/IDEA-358073/Support-changes-in-documentation-inheritance
>   * https://bugs.openjdk.org/browse/JDK-8338833
> 
> And a bonus one:
> 
>   * https://bugs.openjdk.org/browse/JDK-8338838

This pull request has now been integrated.

Changeset: a5e28005
Author:    Pavel Rappo <prappo at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/a5e28005fa95426f811e1ed98a7d726cbdbe196d
Stats:     56 lines in 29 files changed: 0 ins; 46 del; 10 mod

8338834: Remove unused import declarations in java.compiler

Reviewed-by: darcy

-------------

PR: https://git.openjdk.org/jdk/pull/20679