<i18n dev> RFR: 8263090: Avoid reading volatile fields twice in Locale.getDefault(Category)

Naoto Sato naoto at openjdk.java.net
Fri Mar 5 17:32:06 UTC 2021


On Fri, 5 Mar 2021 14:14:14 GMT, Claes Redestad <redestad at openjdk.org> wrote:

> This patch refactors Locale.getDefault(Category) so that the volatile field holding the Locale is typically only read once. This has a small performance advantage, and might be more robust if initialization is racy.

src/java.base/share/classes/java/util/Locale.java line 959:

> 957:     }
> 958: 
> 959:     private static Locale getDisplayLocale() {

Should this be `synchronized`?

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

PR: https://git.openjdk.java.net/jdk/pull/2845


More information about the i18n-dev mailing list