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

Timestamp:

2016-04-07T00:02:22+02:00 (10 years ago)

Author:

Don-vip

Message:

fix coverity 1011666, 1011667, 1011836, 1011837, 1011838

File:

• trunk/src/org/openstreetmap/josm/actions/mapmode/ExtrudeAction.java (modified) (5 diffs)

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

@@ -763 +763 @@
         }
         return bestMovement;
-
-
     }
 
@@ -1026 +1024 @@
                 Point p4 = mv.getPoint(newN2en);
 
-                Point2D normalUnitVector = getNormalUniVector();
+                Point2D normalUnitVector = activeMoveDirection != null ? getNormalUniVector() : null;
 
                 if (mode == Mode.extrude || mode == Mode.create_new) {
@@ -1033 +1031 @@
                     // Draw rectangle around new area.
                     GeneralPath b = new GeneralPath();
-                    b.moveTo(p1.x, p1.y); b.lineTo(p3.x, p3.y);
-                    b.lineTo(p4.x, p4.y); b.lineTo(p2.x, p2.y);
+                    b.moveTo(p1.x, p1.y);
+                    b.lineTo(p3.x, p3.y);
+                    b.lineTo(p4.x, p4.y);
+                    b.lineTo(p2.x, p2.y);
                     b.lineTo(p1.x, p1.y);
                     g2.draw(b);
@@ -1042 +1042 @@
                         drawReferenceSegment(g2, mv, dualAlignSegment1);
                         drawReferenceSegment(g2, mv, dualAlignSegment2);
-                    } else if (activeMoveDirection != null) {
+                    } else if (activeMoveDirection != null && normalUnitVector != null) {
                         // Draw reference way
                         drawReferenceSegment(g2, mv, activeMoveDirection);
@@ -1052 +1052 @@
                             double headingMoveDir = Math.atan2(normalUnitVector.getY(), normalUnitVector.getX());
                             double headingDiff = headingRefWS - headingMoveDir;
-                            if (headingDiff < 0) headingDiff += 2 * Math.PI;
+                            if (headingDiff < 0)
+                                headingDiff += 2 * Math.PI;
                             boolean mirrorRA = Math.abs(headingDiff - Math.PI) > 1e-5;
                             Point pr1 = mv.getPoint(activeMoveDirection.p1);