On Fri, 17 Oct 2025 11:27:47 GMT, Marc Chevalier <[email protected]> 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 This pull request has now been integrated. Changeset: e0aaf290 Author: Marc Chevalier <[email protected]> URL: https://git.openjdk.org/valhalla/commit/e0aaf29064ed67c240f5275e9b3d324ce5550382 Stats: 1587 lines in 54 files changed: 40 ins; 53 del; 1494 mod 8368988: [lworld] Rename classes in valhalla/inlinetypes to be distinct Reviewed-by: thartmann ------------- PR: https://git.openjdk.org/valhalla/pull/1683
