RFR: 8349847: Support configuring individual lint categories as errors [v9]
Archie Cobbs
acobbs at openjdk.org
Thu Aug 14 20:58:22 UTC 2025
On Thu, 14 Aug 2025 14:47:53 GMT, Vicente Romero <vromero at openjdk.org> wrote:
>> Archie Cobbs has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 16 commits:
>>
>> - Address review comments.
>> - Merge branch 'master' into JDK-8349847
>> - Merge branch 'master' into JDK-8349847 to fix cross-compiling build problem.
>> - Merge branch 'master' into JDK-8349847 to fix conflicts.
>> - Merge branch 'master' into JDK-8349847 to fix conflicts.
>> - Merge branch 'JDK-8359596' into JDK-8349847 to add "-Xlint:none" fix.
>> - Ensure that "-Xlint:none" still works for the affected warnings.
>>
>> The extra checks for "-Xlint:none" are needed now because of JDK-8352612,
>> which changed the behavior of "-Xlint:none" to no longer imply "-nowarn",
>> which allowed the affected warnings to get away with skipping that check.
>> - Minor naming and Javadoc tweaks.
>> - Merge branch 'JDK-8359596' into JDK-8349847
>> - No need for /nodynamiccopyright/ with this test.
>> - ... and 6 more: https://git.openjdk.org/jdk/compare/8cd79752...e8f5539f
>
> I'm OK with the last version. It could be said that more tests could be added but at the same time given that we are reusing code paths already tested for `Xlint`, then we should be fine. Kind of on the fence here
@vicente-romero-oracle thanks for the review!
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23622#issuecomment-3189847960
More information about the compiler-dev
mailing list