On Tue, 12 Aug 2025 09:21:07 GMT, Per Minborg <pminb...@openjdk.org> wrote:

>> This PR proposes to release the underlying function if a stable function or 
>> collection has invoked its underlying supplier exhaustively so that it can 
>> be collected.
>> 
>> This PR passes tier1, tier2, and tier3 testing on multiple platforms.
>
> Per Minborg has updated the pull request with a new target base due to a 
> merge or a rebase. The incremental webrev excludes the unrelated changes 
> brought in by the merge/rebase. The pull request contains 12 additional 
> commits since the last revision:
> 
>  - Add comment for outer.outer
>  - Merge branch 'master' into sv-release-supplier2
>  - Merge branch 'master' into sv-release-supplier2
>  - Add a new method to StableValueImpl for improved speed
>  - Merge branch 'master' into sv-release-supplier2
>  - Fix performance regression
>  - Clean up
>  - Fix bug
>  - Simplify
>  - Revert to records
>  - ... and 2 more: https://git.openjdk.org/jdk/compare/b367a692...243479df

src/java.base/share/classes/jdk/internal/lang/stable/StableEnumFunction.java 
line 52:

> 50:  *                         of the valid inputs (as there might be "holes")
> 51:  * @param delegates        a delegate array of inputs to StableValue 
> mappings
> 52:  * @param underlyingHolder of the original underlying Function

Suggestion:

 * @param underlyingHolder the holder of the original underlying Function

src/java.base/share/classes/jdk/internal/lang/stable/StableFunction.java line 
45:

> 43:  *
> 44:  * @param values           a delegate map of inputs to StableValue 
> mappings
> 45:  * @param underlyingHolder of the original underlying Function

Suggestion:

 * @param underlyingHolder the holder of the original underlying Function

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25878#discussion_r2273099020
PR Review Comment: https://git.openjdk.org/jdk/pull/25878#discussion_r2273102356

Reply via email to