[lworld] RFR: 8368497: [lworld] redo: New lint category `initialization` for code that would not be allowed in the prologue [v2]
Vicente Romero
vromero at openjdk.org
Thu Oct 2 14:16:08 UTC 2025
On Wed, 1 Oct 2025 20:51:25 GMT, Chen Liang <liach at openjdk.org> wrote:
>> Vicente Romero has updated the pull request incrementally with one additional commit since the last revision:
>>
>> do not generate proxy variables for warned code
>
> test/lib/jdk/test/lib/NetworkConfiguration.java line 1:
>
>> 1: /*
>
> We should probably ignore initialization warnings for test libs at: https://github.com/openjdk/valhalla/blob/2bc50b323a5c57f5d55094196729f54b7dd98302/make/test/BuildTestLib.gmk#L65
yes we could, but I prefer not to. I found bugs in the original implementation by analyzing case by case the reason for the warnings. If we remove it for all tests then we could be blinded to lwrking bugs
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1650#discussion_r2398992215
More information about the valhalla-dev
mailing list