Re: RFR: 8299852: Modernize ConcurrentHashMap [v4]

2023-02-20 Thread Per Minborg
> `java.util.concurrent.ConcurrentHashMap` is relatively old and has not been > updated to reflect the current state of Java and can be modernized: > > * Add `@Serial` annotations > * Seal classes and restrict subclassing for internal classes > * Use pattern matching for instance > * Remove r

Re: RFR: 8299852: Modernize ConcurrentHashMap [v3]

2023-02-20 Thread Per Minborg
> `java.util.concurrent.ConcurrentHashMap` is relatively old and has not been > updated to reflect the current state of Java and can be modernized: > > * Add `@Serial` annotations > * Seal classes and restrict subclassing for internal classes > * Use pattern matching for instance > * Remove r

Re: RFR: 8299852: Modernize ConcurrentHashMap [v2]

2023-01-22 Thread Attila Szegedi
On Thu, 12 Jan 2023 09:28:46 GMT, Per Minborg wrote: >> `java.util.concurrent.ConcurrentHashMap` is relatively old and has not been >> updated to reflect the current state of Java and can be modernized: >> >> * Add `@Serial` annotations >> * Seal classes and restrict subclassing for internal

Re: RFR: 8299852: Modernize ConcurrentHashMap [v2]

2023-01-12 Thread Per Minborg
> `java.util.concurrent.ConcurrentHashMap` is relatively old and has not been > updated to reflect the current state of Java and can be modernized: > > * Add `@Serial` annotations > * Seal classes and restrict subclassing for internal classes > * Use pattern matching for instance > * Remove r

Re: RFR: 8299852: Modernize ConcurrentHashMap

2023-01-12 Thread ExE Boss
On Tue, 10 Jan 2023 10:32:34 GMT, Per Minborg wrote: > `java.util.concurrent.ConcurrentHashMap` is relatively old and has not been > updated to reflect the current state of Java and can be modernized: > > * Add `@Serial` annotations > * Seal classes and restrict subclassing for internal class

Re: RFR: 8299852: Modernize ConcurrentHashMap

2023-01-10 Thread Martin Buchholz
On Tue, 10 Jan 2023 10:32:34 GMT, Per Minborg wrote: > `java.util.concurrent.ConcurrentHashMap` is relatively old and has not been > updated to reflect the current state of Java and can be modernized: > > * Add `@Serial` annotations > * Seal classes and restrict subclassing for internal class

Re: RFR: 8299852: Modernize ConcurrentHashMap

2023-01-10 Thread Martin Buchholz
On Tue, 10 Jan 2023 10:32:34 GMT, Per Minborg wrote: > `java.util.concurrent.ConcurrentHashMap` is relatively old and has not been > updated to reflect the current state of Java and can be modernized: > > * Add `@Serial` annotations > * Seal classes and restrict subclassing for internal class

Re: RFR: 8299852: Modernize ConcurrentHashMap

2023-01-10 Thread Martin Buchholz
On Tue, 10 Jan 2023 10:32:34 GMT, Per Minborg wrote: > `java.util.concurrent.ConcurrentHashMap` is relatively old and has not been > updated to reflect the current state of Java and can be modernized: > > * Add `@Serial` annotations > * Seal classes and restrict subclassing for internal class

Re: RFR: 8299852: Modernize ConcurrentHashMap

2023-01-10 Thread Martin Buchholz
On Tue, 10 Jan 2023 10:32:34 GMT, Per Minborg wrote: > `java.util.concurrent.ConcurrentHashMap` is relatively old and has not been > updated to reflect the current state of Java and can be modernized: > > * Add `@Serial` annotations > * Seal classes and restrict subclassing for internal class

Re: RFR: 8299852: Modernize ConcurrentHashMap

2023-01-10 Thread Per Minborg
On Tue, 10 Jan 2023 12:04:10 GMT, Pavel Rappo wrote: > I'm sure others will note or have already noted elsewhere that we don't do > direct PRs against JSR 166. Yes. I will change the PR to a draft as we explore ways to integrate the spirit of the proposed changes. - PR: https://g

Re: RFR: 8299852: Modernize ConcurrentHashMap

2023-01-10 Thread Pavel Rappo
On Tue, 10 Jan 2023 10:32:34 GMT, Per Minborg wrote: > `java.util.concurrent.ConcurrentHashMap` is relatively old and has not been > updated to reflect the current state of Java and can be modernized: > > * Add `@Serial` annotations > * Seal classes and restrict subclassing for internal class