Changeset 6986 in josm for trunk/src/org/openstreetmap/josm/actions

Timestamp:

2014-04-16T00:54:15+02:00 (10 years ago)

Author:

Don-vip

Message:

sonar - fix various minor issues

Location:

trunk/src/org/openstreetmap/josm/actions

Files:

• OrthogonalizeAction.java (modified) (2 diffs)
• UnJoinNodeWayAction.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/actions/OrthogonalizeAction.java

@@ -48 +48 @@
 
     /**
-     * Constructor
+     * Constructs a new {@code OrthogonalizeAction}.
      */
     public OrthogonalizeAction() {
@@ -519 +519 @@
      */
     private static class EN {
+        private EN() {
+            // Hide implicit public constructor for utility class
+        }
         // rotate counter-clock-wise
         public static EastNorth rotate_cc(EastNorth pivot, EastNorth en, double angle) {

trunk/src/org/openstreetmap/josm/actions/UnJoinNodeWayAction.java

@@ -10 +10 @@
 import java.util.ArrayList;
 import java.util.Collection;
-import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
@@ -18 +17 @@
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.command.RemoveNodesCommand;
-import org.openstreetmap.josm.command.Command;
 import org.openstreetmap.josm.data.osm.Node;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
@@ -24 +22 @@
 import org.openstreetmap.josm.tools.Shortcut;
 
+/**
+ * Disconnect nodes from a way they currently belong to.
+ * @since 6253
+ */
 public class UnJoinNodeWayAction extends JosmAction {
+    
+    /**
+     * Constructs a new {@code UnJoinNodeWayAction}.
+     */
     public UnJoinNodeWayAction() {
         super(tr("Disconnect Node from Way"), "unjoinnodeway",