Integrated: JDK-8280744: Allow SuppressWarnings to be used in all declaration contexts
Joe Darcy
darcy at openjdk.java.net
Fri Jan 28 00:48:13 UTC 2022
On Wed, 26 Jan 2022 21:22:58 GMT, Joe Darcy <darcy at openjdk.org> wrote:
> The SuppressWarnings annotation type is declared have multiple targets, including modules; however, "package" is left off of its target list. As package-info file are another kind of declaration where a compiler could give warnings, allowing SuppressWarnings in that case is reasonable. Wanting SuppressWarnings in package-info file came up while working on doclint warnings (JDK-8280534).
>
> Please also review the companion CSR:
> https://bugs.openjdk.java.net/browse/JDK-8280745
>
> While the SuppressWarnings annotation does have a JLS section (9.6.4.5. @SuppressWarnings), the section doesn't mention the target list therefore doesn't appear to need to be updated.
This pull request has now been integrated.
Changeset: 78574057
Author: Joe Darcy <darcy at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/78574057a4758fc3da5f39af77df09dc2232a9a1
Stats: 56 lines in 1 file changed: 37 ins; 12 del; 7 mod
8280744: Allow SuppressWarnings to be used in all declaration contexts
Reviewed-by: iris
-------------
PR: https://git.openjdk.java.net/jdk/pull/7239
More information about the core-libs-dev
mailing list