Changeset 23436

Timestamp:

2010-12-10T20:40:28+13:00 (13 years ago)

Author:

ak19

Message:

Synchronising a couple of methods in FilenameEncoding to ensure proper lock on state variable (refreshRequired) used in conditions

File:

• main/trunk/gli/src/org/greenstone/gatherer/metadata/FilenameEncoding.java (modified) (1 diff)

main/trunk/gli/src/org/greenstone/gatherer/metadata/FilenameEncoding.java

@@ -69 +69 @@
     public static Map map = new HashMap();
 
+//*********************** BUSY REFRESHING / REQUIRING  REFRESH *********************
+
     /** Set to true if filename encoding metadata was changed. Called by the enter keyPress 
     * event in gui.EnrichPane and when the gs.FilenameEncoding field loses focus. */
     private static boolean refreshRequired = false;
-    
-//*********************** BUSY REFRESHING / REQUIRING  REFRESH *********************
-
-    public static boolean isRefreshRequired() { 
+
+    synchronized public static boolean isRefreshRequired() { 
         return refreshRequired; 
     }
     
-    public static void setRefreshRequired(boolean state) { 
+    synchronized public static void setRefreshRequired(boolean state) { 
         if(MULTIPLE_FILENAME_ENCODINGS_SUPPORTED) {
             refreshRequired = state;