Re: RFR: 8351038: ConcurrentModificationException in EventType constructor [v3]

2025-03-19 Thread Andy Goryachev
On Tue, 18 Mar 2025 17:05:17 GMT, Ambarish Rapte wrote: >> 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 >> comm

Re: RFR: 8351038: ConcurrentModificationException in EventType constructor [v3]

2025-03-19 Thread Michael Strauß
On Tue, 18 Mar 2025 16:57:31 GMT, Andy Goryachev 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.

Re: RFR: 8351038: ConcurrentModificationException in EventType constructor [v3]

2025-03-18 Thread Ambarish Rapte
On Tue, 18 Mar 2025 16:57:31 GMT, Andy Goryachev 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.

Re: RFR: 8351038: ConcurrentModificationException in EventType constructor [v3]

2025-03-18 Thread Andy Goryachev
> - 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

Re: RFR: 8351038: ConcurrentModificationException in EventType constructor [v2]

2025-03-18 Thread Ambarish Rapte
On Mon, 10 Mar 2025 14:31:33 GMT, Andy Goryachev 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.

Re: RFR: 8351038: ConcurrentModificationException in EventType constructor

2025-03-11 Thread Michael Strauß
On Wed, 5 Mar 2025 18:40:36 GMT, Andy Goryachev 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. > >

Re: RFR: 8351038: ConcurrentModificationException in EventType constructor [v2]

2025-03-11 Thread Andy Goryachev
> - 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