RFR: 8265754: Move suspend/resume API from HandshakeState [v8]
Anton Artemov
duke at openjdk.org
Thu Jun 5 07:49:37 UTC 2025
> Hi,
>
> in this PR the suspend/resume methods are moved away from the HandshakeState class into a new separate class HandshakeSuspender. The idea is that the state class should be a state keeper and should not be responsible for actions which change the state. Such actions should be done by a separate class.
>
> Tested in tiers 1-4.
Anton Artemov 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 25 additional commits since the last revision:
- 8265754: Added a comment.
- Merge remote-tracking branch 'origin/master' into JDK-8265754
- 8265754: Fixed build problem
- 8265754: Fixed typo.
- 8265754: Fixing spaces.
- 8265754: Refactoring into separate file.
- 8265754: Refactoring.
- 8265754: Refactoring.
- 8265754: Refactoring.
- 8265754: Refactoring.
- ... and 15 more: https://git.openjdk.org/jdk/compare/b18dbfea...c07b29ea
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/25407/files
- new: https://git.openjdk.org/jdk/pull/25407/files/4c984b2e..c07b29ea
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=25407&range=07
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=25407&range=06-07
Stats: 18166 lines in 446 files changed: 15049 ins; 1382 del; 1735 mod
Patch: https://git.openjdk.org/jdk/pull/25407.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/25407/head:pull/25407
PR: https://git.openjdk.org/jdk/pull/25407
More information about the hotspot-runtime-dev
mailing list