Ignore:
Timestamp:
2015-10-10T14:08:30+02:00 (11 years ago)
Author:
Don-vip
Message:

sonar - squid:S2131 - Primitives should not be boxed just for "String" conversion

Location:
trunk/src/org/openstreetmap/josm/tools
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/tools/PlatformHookOsx.java

    r8846 r8848  
    289289            canHtml = false;
    290290        }
    291         String result = "";
     291        StringBuilder result = new StringBuilder();
    292292        if (canHtml) {
    293             result += "<html>";
    294         }
    295         result += name;
     293            result.append("<html>");
     294        }
     295        result.append(name);
    296296        if (sc != null && !sc.getKeyText().isEmpty()) {
    297             result += ' ';
     297            result.append(' ');
    298298            if (canHtml) {
    299                 result += "<font size='-2'>";
    300             }
    301             result += '('+sc.getKeyText()+')';
     299                result.append("<font size='-2'>");
     300            }
     301            result.append('(').append(sc.getKeyText()).append(')');
    302302            if (canHtml) {
    303                 result += "</font>";
     303                result.append("</font>");
    304304            }
    305305        }
    306306        if (canHtml) {
    307             result += "&nbsp;</html>";
    308         }
    309         return result;
     307            result.append("&nbsp;</html>");
     308        }
     309        return result.toString();
    310310    }
    311311

  • trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java

    r8846 r8848  
    149149    @Override
    150150    public String makeTooltip(String name, Shortcut sc) {
    151         String result = "";
    152         result += "<html>";
    153         result += name;
     151        StringBuilder result = new StringBuilder();
     152        result.append("<html>").append(name);
    154153        if (sc != null && !sc.getKeyText().isEmpty()) {
    155             result += ' ';
    156             result += "<font size='-2'>";
    157             result += '('+sc.getKeyText()+')';
    158             result += "</font>";
    159         }
    160         return result + "&nbsp;</html>";
     154            result.append(' ')
     155                  .append("<font size='-2'>")
     156                  .append('(').append(sc.getKeyText()).append(')')
     157                  .append("</font>");
     158        }
     159        return result.append("&nbsp;</html>").toString();
    161160    }
    162161
Note: See TracChangeset for help on using the changeset viewer.