Changeset 10228 in josm for trunk/src/org/openstreetmap/josm/actions/mapmode

Timestamp:

2016-05-16T14:59:34+02:00 (8 years ago)

Author:

Don-vip

Message:

findbugs: SC_START_IN_CTOR + UW_UNCOND_WAIT + UM_UNNECESSARY_MATH + UPM_UNCALLED_PRIVATE_METHOD + DM_STRING_TOSTRING + DM_BOXED_PRIMITIVE_FOR_COMPARE + SIC_INNER_SHOULD_BE_STATIC_NEEDS_THIS

File:

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

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

@@ -896 +896 @@
 
     /**
-     * if one of the ends of {@code way} is given {@code  node},
-     * then set  currentBaseNode = node and previousNode = adjacent node of way
+     * if one of the ends of {@code way} is given {@code node},
+     * then set currentBaseNode = node and previousNode = adjacent node of way
      * @param way way to continue
      * @param node starting node
@@ -1692 +1692 @@
         }
 
-        private void nextSnapMode() {
-            if (snapOn) {
-                // turn off snapping if we are in fixed mode or no actile snapping line exist
-                if (fixed || !active) {
-                    snapOn = false;
-                    unsetFixedMode();
-                } else {
-                    setFixedMode();
-                }
-            } else {
-                snapOn = true;
-                unsetFixedMode();
-            }
-            checkBox.setState(snapOn);
-            customBaseHeading = -1;
-        }
-
         private void enableSnapping() {
             snapOn = true;
@@ -1729 +1712 @@
         }
 
-        public  void unsetFixedMode() {
+        public void unsetFixedMode() {
             fixed = false;
             absoluteFix = false;
@@ -1736 +1719 @@
         }
 
-        public  boolean isActive() {
+        public boolean isActive() {
             return active;
         }
 
-        public  boolean isSnapOn() {
+        public boolean isSnapOn() {
             return snapOn;
         }