Ticket #8902: stringbuilder.diff

File stringbuilder.diff, 24.2 KB (added by shinigami, 11 years ago)

not-escaping local StringBuffer => StringBuilder

  • src/org/openstreetmap/josm/actions/OpenFileAction.java

     
    132132        }
    133133
    134134        protected void alertFilesNotMatchingWithImporter(Collection<File> files, FileImporter importer) {
    135             final StringBuffer msg = new StringBuffer();
     135            final StringBuilder msg = new StringBuilder();
    136136            msg.append("<html>");
    137137            msg.append(
    138138                    trn(
    … …  
    159159        }
    160160
    161161        protected void alertFilesWithUnknownImporter(Collection<File> files) {
    162             final StringBuffer msg = new StringBuffer();
     162            final StringBuilder msg = new StringBuilder();
    163163            msg.append("<html>");
    164164            msg.append(
    165165                    trn(

  • src/org/openstreetmap/josm/actions/downloadtasks/PostDownloadHandler.java

     
    103103        // multiple error object? prepare a HTML list
    104104        //
    105105        if (!errors.isEmpty()) {
    106             final StringBuffer sb = new StringBuffer();
     106            final StringBuilder sb = new StringBuilder();
    107107            for (Object error:errors) {
    108108                if (error instanceof String) {
    109109                    sb.append("<li>").append(error).append("</li>").append("<br>");

  • src/org/openstreetmap/josm/actions/upload/CyclicUploadDependencyException.java

     
    1818    }
    1919
    2020    protected String formatRelation(Relation r) {
    21         StringBuffer sb = new StringBuffer();
     21        StringBuilder sb = new StringBuilder();
    2222        if (r.getName() != null) {
    2323            sb.append("'").append(r.getName()).append("'");
    2424        } else if (!r.isNew()) {
    … …  
    3131
    3232    @Override
    3333    public String getMessage() {
    34         StringBuffer sb = new StringBuffer();
     34        StringBuilder sb = new StringBuilder();
    3535        sb.append(tr("Cyclic dependency between relations:"));
    3636        sb.append("[");
    3737        for (int i=0; i< cycle.size(); i++) {

  • src/org/openstreetmap/josm/data/Bounds.java

     
    309309    }
    310310
    311311    public String encodeAsString(String separator) {
    312         StringBuffer sb = new StringBuffer();
     312        StringBuilder sb = new StringBuilder();
    313313        sb.append(minLat).append(separator).append(minLon)
    314314        .append(separator).append(maxLat).append(separator)
    315315        .append(maxLon);

  • src/org/openstreetmap/josm/data/Version.java

     
    3737        String s = null;
    3838        try {
    3939            BufferedReader in = Utils.openURLReader(resource);
    40             StringBuffer sb = new StringBuffer();
     40            StringBuilder sb = new StringBuilder();
    4141            try {
    4242                for (String line = in.readLine(); line != null; line = in.readLine()) {
    4343                    sb.append(line).append("\n");
    … …  
    144144
    145145        // the revision info
    146146        //
    147         StringBuffer sb = new StringBuffer();
     147        StringBuilder sb = new StringBuilder();
    148148        for(Entry<String,String> property: properties.entrySet()) {
    149149            sb.append(property.getKey()).append(":").append(property.getValue()).append("\n");
    150150        }

  • src/org/openstreetmap/josm/data/imagery/Shape.java

     
    3535    }
    3636
    3737    public String encodeAsString(String separator) {
    38         StringBuffer sb = new StringBuffer();
     38        StringBuilder sb = new StringBuilder();
    3939        for (Coordinate c : coords) {
    4040            if (sb.length() != 0) {
    4141                sb.append(separator);

  • src/org/openstreetmap/josm/data/osm/User.java

     
    203203
    204204    @Override
    205205    public String toString() {
    206         StringBuffer s = new StringBuffer();
     206        StringBuilder s = new StringBuilder();
    207207        s.append("id:"+uid);
    208208        if (names.size() == 1) {
    209209            s.append(" name:"+getName());

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

     
    275275
    276276    @Override
    277277    public String toString() {
    278         StringBuffer buf = new StringBuffer("Headers  : ");
     278        StringBuilder buf = new StringBuilder("Headers  : ");
    279279        buf.append(overviewHeaderCount);
    280280        buf.append("\nSub Hdrs : ");
    281281        buf.append(subGridHeaderCount);

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

     
    289289    }
    290290
    291291    public String getDetails() {
    292         StringBuffer buf = new StringBuffer("Sub Grid : ");
     292        StringBuilder buf = new StringBuilder("Sub Grid : ");
    293293        buf.append(subGridName);
    294294        buf.append("\nParent   : ");
    295295        buf.append(parentSubGridName);

  • src/org/openstreetmap/josm/gui/BookmarkList.java

     
    214214
    215215        protected String buildToolTipText(Bookmark b) {
    216216            Bounds area = b.getArea();
    217             StringBuffer sb = new StringBuffer();
     217            StringBuilder sb = new StringBuilder();
    218218            sb.append("<html>min[latitude,longitude]=<strong>[")
    219219            .append(area.getMin().lat()).append(",").append(area.getMin().lon()).append("]</strong>")
    220220            .append("<br>")

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

     
    10571057        @Override
    10581058        public String toString() {
    10591059            int nChildren = getChildren().size();
    1060             StringBuffer sb = new StringBuffer("MultiSplitLayout.Split");
     1060            StringBuilder sb = new StringBuilder("MultiSplitLayout.Split");
    10611061            sb.append(isRowLayout() ? " ROW [" : " COLUMN [");
    10621062            sb.append(nChildren + ((nChildren == 1) ? " child" : " children"));
    10631063            sb.append("] ");
    … …  
    11121112
    11131113        @Override
    11141114        public String toString() {
    1115             StringBuffer sb = new StringBuffer("MultiSplitLayout.Leaf");
     1115            StringBuilder sb = new StringBuilder("MultiSplitLayout.Leaf");
    11161116            sb.append(" \"");
    11171117            sb.append(getName());
    11181118            sb.append("\"");

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

     
    653653
    654654        @Override
    655655        public String toString() {
    656             StringBuffer sb = new StringBuffer();
     656            StringBuilder sb = new StringBuilder();
    657657            sb.append("min=").append(min.x).append(",").append(min.y).append(",");
    658658            sb.append("max=").append(max.x).append(",").append(max.y).append(",");
    659659            sb.append("zoom=").append(zoomLevel);

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

     
    354354        if (deletedIds.size() > MAX_DELETED_PRIMITIVE_IN_DIALOG) {
    355355            items.add(tr("{0} more...", deletedIds.size() - MAX_DELETED_PRIMITIVE_IN_DIALOG));
    356356        }
    357         StringBuffer sb = new StringBuffer();
     357        StringBuilder sb = new StringBuilder();
    358358        sb.append("<html>");
    359359        sb.append(tr("The following objects could not be copied to the target object<br>because they are deleted in the target dataset:"));
    360360        sb.append("<ul>");

  • src/org/openstreetmap/josm/gui/dialogs/SelectionListDialog.java

     
    710710                    relations++;
    711711                }
    712712            }
    713             StringBuffer text = new StringBuffer();
     713            StringBuilder text = new StringBuilder();
    714714            if(ways != 0) {
    715715                text.append(text.length() > 0 ? ", " : "")
    716716                .append(trn("{0} way", "{0} ways", ways, ways));

  • src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetListCellRenderer.java

     
    3131    }
    3232
    3333    protected void renderLabel(Changeset cs) {
    34         StringBuffer sb = new StringBuffer();
     34        StringBuilder sb = new StringBuilder();
    3535        if (cs.isIncomplete()) {
    3636            sb.append(tr("{0} [incomplete]", cs.getId()));
    3737        } else {

  • src/org/openstreetmap/josm/gui/download/PlaceSelection.java

     
    510510        }
    511511
    512512        protected String lineWrapDescription(String description) {
    513             StringBuffer ret = new StringBuffer();
    514             StringBuffer line = new StringBuffer();
     513            StringBuilder ret = new StringBuilder();
     514            StringBuilder line = new StringBuilder();
    515515            StringTokenizer tok = new StringTokenizer(description, " ");
    516516            while(tok.hasMoreElements()) {
    517517                String t = tok.nextToken();
    … …  
    522522                } else {
    523523                    line.append(" ").append(t).append("<br>");
    524524                    ret.append(line);
    525                     line = new StringBuffer();
     525                    line = new StringBuilder();
    526526                }
    527527            }
    528528            ret.insert(0, "<html>");

  • src/org/openstreetmap/josm/gui/help/HelpBrowser.java

     
    136136                        getClass().getResourceAsStream("/data/help-browser.css")
    137137                )
    138138        );
    139         StringBuffer css = new StringBuffer();
     139        StringBuilder css = new StringBuilder();
    140140        try {
    141141            String line = null;
    142142            while ((line = reader.readLine()) != null) {

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

     
    5353        }
    5454        if (cs != null) {
    5555            setIcon(icon);
    56             StringBuffer sb = new StringBuffer();
     56            StringBuilder sb = new StringBuilder();
    5757            if (cs.get("comment") != null) {
    5858                sb.append(cs.getId()).append(" - ").append(cs.get("comment"));
    5959            } else if (cs.get("name") != null) {

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

     
    3636    private ConfigurationParameterRequestHandler configHandler;
    3737
    3838    protected String buildChangesetSummary() {
    39         StringBuffer msg = new StringBuffer();
     39        StringBuilder msg = new StringBuilder();
    4040        if (selectedChangeset == null || selectedChangeset.isNew()) {
    4141            msg.append(tr("Objects are uploaded to a <strong>new changeset</strong>."));
    4242        } else {
    … …  
    170170    }
    171171
    172172    protected void updateSummary() {
    173         StringBuffer sb = new StringBuffer();
     173        StringBuilder sb = new StringBuilder();
    174174        sb.append("<html>");
    175175        sb.append(buildStrategySummary());
    176176        sb.append("<br><br>");

  • src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

     
    296296            for (String tzStr : tmp) {
    297297                TimeZone tz = TimeZone.getTimeZone(tzStr);
    298298
    299                 String tzDesc = new StringBuffer(tzStr).append(" (")
     299                String tzDesc = new StringBuilder(tzStr).append(" (")
    300300                .append(formatTimezone(tz.getRawOffset() / 3600000.0))
    301301                .append(')').toString();
    302302                vtTimezones.add(tzDesc);
    … …  
    314314                defaultTz = TimeZone.getTimeZone(tzId);
    315315            }
    316316
    317             cbTimezones.setSelectedItem(new StringBuffer(defaultTz.getID()).append(" (")
     317            cbTimezones.setSelectedItem(new StringBuilder(defaultTz.getID()).append(" (")
    318318                    .append(formatTimezone(defaultTz.getRawOffset() / 3600000.0))
    319319                    .append(')').toString());
    320320
    … …  
    12991299    }
    13001300
    13011301    private String formatTimezone(double timezone) {
    1302         StringBuffer ret = new StringBuffer();
     1302        StringBuilder ret = new StringBuilder();
    13031303
    13041304        if (timezone < 0) {
    13051305            ret.append('-');
    … …  
    13251325            return 0;
    13261326
    13271327        char sgnTimezone = '+';
    1328         StringBuffer hTimezone = new StringBuffer();
    1329         StringBuffer mTimezone = new StringBuffer();
     1328        StringBuilder hTimezone = new StringBuilder();
     1329        StringBuilder mTimezone = new StringBuilder();
    13301330        int state = 1; // 1=start/sign, 2=hours, 3=minutes.
    13311331        for (int i = 0; i < timezone.length(); i++) {
    13321332            char c = timezone.charAt(i);

  • src/org/openstreetmap/josm/gui/layer/geoimage/ImageViewerDialog.java

     
    234234        if (entry != null) {
    235235            imgDisplay.setImage(entry.getFile(), entry.getExifOrientation());
    236236            setTitle("Geotagged Images" + (entry.getFile() != null ? " - " + entry.getFile().getName() : ""));
    237             StringBuffer osd = new StringBuffer(entry.getFile() != null ? entry.getFile().getName() : "");
     237            StringBuilder osd = new StringBuilder(entry.getFile() != null ? entry.getFile().getName() : "");
    238238            if (entry.getElevation() != null) {
    239239                osd.append(tr("\nAltitude: {0} m", entry.getElevation().longValue()));
    240240            }

  • src/org/openstreetmap/josm/io/ChangesetQuery.java

     
    230230     * @return the query string
    231231     */
    232232    public String getQueryString() {
    233         StringBuffer sb = new StringBuffer();
     233        StringBuilder sb = new StringBuilder();
    234234        if (uid != null) {
    235235            sb.append("user").append("=").append(uid);
    236236        } else if (userName != null) {

  • src/org/openstreetmap/josm/io/OsmApi.java

     
    314314     * @return base URL string
    315315     */
    316316    public String getBaseUrl() {
    317         StringBuffer rv = new StringBuffer(serverUrl);
     317        StringBuilder rv = new StringBuilder(serverUrl);
    318318        if (version != null) {
    319319            rv.append("/");
    320320            rv.append(version);
    … …  
    589589     *    been exhausted), or rewrapping a Java exception.
    590590     */
    591591    private String sendRequest(String requestMethod, String urlSuffix,String requestBody, ProgressMonitor monitor, boolean doAuthenticate, boolean fastFail) throws OsmTransferException {
    592         StringBuffer responseBody = new StringBuffer();
     592        StringBuilder responseBody = new StringBuilder();
    593593        int retries = fastFail ? 0 : getMaxRetries();
    594594
    595595        while(true) { // the retry loop

  • src/org/openstreetmap/josm/io/OsmServerBackreferenceReader.java

     
    134134        progressMonitor.beginTask(null, 2);
    135135        try {
    136136            progressMonitor.indeterminateSubTask(tr("Downloading from OSM Server..."));
    137             StringBuffer sb = new StringBuffer();
     137            StringBuilder sb = new StringBuilder();
    138138            sb.append(primitiveType.getAPIName())
    139139            .append("/").append(id).append("/ways");
    140140
    … …  
    168168        progressMonitor.beginTask(null, 2);
    169169        try {
    170170            progressMonitor.subTask(tr("Contacting OSM Server..."));
    171             StringBuffer sb = new StringBuffer();
     171            StringBuilder sb = new StringBuilder();
    172172            sb.append(primitiveType.getAPIName())
    173173            .append("/").append(id).append("/relations");
    174174

  • src/org/openstreetmap/josm/io/OsmServerChangesetReader.java

     
    5858        }
    5959        try {
    6060            monitor.beginTask(tr("Reading changesets..."));
    61             StringBuffer sb = new StringBuffer();
     61            StringBuilder sb = new StringBuilder();
    6262            sb.append("changesets?").append(query.getQueryString());
    6363            InputStream in = getInputStream(sb.toString(), monitor.createSubTaskMonitor(1, true));
    6464            if (in == null)
    … …  
    9292        }
    9393        try {
    9494            monitor.beginTask(tr("Reading changeset {0} ...",id));
    95             StringBuffer sb = new StringBuffer();
     95            StringBuilder sb = new StringBuilder();
    9696            sb.append("changeset/").append(id);
    9797            InputStream in = getInputStream(sb.toString(), monitor.createSubTaskMonitor(1, true));
    9898            if (in == null)
    … …  
    137137                    continue;
    138138                }
    139139                i++;
    140                 StringBuffer sb = new StringBuffer();
     140                StringBuilder sb = new StringBuilder();
    141141                sb.append("changeset/").append(id);
    142142                InputStream in = getInputStream(sb.toString(), monitor.createSubTaskMonitor(1, true));
    143143                if (in == null)
    … …  
    177177        }
    178178        try {
    179179            monitor.beginTask(tr("Downloading changeset content"));
    180             StringBuffer sb = new StringBuffer();
     180            StringBuilder sb = new StringBuilder();
    181181            sb.append("changeset/").append(id).append("/download");
    182182            InputStream in = getInputStream(sb.toString(), monitor.createSubTaskMonitor(1, true));
    183183            if (in == null)

  • src/org/openstreetmap/josm/io/OsmServerHistoryReader.java

     
    5959        progressMonitor.beginTask("");
    6060        try {
    6161            progressMonitor.indeterminateSubTask(tr("Contacting OSM Server..."));
    62             StringBuffer sb = new StringBuffer();
     62            StringBuilder sb = new StringBuilder();
    6363            sb.append(primitiveType.getAPIName())
    6464            .append("/").append(id).append("/history");
    6565

  • src/org/openstreetmap/josm/io/OsmServerObjectReader.java

     
    121121        InputStream in = null;
    122122        try {
    123123            progressMonitor.indeterminateSubTask(tr("Downloading OSM data..."));
    124             StringBuffer sb = new StringBuffer();
     124            StringBuilder sb = new StringBuilder();
    125125            sb.append(id.getType().getAPIName());
    126126            sb.append("/");
    127127            sb.append(id.getUniqueId());

  • src/org/openstreetmap/josm/plugins/PluginHandler.java

     
    734734    }
    735735
    736736    private static void alertFailedPluginUpdate(Component parent, Collection<PluginInformation> plugins) {
    737         StringBuffer sb = new StringBuffer();
     737        StringBuilder sb = new StringBuilder();
    738738        sb.append("<html>");
    739739        sb.append(trn(
    740740                "Updating the following plugin has failed:",
    … …  
    10821082                )
    10831083        };
    10841084
    1085         StringBuffer msg = new StringBuffer();
     1085        StringBuilder msg = new StringBuilder();
    10861086        msg.append("<html>");
    10871087        msg.append(tr("An unexpected exception occurred that may have come from the ''{0}'' plugin.", plugin.getPluginInformation().name));
    10881088        msg.append("<br>");

  • src/org/openstreetmap/josm/tools/WindowGeometry.java

     
    247247     * @param preferenceKey the preference key
    248248     */
    249249    public void remember(String preferenceKey) {
    250         StringBuffer value = new StringBuffer();
     250        StringBuilder value = new StringBuilder();
    251251        value.append("x=").append(topLeft.x).append(",")
    252252        .append("y=").append(topLeft.y).append(",")
    253253        .append("width=").append(extent.width).append(",")