[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 16:50:30 UTC 2025
On Wed, 1 Oct 2025 20:47:10 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/langtools/tools/javac/SuperInit/InitializationWarningTester.java line 143:
>
>> 141: javaFile = p;
>> 142: } else if (p.toString().endsWith("out")) {
>> 143: goldenFile = p;
>
> Can we just do `baseDir.resolve(className + ".java")`, and do a `Files.exists` check for the golden file, instead of having complicated stuf like this?
yep I wanted to clean that before pushing but then I forgot, thanks
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1650#discussion_r2399414143
More information about the valhalla-dev
mailing list