RFR: 8346294: Invalid lint category specified in compiler.properties [v2]

Maurizio Cimadamore mcimadamore at openjdk.org
Mon Dec 16 19:29:36 UTC 2024


On Mon, 16 Dec 2024 19:21:54 GMT, Archie Cobbs <acobbs at openjdk.org> wrote:

>> Please review this fix for an incorrect `lint:` tag in `compiler.properties`, plus an adjustment to the build process to automatically detect and fail the build in case of any similar typos in the future.
>
> Archie Cobbs has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Cleanups & refactoring based on review suggestions.

Marked as reviewed by mcimadamore (Reviewer).

src/jdk.compiler/share/classes/com/sun/tools/javac/code/Lint.java line 451:

> 449:         public void visitConstant(Attribute.Constant value) {
> 450:             if (value.type.tsym == syms.stringType.tsym) {
> 451:                 LintCategory.get((String)value.value)

I think use of `Optional` makes sense here too, so bonus point for the extra consolidation!

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

PR Review: https://git.openjdk.org/jdk/pull/22769#pullrequestreview-2507083357
PR Review Comment: https://git.openjdk.org/jdk/pull/22769#discussion_r1887385740


More information about the compiler-dev mailing list