RFR: 8354121: Use an anonymous class rather than a lambda in AbstractMemorySegmentImpl::cleanupAction [v3]
Per Minborg
pminborg at openjdk.org
Wed Apr 9 13:02:08 UTC 2025
> This PR proposes to use an anonymous class rather than a lambda in order to improve startup time.
>
> We need to make sure the regression is fixed by this. It might be the case that the regression is there because [JDK-8347047](https://bugs.openjdk.org/browse/JDK-8347047) actually fixed an issue so that the segments could be collected by the GC.
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 four additional commits since the last revision:
- Use a record instead of anno class
- Merge branch 'master' into ms-reinterpret-remove-lambda
- Add comment on anonymous class
- Replace lambda with anonymous class
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/24514/files
- new: https://git.openjdk.org/jdk/pull/24514/files/49c4b352..730c064f
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=24514&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=24514&range=01-02
Stats: 8587 lines in 274 files changed: 5246 ins; 2360 del; 981 mod
Patch: https://git.openjdk.org/jdk/pull/24514.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/24514/head:pull/24514
PR: https://git.openjdk.org/jdk/pull/24514
More information about the core-libs-dev
mailing list