Changeset 10315 in josm


Ignore:
Timestamp:
2016-06-02T02:22:40+02:00 (3 years ago)
Author:
Don-vip
Message:

findbugs, code optimization/cleanup

Location:
trunk/src/org/openstreetmap/josm
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java

    r10212 r10315  
    218218        protected int getNumDataLayers() {
    219219            if (!Main.isDisplayingMapView()) return 0;
    220             int count = 0;
    221             Collection<Layer> layers = Main.map.mapView.getAllLayers();
    222             for (Layer layer : layers) {
    223                 if (layer instanceof OsmDataLayer) {
    224                     count++;
    225                 }
    226             }
    227             return count;
     220            return Utils.filteredCollection(Main.map.mapView.getAllLayers(), OsmDataLayer.class).size();
    228221        }
    229222
  • trunk/src/org/openstreetmap/josm/io/OsmServerLocationReader.java

    r10212 r10315  
    66import java.io.IOException;
    77import java.io.InputStream;
    8 import java.util.ArrayList;
     8import java.util.Collections;
    99import java.util.List;
    1010
     
    178178            in = getInputStream(url, progressMonitor.createSubTaskMonitor(1, true));
    179179            if (in == null) {
    180                 return new ArrayList<>();
     180                return Collections.emptyList();
    181181            }
    182182            progressMonitor.subTask(tr("Downloading OSM notes..."));
  • trunk/src/org/openstreetmap/josm/tools/Utils.java

    r10294 r10315  
    395395            return Arrays.copyOf(array, array.length);
    396396        }
    397         return null;
     397        return array;
    398398    }
    399399
     
    408408            return Arrays.copyOf(array, array.length);
    409409        }
    410         return null;
     410        return array;
    411411    }
    412412
     
    421421            return Arrays.copyOf(array, array.length);
    422422        }
    423         return null;
     423        return array;
    424424    }
    425425
Note: See TracChangeset for help on using the changeset viewer.