RFR: 8351038: ConcurrentModificationException in EventType constructor [v3]

Michael Strauß mstrauss at openjdk.org
Wed Mar 19 12:36:22 UTC 2025


On Tue, 18 Mar 2025 16:57:31 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> - 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

Marked as reviewed by mstrauss (Reviewer).

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

PR Review: https://git.openjdk.org/jfx/pull/1729#pullrequestreview-2698244536


More information about the openjfx-dev mailing list