Changeset 29535 in osm for applications/editors/josm/plugins/reltoolbox/src/relcontext/actions

Timestamp:

2013-04-23T13:43:26+02:00 (12 years ago)

Author:

donvip

Message:

[josm_reltoolbox] Fix some warnings

Location:

applications/editors/josm/plugins/reltoolbox/src/relcontext/actions

Files:

• DownloadParentsAction.java (modified) (2 diffs)
• ReconstructPolygonAction.java (modified) (1 diff)
• TheRing.java (modified) (7 diffs)

applications/editors/josm/plugins/reltoolbox/src/relcontext/actions/DownloadParentsAction.java

@@ -2 +2 @@
 
 import static org.openstreetmap.josm.tools.I18n.tr;
+
 import java.awt.event.ActionEvent;
 import java.util.ArrayList;
@@ -8 +9 @@
 import java.util.List;
 import java.util.Set;
+
 import javax.swing.AbstractAction;
+
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.actions.downloadtasks.DownloadReferrersTask;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.data.osm.Relation;
-import org.openstreetmap.josm.data.osm.RelationMember;
 import org.openstreetmap.josm.gui.dialogs.relation.DownloadRelationMemberTask;
 import org.openstreetmap.josm.gui.dialogs.relation.DownloadRelationTask;
 import org.openstreetmap.josm.tools.ImageProvider;
+
 import relcontext.ChosenRelation;
 import relcontext.ChosenRelationListener;

applications/editors/josm/plugins/reltoolbox/src/relcontext/actions/ReconstructPolygonAction.java

@@ -107 +107 @@
                                 Way tmp = w;
                                 w = candidateWay;
-                                candidateWay = w;
+                                candidateWay = tmp;
                             }
                             commands.add(new DeleteCommand(w));

applications/editors/josm/plugins/reltoolbox/src/relcontext/actions/TheRing.java

@@ -1 +1 @@
 package relcontext.actions;
 
-import java.util.*;
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+
 import javax.swing.JOptionPane;
+
 import org.openstreetmap.josm.Main;
-import org.openstreetmap.josm.command.*;
-import org.openstreetmap.josm.data.coor.EastNorth;
-import org.openstreetmap.josm.data.osm.*;
+import org.openstreetmap.josm.command.AddCommand;
+import org.openstreetmap.josm.command.ChangeCommand;
+import org.openstreetmap.josm.command.Command;
+import org.openstreetmap.josm.command.DeleteCommand;
+import org.openstreetmap.josm.data.osm.Node;
+import org.openstreetmap.josm.data.osm.OsmPrimitive;
+import org.openstreetmap.josm.data.osm.Relation;
+import org.openstreetmap.josm.data.osm.RelationMember;
+import org.openstreetmap.josm.data.osm.Way;
 import org.openstreetmap.josm.tools.Geometry;
 import org.openstreetmap.josm.tools.Geometry.PolygonIntersection;
@@ -326 +338 @@
             }
         }
-        // todo: когда два кольца меняют одно и то же отношение, в список команд добавляется
-        // изменение базового отношения на новое, а не предыдущего
-        // поэтому сохраняется только первое изменение
+        // todo: когда два кольца мен�ют одно и то же отношение, в �пи�ок команд добавл�ет��
+        // изменение базового отношени� на новое, а не предыдущего
+        // по�тому �охран�ет�� только первое изменение
 
         List<Command> commands = new ArrayList<Command>();
@@ -389 +401 @@
      * Appends "append" to "base" so the closed polygon forms.
      */
-    private static void closePolygon( List<Node> base, List<Node> append ) {
+    /*private static void closePolygon( List<Node> base, List<Node> append ) {
         if( append.get(0).equals(base.get(0)) && append.get(append.size() - 1).equals(base.get(base.size() - 1)) ) {
             List<Node> ap2 = new ArrayList<Node>(append);
@@ -397 +409 @@
         base.remove(base.size() - 1);
         base.addAll(append);
-    }
+    }*/
 
     /**
      * Checks if a middle point between two nodes is inside a polygon. Useful to check if the way is inside.
      */
-    private static boolean segmentInsidePolygon( Node n1, Node n2, List<Node> polygon ) {
+    /*private static boolean segmentInsidePolygon( Node n1, Node n2, List<Node> polygon ) {
         EastNorth en1 = n1.getEastNorth();
         EastNorth en2 = n2.getEastNorth();
         Node testNode = new Node(new EastNorth((en1.east() + en2.east()) / 2.0, (en1.north() + en2.north()) / 2.0));
         return Geometry.nodeInsidePolygon(testNode, polygon);
-    }
+    }*/
     
     private static void log( String s ) {
@@ -420 +432 @@
         private boolean isRing;
 
-        private RingSegment() {
-        }
+        /*private RingSegment() {
+        }*/
 
         public RingSegment( Way w ) {
@@ -435 +447 @@
         }
 
-        public RingSegment( RingSegment ref ) {
+        /*public RingSegment( RingSegment ref ) {
             this.nodes = null;
             this.references = ref;
-        }
+        }*/
 
         /**
@@ -559 +571 @@
          * @return true if ways are equals, or one references another.
          */
-        public boolean isReferencingEqual( RingSegment other ) {
+        /*public boolean isReferencingEqual( RingSegment other ) {
             return this.equals(other) || (other.isReference() && other.references == this) || (isReference() && references == other);
-        }
+        }*/
 
         @Override