RFR: 8291027: Some of TimeZone methods marked 'synchronized' unnecessarily

2024-01-16 Thread Andrey Turbanov
8291027: Some of TimeZone methods marked 'synchronized' unnecessarily - Commit messages: - 8291027: Some of TimeZone methods marked 'synchronized' unnecessarily Changes: https://git.openjdk.org/jdk/pull/17441/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=17441&range=00 I

Re: RFR: 8291027: Some of TimeZone methods marked 'synchronized' unnecessarily

2024-03-05 Thread Andrey Turbanov
On Tue, 16 Jan 2024 10:19:44 GMT, Andrey Turbanov wrote: > There are 3 methods in `java.util.TimeZone` which are `public static` and > marked as `synchronized`: > 1. getTimeZone(String) > 2. getAvailableIDs(int) > 3. getAvailableIDs() > > This means it is a bottle neck for the whole VM. > I've

Re: RFR: 8291027: Some of TimeZone methods marked 'synchronized' unnecessarily

2024-01-16 Thread Andrey Turbanov
On Tue, 16 Jan 2024 10:19:44 GMT, Andrey Turbanov wrote: > 8291027: Some of TimeZone methods marked 'synchronized' unnecessarily src/java.base/share/classes/java/util/TimeZone.java line 629: > 627: */ > 628: public static String[] getAvailableIDs(int rawOffset) { > 629: return

Re: RFR: 8291027: Some of TimeZone methods marked 'synchronized' unnecessarily

2024-01-17 Thread Naoto Sato
On Tue, 16 Jan 2024 11:12:44 GMT, Andrey Turbanov wrote: >> 8291027: Some of TimeZone methods marked 'synchronized' unnecessarily > > src/java.base/share/classes/java/util/TimeZone.java line 629: > >> 627: */ >> 628: public static String[] getAvailableIDs(int rawOffset) { >> 629:

Re: RFR: 8291027: Some of TimeZone methods marked 'synchronized' unnecessarily

2024-01-24 Thread Andrey Turbanov
On Wed, 17 Jan 2024 18:19:18 GMT, Naoto Sato wrote: >> src/java.base/share/classes/java/util/TimeZone.java line 629: >> >>> 627: */ >>> 628: public static String[] getAvailableIDs(int rawOffset) { >>> 629: return ZoneInfo.getAvailableIDs(rawOffset); >> >> BTW can we call `ZoneI