Changeset 8285 in josm for trunk/src/org/openstreetmap/josm/actions/mapmode/ParallelWayAction.java

Timestamp:

2015-04-28T00:49:49+02:00 (9 years ago)

Author:

Don-vip

Message:

fix sonar squid:S2039 - Member variable visibility should be specified

File:

• trunk/src/org/openstreetmap/josm/actions/mapmode/ParallelWayAction.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/actions/mapmode/ParallelWayAction.java

@@ -21 +21 @@
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.Bounds;
-import org.openstreetmap.josm.data.SystemOfMeasurement;
 import org.openstreetmap.josm.data.Preferences.PreferenceChangeEvent;
 import org.openstreetmap.josm.data.Preferences.PreferenceChangedListener;
+import org.openstreetmap.josm.data.SystemOfMeasurement;
 import org.openstreetmap.josm.data.coor.EastNorth;
 import org.openstreetmap.josm.data.osm.Node;
@@ -124 +124 @@
     private EastNorth helperLineEnd;
 
-    Stroke helpLineStroke;
-    Stroke refLineStroke;
-    Color mainColor;
-
+    private Stroke helpLineStroke;
+    private Stroke refLineStroke;
+    private Color mainColor;
+
+    /**
+     * Constructs a new {@code ParallelWayAction}.
+     * @param mapFrame Map frame
+     */
     public ParallelWayAction(MapFrame mapFrame) {
         super(tr("Parallel"), "parallel", tr("Make parallel copies of ways"),
@@ -303 +307 @@
 
         // Since the created way is left selected, we need to unselect again here
-        if (pWays != null && pWays.ways != null) {
-            getCurrentDataSet().clearSelection(pWays.ways);
+        if (pWays != null && pWays.getWays() != null) {
+            getCurrentDataSet().clearSelection(pWays.getWays());
             pWays = null;
         }
@@ -545 +549 @@
             pWays = new ParallelWays(sourceWays, copyTags, referenceWayIndex);
             pWays.commit();
-            getCurrentDataSet().setSelected(pWays.ways);
+            getCurrentDataSet().setSelected(pWays.getWays());
             return true;
         } catch (IllegalArgumentException e) {