On Fri, 9 May 2025 20:25:40 GMT, Vicente Romero <vrom...@openjdk.org> wrote:

> I was working on something in this line too, just pushed the patch

Looks reasonable, a couple of comments...

Can't `LintCategory.alias` be `final`? If so I think this obviates the patch to 
`DetectMutableStaticFields.java`.

This is not a general solution to aliasing of lint categories. For example, if 
someone were to add an alias `filenames` for `path`, then [this 
code](https://github.com/openjdk/jdk/blob/68a118509a562334eb8c099129d79520041b10cf/src/jdk.compiler/share/classes/com/sun/tools/javac/main/Arguments.java#L506-L507)
 and a few other examples like it would break.

Do we care about that? If so then we might want to implement a proper fix for 
lint aliases as a separate pre-requisite to this issue.

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

PR Comment: https://git.openjdk.org/jdk/pull/24746#issuecomment-2867878715

Reply via email to