Changeset 6228 in josm for trunk/src/org/openstreetmap/josm/gui/layer

Timestamp:

2013-09-11T00:14:06+02:00 (11 years ago)

Author:

Don-vip

Message:

Sonar - replace array copy loops by more efficient calls to System.arraycopy

Location:

trunk/src/org/openstreetmap/josm/gui/layer

Files:

• GpxLayer.java (modified) (1 diff)
• gpx/ChooseTrackVisibilityAction.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java

@@ -818 +818 @@
         final int m = Math.min(l, back.length);
         trackVisibility = new boolean[l];
-        for(int i=0; i < m; i++) {
-            trackVisibility[i] = back[i];
-        }
+        System.arraycopy(back, 0, trackVisibility, 0, m);
         for(int i=m; i < l; i++) {
             trackVisibility[i] = true;

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

@@ -47 +47 @@
     JTable table;
 
+    /**
+     * Constructs a new {@code ChooseTrackVisibilityAction}.
+     * @param layer The associated GPX layer
+     */
     public ChooseTrackVisibilityAction(final GpxLayer layer) {
         super(tr("Choose visible tracks"), ImageProvider.get("dialogs/filter"));
@@ -217 +221 @@
         // cancel for unknown buttons and copy back original settings
         if (v != 1 && v != 2) {
-            for (int i = 0; i < layer.trackVisibility.length; i++) {
-                layer.trackVisibility[i] = trackVisibilityBackup[i];
-            }
+            System.arraycopy(trackVisibilityBackup, 0, layer.trackVisibility, 0, layer.trackVisibility.length);
             Main.map.repaint();
             return;