Changeset 10217 in josm for trunk/src/org/openstreetmap/josm/gui/tagging/TagEditorModel.java

Timestamp:

2016-05-15T16:37:33+02:00 (8 years ago)

Author:

Don-vip

Message:

findbugs - SF_SWITCH_NO_DEFAULT + various sonar fixes

File:

• trunk/src/org/openstreetmap/josm/gui/tagging/TagEditorModel.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/gui/tagging/TagEditorModel.java

@@ -80 +80 @@
     }
 
+    /**
+     * Adds property change listener.
+     * @param listener property change listener to add
+     */
     public void addPropertyChangeListener(PropertyChangeListener listener) {
         propChangeSupport.addPropertyChangeListener(listener);
@@ -102 +106 @@
     }
 
+    /**
+     * Removes property change listener.
+     * @param listener property change listener to remove
+     */
     public void removePropertyChangeListener(PropertyChangeListener listener) {
         propChangeSupport.removePropertyChangeListener(listener);
@@ -146 +154 @@
             case 1:
                 String v = (String) value;
-                if (tag.getValueCount() > 1 && !v.isEmpty()) {
-                    updateTagValue(tag, v);
-                } else if (tag.getValueCount() <= 1) {
+                if ((tag.getValueCount() > 1 && !v.isEmpty()) || tag.getValueCount() <= 1) {
                     updateTagValue(tag, v);
                 }
+                break;
+            default: // Do nothing
             }
         }
@@ -653 +661 @@
         }
 
-        public void apply() {
+        void apply() {
             rowSelectionModel.setValueIsAdjusting(true);
             colSelectionModel.setValueIsAdjusting(true);