Ignore:
Timestamp:
2015-05-17T05:48:46+02:00 (10 years ago)
Author:
Don-vip
Message:

Consecutively calls to StringBuffer/StringBuilder .append should reuse the target object

Location:
trunk/src/org/openstreetmap/josm/gui
Files:
17 edited

Legend:

Unmodified
Added
Removed

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

    r8342 r8379  
    512512    private String buildDefaultToolTip(long id, Map<String, String> tags) {
    513513        StringBuilder sb = new StringBuilder();
    514         sb.append("<html>");
    515         sb.append("<strong>id</strong>=")
     514        sb.append("<html>")
     515        .append("<strong>id</strong>=")
    516516        .append(id)
    517517        .append("<br>");

  • trunk/src/org/openstreetmap/josm/gui/bbox/TileSelectionBBoxChooser.java

    r8308 r8379  
    631631    /**
    632632     * Represents a rectangular area of tiles at a given zoom level.
    633      *
    634633     */
    635634    private static class TileBounds {
    636         public Point min;
    637         public Point max;
    638         public int zoomLevel;
    639 
    640         public TileBounds() {
     635        private Point min;
     636        private Point max;
     637        private int zoomLevel;
     638
     639        private TileBounds() {
    641640            zoomLevel = 0;
    642641            min = new Point(0,0);
    … …  
    644643        }
    645644
    646         public TileBounds(Point min, Point max, int zoomLevel) {
     645        private TileBounds(Point min, Point max, int zoomLevel) {
    647646            this.min = min;
    648647            this.max = max;
    … …  
    653652        public String toString() {
    654653            StringBuilder sb = new StringBuilder();
    655             sb.append("min=").append(min.x).append(",").append(min.y).append(",");
    656             sb.append("max=").append(max.x).append(",").append(max.y).append(",");
    657             sb.append("zoom=").append(zoomLevel);
     654            sb.append("min=").append(min.x).append(",").append(min.y).append(",")
     655              .append("max=").append(max.x).append(",").append(max.y).append(",")
     656              .append("zoom=").append(zoomLevel);
    658657            return sb.toString();
    659658        }
    … …  
    667666        private Point max;
    668667
    669         public TileBoundsMapView() {
     668        private TileBoundsMapView() {
    670669            setBorder(BorderFactory.createLineBorder(Color.DARK_GRAY));
    671670            TileLoader loader = tileController.getTileLoader();

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

    r8375 r8379  
    357357        }
    358358        StringBuilder sb = new StringBuilder();
    359         sb.append("<html>");
    360         sb.append(tr("The following objects could not be copied to the target object<br>because they are deleted in the target dataset:"));
    361         sb.append(Utils.joinAsHtmlUnorderedList(items));
    362         sb.append("</html>");
     359        sb.append("<html>")
     360          .append(tr("The following objects could not be copied to the target object<br>because they are deleted in the target dataset:"))
     361          .append(Utils.joinAsHtmlUnorderedList(items))
     362          .append("</html>");
    363363        HelpAwareOptionPane.showOptionDialog(
    364364                Main.parent,

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

    r8377 r8379  
    606606                text.append(tableRow(tr("Background:"), Utils.toString(s.getBackgroundColorOverride())));
    607607            }
    608             text.append(tableRow(tr("Style is currently active?"), s.active ? tr("Yes") : tr("No")));
    609             text.append("</table>");
     608            text.append(tableRow(tr("Style is currently active?"), s.active ? tr("Yes") : tr("No")))
     609                .append("</table>");
    610610            p.add(new JScrollPane(new HtmlPanel(text.toString())), GBC.eol().fill(GBC.BOTH));
    611611            return p;

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

    r7299 r8379  
    3535    protected String buildToolTipText(Changeset cs) {
    3636        StringBuilder sb = new StringBuilder();
    37         sb.append("<html>");
    38         sb.append("<strong>").append(tr("Changeset id:")).append("</strong>").append(cs.getId()).append("<br>");
     37        sb.append("<html>")
     38          .append("<strong>").append(tr("Changeset id:")).append("</strong>").append(cs.getId()).append("<br>");
    3939        if (cs.getCreatedAt() != null) {
    4040            sb.append("<strong>").append(tr("Created at:")).append("</strong>").append(

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

    r8308 r8379  
    7979        SaveLayerInfo info = (SaveLayerInfo)value;
    8080        StringBuilder sb = new StringBuilder();
    81         sb.append("<html>");
    82         sb.append(addLblLayerName(info));
    83         sb.append("<br>");
     81        sb.append("<html>")
     82          .append(addLblLayerName(info))
     83          .append("<br>");
    8484        add(btnFileChooser, GBC.std());
    85         sb.append(addLblFilename(info));
    86 
    87         sb.append("</html>");
     85        sb.append(addLblFilename(info))
     86          .append("</html>");
    8887        setToolTipText(sb.toString());
    8988        return this;
    … …  
    9897
    9998        StringBuilder sb = new StringBuilder();
    100         sb.append("<html>");
    101         sb.append(addLblLayerName(info));
    102         sb.append("<br/>");
     99        sb.append("<html>")
     100          .append(addLblLayerName(info))
     101          .append("<br/>");
    103102
    104103        add(btnFileChooser, GBC.std());
    … …  
    106105        tfFilename.selectAll();
    107106
    108         sb.append(tfFilename.getToolTipText());
    109         sb.append("</html>");
     107        sb.append(tfFilename.getToolTipText())
     108          .append("</html>");
    110109        setToolTipText(sb.toString());
    111110        return this;

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

    r8308 r8379  
    142142    protected void updateSummary() {
    143143        StringBuilder sb = new StringBuilder();
    144         sb.append("<html>");
    145         sb.append(buildStrategySummary());
    146         sb.append("<br><br>");
    147         sb.append(buildChangesetSummary());
    148         sb.append("</html>");
     144        sb.append("<html>")
     145          .append(buildStrategySummary())
     146          .append("<br><br>")
     147          .append(buildChangesetSummary())
     148          .append("</html>");
    149149        jepMessage.setText(sb.toString());
    150150    }

  • trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java

    r8378 r8379  
    157157                info.append("</td></tr>");
    158158            }
    159 
    160159            info.append("</table><br><br>");
    161 
    162         }
    163 
    164         info.append(tr("Length: {0}", NavigatableComponent.getSystemOfMeasurement().getDistText(data.length()))).append("<br>");
    165 
    166         info.append(trn("{0} route, ", "{0} routes, ", data.routes.size(), data.routes.size())).append(
     160        }
     161
     162        info.append(tr("Length: {0}", NavigatableComponent.getSystemOfMeasurement().getDistText(data.length()))).append("<br>")
     163            .append(trn("{0} route, ", "{0} routes, ", data.routes.size(), data.routes.size())).append(
    167164                trn("{0} waypoint", "{0} waypoints", data.waypoints.size(), data.waypoints.size())).append("<br>");
    168165
    … …  
    222219        }
    223220
    224         info.append(trn("{0} track, ", "{0} tracks, ", data.tracks.size(), data.tracks.size()));
    225         info.append(trn("{0} route, ", "{0} routes, ", data.routes.size(), data.routes.size()));
    226         info.append(trn("{0} waypoint", "{0} waypoints", data.waypoints.size(), data.waypoints.size())).append("<br>");
    227 
    228         info.append(tr("Length: {0}", NavigatableComponent.getSystemOfMeasurement().getDistText(data.length())));
    229         info.append("<br>");
    230 
    231         return info.append("</html>").toString();
     221        info.append(trn("{0} track, ", "{0} tracks, ", data.tracks.size(), data.tracks.size()))
     222            .append(trn("{0} route, ", "{0} routes, ", data.routes.size(), data.routes.size()))
     223            .append(trn("{0} waypoint", "{0} waypoints", data.waypoints.size(), data.waypoints.size())).append("<br>")
     224            .append(tr("Length: {0}", NavigatableComponent.getSystemOfMeasurement().getDistText(data.length())))
     225            .append("<br>")
     226            .append("</html>");
     227        return info.toString();
    232228    }
    233229

  • trunk/src/org/openstreetmap/josm/gui/layer/NoteLayer.java

    r8345 r8379  
    112112        if (noteData.getSelectedNote() != null) {
    113113            StringBuilder sb = new StringBuilder("<html>");
    114             sb.append(tr("Note"));
    115             sb.append(" ").append(noteData.getSelectedNote().getId());
     114            sb.append(tr("Note"))
     115              .append(" ").append(noteData.getSelectedNote().getId());
    116116            for (NoteComment comment : noteData.getSelectedNote().getComments()) {
    117117                String commentText = comment.getText();
    … …  
    195195    public Object getInfoComponent() {
    196196        StringBuilder sb = new StringBuilder();
    197         sb.append(tr("Notes layer"));
    198         sb.append("\n");
    199         sb.append(tr("Total notes:"));
    200         sb.append(" ");
    201         sb.append(noteData.getNotes().size());
    202         sb.append("\n");
    203         sb.append(tr("Changes need uploading?"));
    204         sb.append(" ");
    205         sb.append(isModified());
     197        sb.append(tr("Notes layer"))
     198          .append("\n")
     199          .append(tr("Total notes:"))
     200          .append(" ")
     201          .append(noteData.getNotes().size())
     202          .append("\n")
     203          .append(tr("Changes need uploading?"))
     204          .append(" ")
     205          .append(isModified());
    206206        return sb.toString();
    207207    }

  • trunk/src/org/openstreetmap/josm/gui/mappaint/TextElement.java

    r8291 r8379  
    178178    protected String toStringImpl() {
    179179        StringBuilder sb = new StringBuilder();
    180         sb.append("labelCompositionStrategy=" + labelCompositionStrategy);
    181         sb.append(" font=" + font);
     180        sb.append("labelCompositionStrategy=" + labelCompositionStrategy)
     181          .append(" font=" + font);
    182182        if (xOffset != 0) {
    183183            sb.append(" xOffset=" + xOffset);
    … …  
    188188        sb.append(" color=" + Utils.toString(color));
    189189        if (haloRadius != null) {
    190             sb.append(" haloRadius=" + haloRadius);
    191             sb.append(" haloColor=" + haloColor);
     190            sb.append(" haloRadius=" + haloRadius)
     191              .append(" haloColor=" + haloColor);
    192192        }
    193193        return sb.toString();

  • trunk/src/org/openstreetmap/josm/gui/preferences/imagery/AddTMSLayerPanel.java

    r8308 r8379  
    7171            a.append("[").append(z).append("]");
    7272        }
    73         a.append(":");
    74         a.append(getImageryRawUrl());
     73        a.append(":").append(getImageryRawUrl());
    7574        return a.toString();
    7675    }

  • trunk/src/org/openstreetmap/josm/gui/preferences/plugin/PluginListPanel.java

    r8342 r8379  
    178178    private static void alertPluginStillRequired(Component parent, String plugin, Set<String> otherPlugins) {
    179179        StringBuilder sb = new StringBuilder();
    180         sb.append("<html>");
    181         sb.append(trn("Plugin {0} is still required by this plugin:",
     180        sb.append("<html>")
     181          .append(trn("Plugin {0} is still required by this plugin:",
    182182                "Plugin {0} is still required by these {1} plugins:",
    183183                otherPlugins.size(),
    184184                plugin,
    185                 otherPlugins.size()
    186         ));
    187         sb.append(Utils.joinAsHtmlUnorderedList(otherPlugins));
    188         sb.append("</html>");
     185                otherPlugins.size()))
     186          .append(Utils.joinAsHtmlUnorderedList(otherPlugins))
     187          .append("</html>");
    189188        JOptionPane.showMessageDialog(
    190189                parent,

  • trunk/src/org/openstreetmap/josm/gui/preferences/plugin/PluginPreference.java

    r8378 r8379  
    127127        final Collection<PluginInformation> failed = task.getFailedPlugins();
    128128        final StringBuilder sb = new StringBuilder();
    129         sb.append("<html>");
    130         sb.append(buildDownloadSummary(task));
     129        sb.append("<html>")
     130          .append(buildDownloadSummary(task));
    131131        if (restartRequired) {
    132132            sb.append(tr("Please restart JOSM to activate the downloaded plugins."));

  • trunk/src/org/openstreetmap/josm/gui/preferences/projection/CustomProjectionChoice.java

    r8056 r8379  
    169169        private JComponent build() {
    170170            StringBuilder s = new StringBuilder();
    171             s.append("<b>+proj=...</b> - <i>"+tr("Projection name")+"</i><br>");
    172             s.append("&nbsp;&nbsp;&nbsp;&nbsp;"+tr("Supported values:")+" ");
    173             s.append(listKeys(Projections.projs)+"<br>");
    174             s.append("<b>+lat_0=..., +lat_1=..., +lat_2=...</b> - <i>"+tr("Projection parameters")+"</i><br>");
    175             s.append("<b>+x_0=..., +y_0=...</b> - <i>"+tr("False easting and false northing")+"</i><br>");
    176             s.append("<b>+lon_0=...</b> - <i>"+tr("Central meridian")+"</i><br>");
    177             s.append("<b>+k_0=...</b> - <i>"+tr("Scaling factor")+"</i><br>");
    178             s.append("<b>+ellps=...</b> - <i>"+tr("Ellipsoid name")+"</i><br>");
    179             s.append("&nbsp;&nbsp;&nbsp;&nbsp;"+tr("Supported values:")+" ");
    180             s.append(listKeys(Projections.ellipsoids)+"<br>");
    181             s.append("<b>+a=..., +b=..., +rf=..., +f=..., +es=...</b> - <i>"+tr("Ellipsoid parameters")+"</i><br>");
    182             s.append("<b>+datum=...</b> - <i>"+tr("Datum name")+"</i><br>");
    183             s.append("&nbsp;&nbsp;&nbsp;&nbsp;"+tr("Supported values:")+" ");
    184             s.append(listKeys(Projections.datums)+"<br>");
    185             s.append("<b>+towgs84=...</b> - <i>"+tr("3 or 7 term datum transform parameters")+"</i><br>");
    186             s.append("<b>+nadgrids=...</b> - <i>"+tr("NTv2 grid file")+"</i><br>");
    187             s.append("&nbsp;&nbsp;&nbsp;&nbsp;"+tr("Built-in:")+" ");
    188             s.append(listKeys(Projections.nadgrids)+"<br>");
    189             s.append("<b>+bounds=</b>minlon,minlat,maxlon,maxlat - <i>"+tr("Projection bounds (in degrees)")+"</i><br>");
    190             s.append("<b>+wmssrs=</b>EPSG:123456 - <i>"+tr("Sets the SRS=... parameter in the WMS request")+"</i><br>");
     171            s.append("<b>+proj=...</b> - <i>"+tr("Projection name")+"</i><br>")
     172             .append("&nbsp;&nbsp;&nbsp;&nbsp;"+tr("Supported values:")+" ")
     173             .append(listKeys(Projections.projs)+"<br>")
     174             .append("<b>+lat_0=..., +lat_1=..., +lat_2=...</b> - <i>"+tr("Projection parameters")+"</i><br>")
     175             .append("<b>+x_0=..., +y_0=...</b> - <i>"+tr("False easting and false northing")+"</i><br>")
     176             .append("<b>+lon_0=...</b> - <i>"+tr("Central meridian")+"</i><br>")
     177             .append("<b>+k_0=...</b> - <i>"+tr("Scaling factor")+"</i><br>")
     178             .append("<b>+ellps=...</b> - <i>"+tr("Ellipsoid name")+"</i><br>")
     179             .append("&nbsp;&nbsp;&nbsp;&nbsp;"+tr("Supported values:")+" ")
     180             .append(listKeys(Projections.ellipsoids)+"<br>")
     181             .append("<b>+a=..., +b=..., +rf=..., +f=..., +es=...</b> - <i>"+tr("Ellipsoid parameters")+"</i><br>")
     182             .append("<b>+datum=...</b> - <i>"+tr("Datum name")+"</i><br>")
     183             .append("&nbsp;&nbsp;&nbsp;&nbsp;"+tr("Supported values:")+" ")
     184             .append(listKeys(Projections.datums)+"<br>")
     185             .append("<b>+towgs84=...</b> - <i>"+tr("3 or 7 term datum transform parameters")+"</i><br>")
     186             .append("<b>+nadgrids=...</b> - <i>"+tr("NTv2 grid file")+"</i><br>")
     187             .append("&nbsp;&nbsp;&nbsp;&nbsp;"+tr("Built-in:")+" ")
     188             .append(listKeys(Projections.nadgrids)+"<br>")
     189             .append("<b>+bounds=</b>minlon,minlat,maxlon,maxlat - <i>"+tr("Projection bounds (in degrees)")+"</i><br>")
     190             .append("<b>+wmssrs=</b>EPSG:123456 - <i>"+tr("Sets the SRS=... parameter in the WMS request")+"</i><br>");
    191191
    192192            return new HtmlPanel(s.toString());

  • trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPresetItems.java

    r8377 r8379  
    121121
    122122            final StringBuilder res = new StringBuilder("<b>");
    123             res.append(getDisplayValue(true).replaceAll("<", "&lt;").replaceAll(">", "&gt;"));
    124             res.append("</b>");
     123            res.append(getDisplayValue(true).replaceAll("<", "&lt;").replaceAll(">", "&gt;"))
     124               .append("</b>");
    125125            if (getShortDescription(true) != null) {
    126126                // wrap in table to restrict the text width
    127                 res.append("<div style=\"width:300px; padding:0 0 5px 5px\">");
    128                 res.append(getShortDescription(true));
    129                 res.append("</div>");
     127                res.append("<div style=\"width:300px; padding:0 0 5px 5px\">")
     128                   .append(getShortDescription(true))
     129                   .append("</div>");
    130130            }
    131131            return res.toString();

  • trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionListItem.java

    r8376 r8379  
    8888    public String toString() {
    8989        StringBuilder sb = new StringBuilder();
    90         sb.append("<val='");
    91         sb.append(value);
    92         sb.append("',");
    93         sb.append(priority);
    94         sb.append(">");
     90        sb.append("<val='")
     91          .append(value)
     92          .append("',")
     93          .append(priority)
     94          .append(">");
    9595        return sb.toString();
    9696    }

  • trunk/src/org/openstreetmap/josm/gui/widgets/MultiSplitLayout.java

    r8376 r8379  
    10211021            int nChildren = getChildren().size();
    10221022            StringBuffer sb = new StringBuffer("MultiSplitLayout.Split");
    1023             sb.append(isRowLayout() ? " ROW [" : " COLUMN [");
    1024             sb.append(nChildren + ((nChildren == 1) ? " child" : " children"));
    1025             sb.append("] ");
    1026             sb.append(getBounds());
     1023            sb.append(isRowLayout() ? " ROW [" : " COLUMN [")
     1024              .append(nChildren + ((nChildren == 1) ? " child" : " children"))
     1025              .append("] ")
     1026              .append(getBounds());
    10271027            return sb.toString();
    10281028        }
    … …  
    10741074        public String toString() {
    10751075            StringBuffer sb = new StringBuffer("MultiSplitLayout.Leaf");
    1076             sb.append(" \"");
    1077             sb.append(getName());
    1078             sb.append('\"');
    1079             sb.append(" weight=");
    1080             sb.append(getWeight());
    1081             sb.append(' ');
    1082             sb.append(getBounds());
     1076            sb.append(" \"")
     1077              .append(getName())
     1078              .append('\"')
     1079              .append(" weight=")
     1080              .append(getWeight())
     1081              .append(' ')
     1082              .append(getBounds());
    10831083            return sb.toString();
    10841084        }
Note: See TracChangeset for help on using the changeset viewer.