Ignore:
Timestamp:
2016-03-12T23:18:04+01:00 (4 years ago)
Author:
Don-vip
Message:

sonar - remove useless parentheses

Location:
trunk/src/org/openstreetmap/josm/actions
Files:
4 edited

Legend:

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

    r9473 r9970  
    175175        }
    176176        if (targetNode == null) {
    177             targetNode = (oldestNode != null ? oldestNode : lastNode);
     177            targetNode = oldestNode != null ? oldestNode : lastNode;
    178178        }
    179179        return targetNode;
  • trunk/src/org/openstreetmap/josm/actions/UnGlueAction.java

    r9470 r9970  
    697697            for (OsmPrimitive ref : way.getReferrers()) {
    698698                if (ref instanceof Relation && ref.isUsable()) {
    699                     affectedRelations.add((ref.getDisplayName(DefaultNameFormatter.getInstance())));
     699                    affectedRelations.add(ref.getDisplayName(DefaultNameFormatter.getInstance()));
    700700                }
    701701            }
  • trunk/src/org/openstreetmap/josm/actions/mapmode/ModifiersSpec.java

    r8870 r9970  
    2525        char c = str.charAt(2);
    2626        // @formatter:off
    27         alt   = (a == '?' ? UNKNOWN : (a == 'A' ? ON : OFF));
    28         shift = (s == '?' ? UNKNOWN : (s == 'S' ? ON : OFF));
    29         ctrl  = (c == '?' ? UNKNOWN : (c == 'C' ? ON : OFF));
     27        alt   = a == '?' ? UNKNOWN : (a == 'A' ? ON : OFF);
     28        shift = s == '?' ? UNKNOWN : (s == 'S' ? ON : OFF);
     29        ctrl  = c == '?' ? UNKNOWN : (c == 'C' ? ON : OFF);
    3030        // @formatter:on
    3131    }
  • trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java

    r9246 r9970  
    410410        // We don't want to change to draw tool if the user tries to (de)select
    411411        // stuff but accidentally clicks in an empty area when selection is empty
    412         cancelDrawMode = (shift || ctrl);
     412        cancelDrawMode = shift || ctrl;
    413413        didMouseDrag = false;
    414414        initialMoveThresholdExceeded = false;
     
    11521152                } else {
    11531153                    // setup for iterating a sel group again or a new, different one..
    1154                     nxt = (cycleList.contains(cycleStart)) ? cycleStart : first;
     1154                    nxt = cycleList.contains(cycleStart) ? cycleStart : first;
    11551155                    cycleStart = nxt;
    11561156                }
Note: See TracChangeset for help on using the changeset viewer.