Changeset 10305 in josm for trunk/src


Ignore:
Timestamp:
2016-05-31T00:01:20+02:00 (9 years ago)
Author:
Don-vip
Message:

sonar - various code cleanup fixes

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/AbstractDownloadTask.java

    r10242 r10305  
    8484    @Override
    8585    public String acceptsDocumentationSummary() {
    86         StringBuilder buff = new StringBuilder(128).append("<tr><td>");
    87         buff.append(getTitle())
     86        StringBuilder buff = new StringBuilder(128)
     87            .append("<tr><td>")
     88            .append(getTitle())
    8889            .append(":</td><td>");
    8990        String[] patterns = getPatterns();

  • trunk/src/org/openstreetmap/josm/data/projection/datum/NTV2GridShiftFile.java

    r10250 r10305  
    273273    @Override
    274274    public String toString() {
    275         StringBuilder buff = new StringBuilder(256).append("Headers  : ");
    276         buff.append(overviewHeaderCount)
     275        return new StringBuilder(256)
     276            .append("Headers  : ")
     277            .append(overviewHeaderCount)
    277278            .append("\nSub Hdrs : ")
    278279            .append(subGridHeaderCount)
    … …  
    294295            .append(toSemiMajorAxis)
    295296            .append("\nTo Min Ax: ")
    296             .append(toSemiMinorAxis);
    297         return buff.toString();
     297            .append(toSemiMinorAxis)
     298            .toString();
    298299    }
    299300

  • trunk/src/org/openstreetmap/josm/data/projection/datum/NTV2SubGrid.java

    r10242 r10305  
    295295     */
    296296    public String getDetails() {
    297         StringBuilder buff = new StringBuilder(256).append("Sub Grid : ");
    298         buff.append(subGridName)
     297        return new StringBuilder(256)
     298            .append("Sub Grid : ")
     299            .append(subGridName)
    299300            .append("\nParent   : ")
    300301            .append(parentSubGridName)
    … …  
    316317            .append(lonInterval)
    317318            .append("\nNode Cnt : ")
    318             .append(nodeCount);
    319         return buff.toString();
     319            .append(nodeCount)
     320            .toString();
    320321    }
    321322

  • trunk/src/org/openstreetmap/josm/gui/dialogs/InspectPrimitiveDialog.java

    r10213 r10305  
    183183    */
    184184    protected static String buildListOfEditorsText(Iterable<OsmPrimitive> primitives) {
    185         final StringBuilder s = new StringBuilder();
    186185        final Map<String, Integer> editCountByUser = new TreeMap<>(Collator.getInstance(Locale.getDefault()));
    187186
    … …  
    200199
    201200        // Print the count in sorted order
    202         s.append(trn("{0} user last edited the selection:", "{0} users last edited the selection:",
    203                 editCountByUser.size(), editCountByUser.size()));
    204         s.append("\n\n");
     201        final StringBuilder s = new StringBuilder(48)
     202            .append(trn("{0} user last edited the selection:", "{0} users last edited the selection:",
     203                editCountByUser.size(), editCountByUser.size()))
     204            .append("\n\n");
    205205        for (Map.Entry<String, Integer> entry : editCountByUser.entrySet()) {
    206206            final String username = entry.getKey();

  • trunk/src/org/openstreetmap/josm/gui/io/ActionFlagsTableCell.java

    r9751 r10305  
    8585
    8686    private void updatePanel(SaveLayerInfo info) {
    87         StringBuilder sb = new StringBuilder();
    88         sb.append("<html>");
    89         sb.append(tr("Select which actions to perform for this layer, if you click the leftmost button."));
     87        StringBuilder sb = new StringBuilder(128)
     88            .append("<html>")
     89            .append(tr("Select which actions to perform for this layer, if you click the leftmost button."));
    9090        removeAll();
    9191        if (info != null) {
    9292            if (info.isUploadable()) {
    93                 sb.append("<br/>");
    94                 sb.append(tr("Check \"Upload\" to upload the changes to the OSM server."));
     93                sb.append("<br/>")
     94                  .append(tr("Check \"Upload\" to upload the changes to the OSM server."));
    9595                add(checkBoxes[0], GBC.eol().fill(GBC.HORIZONTAL));
    9696            }
    9797            if (info.isSavable()) {
    98                 sb.append("<br/>");
    99                 sb.append(tr("Check \"Save\" to save the layer to the file specified on the left."));
     98                sb.append("<br/>")
     99                  .append(tr("Check \"Save\" to save the layer to the file specified on the left."));
    100100                add(checkBoxes[1], GBC.eol().fill(GBC.HORIZONTAL));
    101101            }

  • trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/NodeElement.java

    r10242 r10305  
    374374        NodeElement that = (NodeElement) obj;
    375375        return Objects.equals(mapImage, that.mapImage) &&
    376                 Objects.equals(mapImageAngle, that.mapImageAngle) &&
    377                 Objects.equals(symbol, that.symbol);
     376               Objects.equals(mapImageAngle, that.mapImageAngle) &&
     377               Objects.equals(symbol, that.symbol);
    378378    }
    379379
    380380    @Override
    381381    public String toString() {
    382         StringBuilder s = new StringBuilder(64).append("NodeElemStyle{");
    383         s.append(super.toString());
     382        StringBuilder s = new StringBuilder(64).append("NodeElemStyle{").append(super.toString());
    384383        if (mapImage != null) {
    385384            s.append(" icon=[" + mapImage + ']');

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

    r10043 r10305  
    6868
    6969    private static void showNmeaInfobox(boolean success, NmeaReader r) {
    70         final StringBuilder msg = new StringBuilder(160).append("<html>");
    71         msg.append(tr("Coordinates imported: {0}", r.getNumberOfCoordinates())).append("<br>")
     70        final StringBuilder msg = new StringBuilder(160).append("<html>")
     71           .append(tr("Coordinates imported: {0}", r.getNumberOfCoordinates())).append("<br>")
    7272           .append(tr("Malformed sentences: {0}", r.getParserMalformed())).append("<br>")
    7373           .append(tr("Checksum errors: {0}", r.getParserChecksumErrors())).append("<br>");

  • trunk/src/org/openstreetmap/josm/tools/bugreport/ReportedException.java

    r10300 r10305  
    210210                str.append(e);
    211211            } else {
    212                 str.append("\n    ... (");
    213                 str.append(value.size());
    214                 str.append(" entries)");
     212                str.append("\n    ... (")
     213                   .append(value.size())
     214                   .append(" entries)");
    215215                break;
    216216            }

  • trunk/src/org/openstreetmap/josm/tools/template_engine/SearchExpressionCondition.java

    r8846 r10305  
    2626    @Override
    2727    public String toString() {
    28         return condition.toString() + " '" + text + '\'';
     28        return condition + " '" + text + '\'';
    2929    }
    3030}
Note: See TracChangeset for help on using the changeset viewer.