On Wed, 12 Nov 2025 16:33:40 GMT, Jorn Vernee <[email protected]> wrote:
>> Per Minborg has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 112 commits: >> >> - Clarify toString spec >> - Merge branch 'master' into lazy-constants >> - Add @AOTSafeClassInitializer >> - Address comments in PR >> - Fix merge mistake >> - Merge master >> - Rework toString implementations >> - Update after doc comments >> - Merge branch 'master' into lazy-constants >> - Revert the AbstractMap.keySet @Stable annotation >> - ... and 102 more: https://git.openjdk.org/jdk/compare/76a1109d...1f439bec > > src/java.base/share/classes/java/util/LazyCollections.java line 578: > >> 576: if (t == null) { >> 577: final T newValue = switch (functionHolder.function()) { >> 578: case Supplier<?> sup -> (T) sup.get(); > > Is the held function ever a Supplier? I don't see a FunctionHolder being > created with one. This is from when `FunctionHolder` was located in the `jdk.internal.lang.stable` package in [GH‑25878], and was used to implement the computing function freeing mechanism in [`StableValue::supplier(…)`]. [GH‑25878]: https://github.com/openjdk/jdk/pull/25878 [`StableValue::supplier(…)`]: https://docs.oracle.com/en/java/javase/25/docs/api/java.base/java/lang/StableValue.html#supplier(java.util.function.Supplier) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27605#discussion_r2519902578
