You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2009/03/14 17:02:19 UTC

svn commit: r754469 - /commons/proper/lang/trunk/src/java/org/apache/commons/lang/LocaleUtils.java

Author: sebb
Date: Sat Mar 14 16:02:18 2009
New Revision: 754469

URL: http://svn.apache.org/viewvc?rev=754469&view=rev
Log:
Add synch for lazily created static field

Modified:
    commons/proper/lang/trunk/src/java/org/apache/commons/lang/LocaleUtils.java

Modified: commons/proper/lang/trunk/src/java/org/apache/commons/lang/LocaleUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/java/org/apache/commons/lang/LocaleUtils.java?rev=754469&r1=754468&r2=754469&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/java/org/apache/commons/lang/LocaleUtils.java (original)
+++ commons/proper/lang/trunk/src/java/org/apache/commons/lang/LocaleUtils.java Sat Mar 14 16:02:18 2009
@@ -42,7 +42,8 @@
     /** Unmodifiable list of available locales. */
     private static final List cAvailableLocaleList;
     /** Unmodifiable set of available locales. */
-    private static Set cAvailableLocaleSet;
+    //@GuardedBy("this")
+    private static Set cAvailableLocaleSet; // lazily created by availableLocaleSet()
     /** Unmodifiable map of language locales by country. */
     private static final Map cLanguagesByCountry = Collections.synchronizedMap(new HashMap());
     /** Unmodifiable map of country locales by language. */
@@ -205,7 +206,7 @@
      *
      * @return the unmodifiable set of available locales
      */
-    public static Set availableLocaleSet() {
+    public static synchronized Set availableLocaleSet() {
         Set set = cAvailableLocaleSet;
         if (set == null) {
             set = new HashSet(availableLocaleList());