Changeset 4920 in josm

Timestamp:

2012-02-12T14:00:28+01:00 (13 years ago)

Author:

stoecker

Message:

removed preferences not stored in file

File:

• trunk/src/org/openstreetmap/josm/data/Preferences.java (modified) (9 diffs)

trunk/src/org/openstreetmap/josm/data/Preferences.java

@@ -266 +266 @@
     }
 
+    /* remove end of 2012 */
     public File getOldPreferenceFile() {
         return new File(getPreferencesDirFile(), "preferences");
@@ -907 +908 @@
     }
 
+    /* remove this workaround end of 2012, replace by direct access to structure */
     synchronized private List<String> getCollectionInternal(String key) {
         List<String> prop = collectionProperties.get(key);
@@ -949 +951 @@
                 if (valueCopy.contains(null)) throw new RuntimeException("Error: Null as list element in preference setting (key '"+key+"')");
                 collectionProperties.put(key, Collections.unmodifiableList(valueCopy));
-                try {
-                    save();
-                } catch(IOException e){
-                    System.out.println(tr("Warning: failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
-                }
+            }
+            try {
+                save();
+            } catch(IOException e){
+                System.out.println(tr("Warning: failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
             }
         }
@@ -1027 +1029 @@
     }
 
+    /* remove this workaround end of 2012 and replace by direct array access */
     synchronized private List<List<String>> getArrayInternal(String key) {
         List<List<String>> prop = arrayProperties.get(key);
@@ -1078 +1081 @@
                 }
                 arrayProperties.put(key, Collections.unmodifiableList(valueCopy));
-                try {
-                    save();
-                } catch(IOException e){
-                    System.out.println(tr("Warning: failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
-                }
+            }
+            try {
+                save();
+            } catch(IOException e){
+                System.out.println(tr("Warning: failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
             }
         }
@@ -1123 +1126 @@
     }
 
+    /* remove this workaround end of 2012 and use direct access to proper variable */
     private synchronized List<Map<String, String>> getListOfStructsInternal(String key) {
         List<Map<String, String>> prop = listOfStructsProperties.get(key);
@@ -1177 +1181 @@
                 }
                 listOfStructsProperties.put(key, Collections.unmodifiableList(valueCopy));
-                try {
-                    save();
-                } catch(IOException e){
-                    System.out.println(tr("Warning: failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
-                }
+            }
+            try {
+                save();
+            } catch(IOException e){
+                System.out.println(tr("Warning: failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
             }
         }
@@ -1220 +1224 @@
     /**
      * Get a list of hashes which are represented by a struct-like class.
-     * It reads lines of the form
-     *  > key.0=prop:val \u001e prop:val \u001e ... \u001e prop:val
-     *  > ...
-     *  > key.N=prop:val \u001e prop:val \u001e ... \u001e prop:val
      * Possible properties are given by fields of the class klass that have
      * the @pref annotation.
@@ -1642 +1642 @@
             /* don't save default values */
             if(s == null || !s.equals(r)) {
+                /* TODO: remove old format exception end of 2012 */
                 if(r.contains("\u001e"))
                 {