Ignore:
Timestamp:
2013-10-03T03:41:25+02:00 (7 years ago)
Author:
Don-vip
Message:

Sonar/Findbugs - fix various problems

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

Legend:

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

    r6246 r6289  
    189189     */
    190190    protected Layer getFirstSelectedLayer() {
    191         if (LayerListDialog.getInstance() == null) return null;
    192191        List<Layer> layers = LayerListDialog.getInstance().getModel().getSelectedLayers();
    193192        if (layers.isEmpty()) return null;
  • trunk/src/org/openstreetmap/josm/actions/UnGlueAction.java

    r6130 r6289  
    368368            }
    369369            if (wayWithSelectedNode == null) {
    370                 wayWithSelectedNode = parentWays.removeFirst();
     370                parentWays.removeFirst();
    371371            }
    372372            for (Way w : parentWays) {
  • trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

    r6248 r6289  
    11701170    @Override
    11711171    public String getModeHelpText() {
    1172         String rv = "";
     1172        StringBuilder rv;
    11731173        /*
    11741174         *  No modifiers: all (Connect, Node Re-Use, Auto-Weld)
     
    11931193        // list is empty we can assume that we won't do any joins
    11941194        if (ctrl || oldHighlights.isEmpty()) {
    1195             rv = tr("Create new node.");
     1195            rv = new StringBuilder(tr("Create new node."));
    11961196        } else {
    11971197            // oldHighlights may store a node or way, check if it's a node
    11981198            OsmPrimitive x = oldHighlights.iterator().next();
    11991199            if (x instanceof Node) {
    1200                 rv = tr("Select node under cursor.");
     1200                rv = new StringBuilder(tr("Select node under cursor."));
    12011201            } else {
    1202                 rv = trn("Insert new node into way.", "Insert new node into {0} ways.",
    1203                         oldHighlights.size(), oldHighlights.size());
     1202                rv = new StringBuilder(trn("Insert new node into way.", "Insert new node into {0} ways.",
     1203                        oldHighlights.size(), oldHighlights.size()));
    12041204            }
    12051205        }
     
    12101210        if (currentBaseNode != null && !wayIsFinished) {
    12111211            if (alt) {
    1212                 rv += " " + tr("Start new way from last node.");
     1212                rv.append(" ").append(tr("Start new way from last node."));
    12131213            } else {
    1214                 rv += " " + tr("Continue way from last node.");
     1214                rv.append(" ").append(tr("Continue way from last node."));
    12151215            }
    12161216            if (snapHelper.isSnapOn()) {
    1217                 rv += " "+ tr("Angle snapping active.");
     1217                rv.append(" ").append(tr("Angle snapping active."));
    12181218            }
    12191219        }
     
    12251225        if (n != null && getCurrentDataSet() != null && getCurrentDataSet().getSelectedNodes().contains(n)) {
    12261226            if (wayIsFinished) {
    1227                 rv = tr("Select node under cursor.");
     1227                rv = new StringBuilder(tr("Select node under cursor."));
    12281228            } else {
    1229                 rv = tr("Finish drawing.");
     1229                rv = new StringBuilder(tr("Finish drawing."));
    12301230            }
    12311231        }
     
    12381238            for (Node m : w.getNodes()) {
    12391239                if (m.equals(mouseOnExistingNode) || mouseOnExistingWays.contains(w)) {
    1240                     rv += " " + tr("Finish drawing.");
     1240                    rv.append(" ").append(tr("Finish drawing."));
    12411241                    break;
    12421242                }
    12431243            }
    12441244        }
    1245         return rv;
     1245        return rv.toString();
    12461246    }
    12471247
Note: See TracChangeset for help on using the changeset viewer.