On Wed, 2 Apr 2025 09:21:17 GMT, Maurizio Cimadamore <mcimadam...@openjdk.org> wrote:
>> Per Minborg has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Add lazy toSting for StableMap::values >> - Make toString for reversed and sublist lazy > > src/java.base/share/classes/java/lang/StableValue.java line 378: > >> 376: * @param value to set >> 377: */ >> 378: boolean trySet(T value); > > This can probably throw `IllegalStateException` too right -- e.g. if used > inside a supplier passed to `orElseSet` ? Nice catch. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r2024766795