On Thu, 18 Dec 2025 13:11:05 GMT, Coleen Phillimore <[email protected]> wrote:
> This replaces Locale with Calendar and removes BaseLocale to test loading > with ClassFileLoadHook, like my previous pull request. Also removes the test > from the ProblemList-enable-valhalla.txt file. > > BaseLocale is also loaded and I thought it was the migrated classes that > caused early loading for Locale, but it was really the module patching that > we used to do for --enable-preview. See call stack in bug report. > > Tested with the test on windows, where it failed. This pull request has now been integrated. Changeset: 1e6e72e2 Author: Coleen Phillimore <[email protected]> URL: https://git.openjdk.org/valhalla/commit/1e6e72e246608d872d7733559da7ba29ed10f69a Stats: 6 lines in 2 files changed: 0 ins; 2 del; 4 mod 8372686: [lworld] runtime/cds/serviceability/ReplaceCriticalClasses.java fails with --enable-preview Reviewed-by: iklam ------------- PR: https://git.openjdk.org/valhalla/pull/1827
