Index: trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java
===================================================================
--- trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java	(revision 8129)
+++ trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java	(revision 8130)
@@ -179,5 +179,5 @@
      * @since 7636
      */
-    public static abstract class AbstractInternalTask extends PleaseWaitRunnable {
+    public abstract static class AbstractInternalTask extends PleaseWaitRunnable {
 
         protected final boolean newLayer;
Index: trunk/src/org/openstreetmap/josm/data/imagery/ImageryLayerInfo.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/imagery/ImageryLayerInfo.java	(revision 8129)
+++ trunk/src/org/openstreetmap/josm/data/imagery/ImageryLayerInfo.java	(revision 8130)
@@ -31,6 +31,6 @@
     private final List<ImageryInfo> layers = new ArrayList<>();
     private final Map<String, ImageryInfo> layerIds = new HashMap<>();
-    private final static List<ImageryInfo> defaultLayers = new ArrayList<>();
-    private final static Map<String, ImageryInfo> defaultLayerIds = new HashMap<>();
+    private static final List<ImageryInfo> defaultLayers = new ArrayList<>();
+    private static final Map<String, ImageryInfo> defaultLayerIds = new HashMap<>();
 
     private static final String[] DEFAULT_LAYER_SITES = {
Index: trunk/src/org/openstreetmap/josm/data/osm/Changeset.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/osm/Changeset.java	(revision 8129)
+++ trunk/src/org/openstreetmap/josm/data/osm/Changeset.java	(revision 8130)
@@ -18,5 +18,5 @@
  * Represents a single changeset in JOSM. For now its only used during
  * upload but in the future we may do more.
- *
+ * @since 625
  */
 public final class Changeset implements Tagged {
@@ -349,5 +349,5 @@
      * @since 7704
      */
-    public synchronized final List<ChangesetDiscussionComment> getDiscussion() {
+    public final synchronized List<ChangesetDiscussionComment> getDiscussion() {
         if (discussion == null) {
             return Collections.emptyList();
@@ -361,5 +361,5 @@
      * @since 7704
      */
-    public synchronized final void addDiscussionComment(ChangesetDiscussionComment comment) {
+    public final synchronized void addDiscussionComment(ChangesetDiscussionComment comment) {
         if (comment == null) {
             return;
Index: trunk/src/org/openstreetmap/josm/data/validation/tests/MapCSSTagChecker.java
===================================================================
--- trunk/src/org/openstreetmap/josm/data/validation/tests/MapCSSTagChecker.java	(revision 8129)
+++ trunk/src/org/openstreetmap/josm/data/validation/tests/MapCSSTagChecker.java	(revision 8130)
@@ -77,7 +77,7 @@
     public static class GroupedMapCSSRule {
         /** MapCSS selectors **/
-        final public List<Selector> selectors;
+        public final List<Selector> selectors;
         /** MapCSS declaration **/
-        final public Declaration declaration;
+        public final Declaration declaration;
 
         /**
Index: trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java
===================================================================
--- trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java	(revision 8129)
+++ trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java	(revision 8130)
@@ -83,5 +83,5 @@
      * stack trace.
      */
-    public final static ReadWriteLock STYLE_SOURCE_LOCK = new ReentrantReadWriteLock();
+    public static final ReadWriteLock STYLE_SOURCE_LOCK = new ReentrantReadWriteLock();
 
     /**
Index: trunk/src/org/openstreetmap/josm/gui/tagging/PresetLabel.java
===================================================================
--- trunk/src/org/openstreetmap/josm/gui/tagging/PresetLabel.java	(revision 8129)
+++ trunk/src/org/openstreetmap/josm/gui/tagging/PresetLabel.java	(revision 8130)
@@ -2,5 +2,4 @@
 package org.openstreetmap.josm.gui.tagging;
 
-import javax.swing.JLabel;
 import java.awt.Cursor;
 import java.awt.Font;
@@ -10,8 +9,14 @@
 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,8 +31,12 @@
      */
     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,20 +45,29 @@
             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
+        }
     }
 }
Index: trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionItemPriority.java
===================================================================
--- trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionItemPriority.java	(revision 8129)
+++ trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionItemPriority.java	(revision 8130)
@@ -37,5 +37,5 @@
     public static final AutoCompletionItemPriority UNKNOWN = new AutoCompletionItemPriority(false, false, false);
 
-    private final static int NO_USER_INPUT = Integer.MAX_VALUE;
+    private static final int NO_USER_INPUT = Integer.MAX_VALUE;
 
     private final int userInput;
@@ -43,9 +43,9 @@
     private final boolean inStandard;
     private final boolean selected;
-    
+
 
     /**
      * Create new AutoCompletionItemPriority object.
-     * 
+     *
      * @param inDataSet true, if the item is found in the currently active data layer
      * @param inStandard true, if the item is a standard tag, e.g. from the presets.
@@ -81,5 +81,5 @@
         return userInput == NO_USER_INPUT ? null : userInput;
     }
-    
+
     /**
      * Imposes an ordering on the priorities.
@@ -115,6 +115,7 @@
     }
 
-    @Override public String toString() {
-        return String.format("<Priority; userInput: %s, inDataSet: %b, inStandard: %b, selected: %b>", 
+    @Override
+    public String toString() {
+        return String.format("<Priority; userInput: %s, inDataSet: %b, inStandard: %b, selected: %b>",
                 userInput == NO_USER_INPUT ? "no" : Integer.toString(userInput), inDataSet, inStandard, selected);
     }