Changeset 14122 in josm for trunk/src

Timestamp:

2018-08-11T18:18:58+02:00 (6 years ago)

Author:

Don-vip

Message:

see #15229 - minor preferences refactoring

Location:

trunk/src/org/openstreetmap/josm

Files:

• data/Preferences.java (modified) (1 diff)
• spi/preferences/AbstractPreferences.java (modified) (1 diff)
• spi/preferences/MemoryPreferences.java (modified) (5 diffs)

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

@@ -693 +693 @@
     }
 
-    /**
-     * Gets a map of all settings that are currently stored
-     * @return The settings
-     */
+    @Override
     public Map<String, Setting<?>> getAllSettings() {
         return new TreeMap<>(settingsMap);

trunk/src/org/openstreetmap/josm/spi/preferences/AbstractPreferences.java

@@ -124 +124 @@
 
     /**
+     * Gets a map of all settings that are currently stored
+     * @return The settings
+     */
+    public abstract Map<String, Setting<?>> getAllSettings();
+
+    /**
      * Set a value for a certain setting. The changed setting is saved to the preference file immediately.
      * Due to caching mechanisms on modern operating systems and hardware, this shouldn't be a performance problem.

trunk/src/org/openstreetmap/josm/spi/preferences/MemoryPreferences.java

@@ -7 +7 @@
 import java.util.Objects;
 import java.util.Set;
+import java.util.TreeMap;
 
 /**
@@ -22 +23 @@
     @Override
     public boolean putSetting(String key, Setting<?> setting) {
-        Setting current = settings.get(key);
+        Setting<?> current = settings.get(key);
         if (setting == null) {
             settings.remove(key);
@@ -33 +34 @@
     @Override
     public <T extends Setting<?>> T getSetting(String key, T def, Class<T> klass) {
-        Setting current = settings.get(key);
+        Setting<?> current = settings.get(key);
         if (current != null && klass.isInstance(current)) {
             @SuppressWarnings("unchecked")
@@ -45 +46 @@
     public Set<String> getKeySet() {
         return Collections.unmodifiableSet(settings.keySet());
+    }
+
+    @Override
+    public Map<String, Setting<?>> getAllSettings() {
+        return new TreeMap<>(settings);
     }
 
@@ -66 +72 @@
         // do nothing
     }
-
 }