RFR: 8351038: ConcurrentModificationException in EventType constructor [v2]

Andy Goryachev angorya at openjdk.org
Mon Mar 10 14:31:33 UTC 2025


> - synchronized `EventType::register()` method
> - simplified internal constructor which is only used for `EventType.ROOT`
> 
> There should negligent impact on performance when `EventTypes` are created in the FX Application Thread.
> 
> There might be a distant potential for a deadlock if the application wraps code that creates `EventTypes` in a block synchronized on a different object.

Andy Goryachev 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:

 - ignored
 - Merge remote-tracking branch 'origin/master' into 8351038.event.type
 - test
 - synchronized

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/1729/files
  - new: https://git.openjdk.org/jfx/pull/1729/files/5492c550..552a645d

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

  Stats: 866 lines in 23 files changed: 802 ins; 28 del; 36 mod
  Patch: https://git.openjdk.org/jfx/pull/1729.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/1729/head:pull/1729

PR: https://git.openjdk.org/jfx/pull/1729


More information about the openjfx-dev mailing list