Changeset 11436 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2017-01-07T01:15:43+01:00 (7 years ago)

Author:

Don-vip

Message:

fix recent Sonar issues

File:

• trunk/src/org/openstreetmap/josm/gui/layer/gpx/GpxDrawHelper.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/gui/layer/gpx/GpxDrawHelper.java

@@ -144 +144 @@
 
     // copied value for line drawing
-    private List<Integer> heatMapPolyX = new ArrayList<>();
-    private List<Integer> heatMapPolyY = new ArrayList<>();
+    private final List<Integer> heatMapPolyX = new ArrayList<>();
+    private final List<Integer> heatMapPolyY = new ArrayList<>();
 
     // setup color maps used by heat map
@@ -369 +369 @@
                          "(" +
                          "segments= " + visibleSegments.size() +
-                         ", per 10000 = " + Utils.getDurationString(10000 * timeDiff / visibleSegments.size()) +
+                         ", per 10000 = " + Utils.getDurationString(10_000 * timeDiff / visibleSegments.size()) +
                          ")"
               );
@@ -449 +449 @@
             dateScale.setRange(minval, maxval);
         }
-
 
         // Now the colors for all the points will be assigned
@@ -719 +718 @@
     }
 
-   /**
+    /**
      * Draw GPX lines by using alpha blending
      * @param g               the common draw object to use
@@ -853 +852 @@
         // transform into lookup table
         return colorTable;
-    }
-
-    /**
-     * Creates a colormap by using a static color map with 1..n colors (RGB 0.0 ..1.0)
-     * @param data array of multiple RGB color [n][3]
-     * @return a dynamic list of color objects
-     */
-    protected static Color[] createColorFromRawArray(double[][] data) {
-
-        // create the array
-        Color[] color = new Color[ data.length ];
-
-        for (int k = 0; k < data.length; k++) {
-           // cast an map to linear array
-           color[k] = new Color((float) data[k][0], (float) data[k][1], (float) data[k][2]);
-        }
-
-        // forward
-        return createColorLut(color);
     }