Changeset 8130 in josm for trunk/src/org/openstreetmap/josm/gui/tagging/PresetLabel.java

Timestamp:

2015-03-09T08:46:22+01:00 (9 years ago)

Author:

Don-vip

Message:

fix Sonar issue squid:ModifiersOrderCheck - Modifiers should be declared in the correct order

File:

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

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

@@ -2 +2 @@
 package org.openstreetmap.josm.gui.tagging;
 
-import javax.swing.JLabel;
 import java.awt.Cursor;
 import java.awt.Font;
@@ -10 +9 @@
 import java.util.Collections;
 
+import javax.swing.JLabel;
+
 public class PresetLabel extends JLabel {
 
     protected final TaggingPreset t;
 
+    /**
+     * Constructs a new {@code PresetLabel}.
+     * @param t the tagging preset
+     */
     public PresetLabel(TaggingPreset t) {
         super(t.getName() + " …");
@@ -26 +31 @@
      */
     public static class PresetLabelMouseListener implements MouseListener {
-        final protected JLabel label;
-        final protected Font hover;
-        final protected Font normal;
+        protected final JLabel label;
+        protected final Font hover;
+        protected final Font normal;
 
+        /**
+         * Constructs a new {@code PresetLabelMouseListener}.
+         * @param lbl Label to highlight
+         */
         public PresetLabelMouseListener(JLabel lbl) {
             label = lbl;
@@ -36 +45 @@
             hover = normal.deriveFont(Collections.singletonMap(TextAttribute.UNDERLINE, TextAttribute.UNDERLINE_LOW_DOTTED));
         }
+
         @Override
-        public void mouseClicked(MouseEvent arg0) {
+        public void mouseClicked(MouseEvent e) {
+            // Do nothing
         }
 
         @Override
-        public void mouseEntered(MouseEvent arg0) {
+        public void mouseEntered(MouseEvent e) {
             label.setFont(hover);
         }
+
         @Override
-        public void mouseExited(MouseEvent arg0) {
+        public void mouseExited(MouseEvent e) {
             label.setFont(normal);
         }
+
         @Override
-        public void mousePressed(MouseEvent arg0) {}
+        public void mousePressed(MouseEvent e) {
+            // Do nothing
+        }
+
         @Override
-        public void mouseReleased(MouseEvent arg0) {}
+        public void mouseReleased(MouseEvent e) {
+            // Do nothing
+        }
     }
 }