Changeset 10173 in josm for trunk/src/org/openstreetmap/josm/gui/widgets

Timestamp:

2016-05-10T14:54:38+02:00 (8 years ago)

Author:

Don-vip

Message:

sonar - squid:S1186 - Methods should not be empty

Location:

trunk/src/org/openstreetmap/josm/gui/widgets

Files:

• MultiSplitPane.java (modified) (1 diff)
• PopupMenuLauncher.java (modified) (2 diffs)
• QuadStateCheckBox.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/widgets/MultiSplitPane.java

@@ -384 +384 @@
 
         @Override
-        public void keyReleased(KeyEvent e) { }
-
-        @Override
-        public void keyTyped(KeyEvent e) { }
+        public void keyReleased(KeyEvent e) {
+            // Do nothing
+        }
+
+        @Override
+        public void keyTyped(KeyEvent e) {
+            // Do nothing
+        }
     }
 

trunk/src/org/openstreetmap/josm/gui/widgets/PopupMenuLauncher.java

@@ -4 +4 @@
 import java.awt.Component;
 import java.awt.Point;
+import java.awt.event.FocusAdapter;
 import java.awt.event.FocusEvent;
-import java.awt.event.FocusListener;
 import java.awt.event.MouseAdapter;
 import java.awt.event.MouseEvent;
@@ -95 +95 @@
     protected void checkFocusAndShowMenu(final Component component, final MouseEvent evt) {
         if (component != null && component.isFocusable() && !component.hasFocus() && component.requestFocusInWindow()) {
-            component.addFocusListener(new FocusListener() {
-                @Override
-                public void focusLost(FocusEvent e) {}
-
+            component.addFocusListener(new FocusAdapter() {
                 @Override
                 public void focusGained(FocusEvent e) {

trunk/src/org/openstreetmap/josm/gui/widgets/QuadStateCheckBox.java

@@ -92 +92 @@
     /** Do not let anyone add mouse listeners */
     @Override
-    public void addMouseListener(MouseListener l) { }
+    public void addMouseListener(MouseListener l) {
+        // Do nothing
+    }
 
     /**
@@ -211 +213 @@
 
         @Override
-        public void setArmed(boolean b) { }
-
-        @Override
-        public void setSelected(boolean b) { }
-
-        @Override
-        public void setPressed(boolean b) { }
+        public void setArmed(boolean b) {
+            // Do nothing
+        }
+
+        @Override
+        public void setSelected(boolean b) {
+            // Do nothing
+        }
+
+        @Override
+        public void setPressed(boolean b) {
+            // Do nothing
+        }
 
         /** We disable focusing on the component when it is not enabled. */