RFR: 8351038: ConcurrentModificationException in EventType constructor [v3]
Andy Goryachev
angorya at openjdk.org
Tue Mar 18 16:57:31 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 six additional commits since the last revision:
- formatting
- Merge remote-tracking branch 'origin/master' into 8351038.event.type
- 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/552a645d..f44d3b7a
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jfx&pr=1729&range=02
- incr: https://webrevs.openjdk.org/?repo=jfx&pr=1729&range=01-02
Stats: 415 lines in 10 files changed: 391 ins; 14 del; 10 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