[lworld] RFR: 8368988: [lworld] Rename classes in valhalla/inlinetypes to be distinct [v3]

Marc Chevalier mchevalier at openjdk.org
Tue Oct 21 10:23:53 UTC 2025


On Tue, 21 Oct 2025 08:22:10 GMT, Marc Chevalier <mchevalier at openjdk.org> wrote:

>> Various cleanup in test package `compiler.valhalla.inlinetypes`:
>> - rename duplicated classes so that IDEs are nice, resolve correctly. New names are a bit... arbitrary. I'm happy to rename name if anyone has opinions. When writing new tests, let's prefer nested classes if we can.
>> - add `package compiler.valhalla.inlinetypes;` where it was missing. IDEs like that, and I vaguely remember we should.
>> - fix flags to take the new path into account, new package, or class renaming. Please, try to pay attention to that: I've did multiple passes to catch'em all, but this is easy to miss.
>> - remove some useless `import`.
>> - some few typos in comments.
>> - maybe more?
>> 
>> Thanks,
>> Marc
>
> Marc Chevalier has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits:
> 
>  - Merge remote-tracking branch 'upstream/lworld' into JDK-8368988
>  - More consistent naming convention
>  - Merge remote-tracking branch 'origin/lworld' into JDK-8368988
>  - post rebase fixes
>  - Fixes
>  - Some cleanup

Thanks @TobiHartmann for the review!

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

PR Comment: https://git.openjdk.org/valhalla/pull/1683#issuecomment-3425825494


More information about the valhalla-dev mailing list