Changeset 6265 in josm for trunk/src


Ignore:
Timestamp:
2013-09-27T01:53:58+02:00 (11 years ago)
Author:
Don-vip
Message:

Sonar/FindBugs:

  • Dodgy - Redundant nullcheck of value known to be non-null
  • Dodgy - Write to static field from instance method
Location:
trunk/src/org/openstreetmap/josm
Files:
12 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/Main.java

    r6248 r6265  
    354354        if (map != null) {
    355355            map.mapView.removeLayer(layer);
    356             if (map != null && map.mapView.getAllLayers().isEmpty()) {
     356            if (map.mapView.getAllLayers().isEmpty()) {
    357357                setMapFrame(null);
    358358            }

  • trunk/src/org/openstreetmap/josm/actions/JoinAreasAction.java

    r6246 r6265  
    14491449     */
    14501450    private void stripTags(Way x) {
    1451         if (x.getKeys() == null)
    1452             return;
    14531451        Way y = new Way(x);
    14541452        for (String key : x.keySet()) {

  • trunk/src/org/openstreetmap/josm/actions/mapmode/ParallelWayAction.java

    r6246 r6265  
    523523    private void clearSourceWays() {
    524524        assert (sourceWays != null);
    525         if (sourceWays == null)
    526             return;
    527525        getCurrentDataSet().clearSelection(sourceWays);
    528526        for (Way w : sourceWays) {

  • trunk/src/org/openstreetmap/josm/actions/mapmode/PlayHeadDragMode.java

    r5960 r6265  
    5555        if ((ev.getModifiersEx() & MouseEvent.BUTTON1_DOWN_MASK) == 0) return;
    5656        Point p = ev.getPoint();
    57         if (p == null) return;
    5857        if (! dragging) {
    5958            if (p.distance(mouseStart) < 3) return;

  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r6102 r6265  
    755755
    756756        @Override public String toString() {
    757             return "user=" + user == null ? "" : user;
     757            return "user=" + (user == null ? "" : user);
    758758        }
    759759    }

  • trunk/src/org/openstreetmap/josm/data/validation/tests/TagChecker.java

    r6264 r6265  
    367367        }
    368368
    369         Map<String, String> props = (p.getKeys() == null) ? Collections.<String, String>emptyMap() : p.getKeys();
    370         for (Entry<String, String> prop : props.entrySet()) {
     369        for (Entry<String, String> prop : p.getKeys().entrySet()) {
    371370            String s = marktr("Key ''{0}'' invalid.");
    372371            String key = prop.getKey();

  • trunk/src/org/openstreetmap/josm/gui/MapStatus.java

    r6248 r6265  
    282282                                    // display them if the middle mouse button is pressed and
    283283                                    // keep them until the mouse is moved
    284                                     if (middleMouseDown || isAtOldPosition)
    285                                     {
     284                                    if (middleMouseDown || isAtOldPosition) {
    286285                                        Collection<OsmPrimitive> osms = mv.getAllNearest(ms.mousePos, OsmPrimitive.isUsablePredicate);
    287 
    288                                         if (osms == null)
    289                                             return;
    290286
    291287                                        final JPanel c = new JPanel(new GridBagLayout());

  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/tags/TagMergeItem.java

    r3083 r6265  
    101101            if (theirTagValue == null) {
    102102                primitive.remove(key);
    103             } else if (theirTagValue != null) {
     103            } else {
    104104                primitive.put(key, theirTagValue);
    105105            }
    … …  
    107107            if (myTagValue == null) {
    108108                primitive.remove(key);
    109             } else if (myTagValue != null) {
     109            } else {
    110110                primitive.put(key, myTagValue);
    111111            }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/sort/WayConnectionTypeCalculator.java

    r5630 r6265  
    264264        }
    265265
    266         if (refNodes == null)
    267             return NONE;
    268 
    269266        for (Node n : refNodes) {
    270267            if (n == null) {

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/GeoImageLayer.java

    r6248 r6265  
    191191                    }
    192192
    193                     Collection<File> children = Arrays.asList(f.listFiles(JpegFileFilter.getInstance()));
     193                    File[] children = f.listFiles(JpegFileFilter.getInstance());
    194194                    if (children != null) {
    195195                        progressMonitor.subTask(tr("Scanning directory {0}", f.getPath()));
    196196                        try {
    197                             addRecursiveFiles(files, children);
     197                            addRecursiveFiles(files, Arrays.asList(children));
    198198                        } catch(NullPointerException npe) {
    199199                            npe.printStackTrace();

  • trunk/src/org/openstreetmap/josm/io/DefaultProxySelector.java

    r6248 r6265  
    3232     */
    3333    private static boolean JVM_WILL_USE_SYSTEM_PROXIES = false;
    34     {
     34    static {
    3535        String v = System.getProperty("java.net.useSystemProxies");
    3636        if (v != null && v.equals(Boolean.TRUE.toString())) {

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RequestProcessor.java

    r6248 r6265  
    392392            return null;
    393393        }
    394         if (handler==null) return null;
    395394
    396395        r.printf("{ \"request\" : \"%s\"", cmd);
Note: See TracChangeset for help on using the changeset viewer.