Integrated: 8339550: Enable javac lint options for tests

Kevin Rushforth kcr at openjdk.org
Mon Sep 16 12:50:15 UTC 2024


On Fri, 13 Sep 2024 14:52:22 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

> This PR enables the `removal` lint option, along with `-Werror`, for compiling shims and test classes. In order to do that, I had to also do the following:
> 
> * Disable the `options` lint warning as is done for the `javafx.swing` module, and for the same reason: using `-source NN -release NN` produces a warning that we know about and don't care to see (and will fail the build with `-Werror`)
> * Change the `apiguardian` dependency from `testRuntimeOnly` to `testImplementation`, since it needs to be a compile-time dependency to avoid many compile-time warnings for a missing annotation
> * Add a missing `@SuppressWarnings("removal")` in one place
> * Fix the varargs warnings filed in [JDK-8338340](https://bugs.openjdk.org/browse/JDK-8338340)
> 
> I tested locally and with our CI test system. The GHA build passes.

This pull request has now been integrated.

Changeset: 3e6a2724
Author:    Kevin Rushforth <kcr at openjdk.org>
URL:       https://git.openjdk.org/jfx/commit/3e6a27240abcffd500a7150a8e934918683d6173
Stats:     10 lines in 4 files changed: 3 ins; 0 del; 7 mod

8339550: Enable javac lint options for tests
8338340: [TestBug] fix varargs parameter type warnings

Reviewed-by: angorya, mhanl

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

PR: https://git.openjdk.org/jfx/pull/1565


More information about the openjfx-dev mailing list