RFR: 8265754: Move suspend/resume API from HandshakeState [v2]

Anton Artemov duke at openjdk.org
Mon May 26 13:30:11 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 10 additional commits since the last revision:

 - Merge remote-tracking branch 'origin/master' into JDK-8265754
 - Merge remote-tracking branch 'origin/master' into JDK-8265754
 - 8265754: Refactoring to static methods.
 - 8265754: Fixed indentation.
 - 8265754: Fixed indentation
 - 8265754: Fixed indentation and lost newline.
 - 8265754: Fixed whitespaces errors.
 - 8265754: More work.
 - Merge remote-tracking branch 'origin/master' into JDK-8265754
 - 8265754: Refactoring suspend/resume in the HandshakeState away

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/25407/files
  - new: https://git.openjdk.org/jdk/pull/25407/files/b852ba9c..0ee1abf2

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=25407&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25407&range=00-01

  Stats: 6334 lines in 76 files changed: 4311 ins; 1622 del; 401 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