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

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

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

Legend:

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

    r8308 r8379  
    157157        protected void alertFilesNotMatchingWithImporter(Collection<File> files, FileImporter importer) {
    158158            final StringBuilder msg = new StringBuilder();
    159             msg.append("<html>");
    160             msg.append(
     159            msg.append("<html>").append(
    161160                    trn(
    162161                            "Cannot open {0} file with the file importer ''{1}''.",
     
    166165                            importer.filter.getDescription()
    167166                    )
    168             ).append("<br>");
    169             msg.append("<ul>");
     167            ).append("<br>").append("<ul>");
    170168            for (File f: files) {
    171169                msg.append("<li>").append(f.getAbsolutePath()).append("</li>");
     
    184182        protected void alertFilesWithUnknownImporter(Collection<File> files) {
    185183            final StringBuilder msg = new StringBuilder();
    186             msg.append("<html>");
    187             msg.append(
     184            msg.append("<html>").append(
    188185                    trn(
    189186                            "Cannot open {0} file because file does not exist or no suitable file importer is available.",
     
    192189                            files.size()
    193190                    )
    194             ).append("<br>");
    195             msg.append("<ul>");
     191            ).append("<br>").append("<ul>");
    196192            for (File f: files) {
    197                 msg.append("<li>");
    198                 msg.append(f.getAbsolutePath());
    199                 msg.append(" (<i>");
    200                 msg.append(f.exists() ? tr("no importer") : tr("does not exist"));
    201                 msg.append("</i>)</li>");
     193                msg.append("<li>").append(f.getAbsolutePath()).append(" (<i>")
     194                   .append(f.exists() ? tr("no importer") : tr("does not exist"))
     195                   .append("</i>)</li>");
    202196            }
    203197            msg.append("</ul>");
  • trunk/src/org/openstreetmap/josm/actions/SearchNotesDownloadAction.java

    r8304 r8379  
    7373
    7474        performSearch(searchTerm);
    75 
    7675    }
    7776
     
    9190
    9291        StringBuilder sb = new StringBuilder();
    93         sb.append(OsmApi.getOsmApi().getBaseUrl());
    94         sb.append("notes/search?limit=");
    95         sb.append(noteLimit);
    96         sb.append("&closed=");
    97         sb.append(closedLimit);
    98         sb.append("&q=");
    99         sb.append(Utils.encodeUrl(searchTerm));
     92        sb.append(OsmApi.getOsmApi().getBaseUrl())
     93            .append("notes/search?limit=")
     94            .append(noteLimit)
     95            .append("&closed=")
     96            .append(closedLimit)
     97            .append("&q=")
     98            .append(Utils.encodeUrl(searchTerm));
    10099
    101100        new DownloadNotesTask().loadUrl(false, sb.toString(), null);
  • trunk/src/org/openstreetmap/josm/actions/ShowStatusReportAction.java

    r8376 r8379  
    6969    public static String getReportHeader() {
    7070        StringBuilder text = new StringBuilder();
    71         text.append(Version.getInstance().getReleaseAttributes());
    72         text.append("\n");
    73         text.append("Identification: " + Version.getInstance().getAgentString());
    74         text.append("\n");
    75         text.append("Memory Usage: ");
    76         text.append(Runtime.getRuntime().totalMemory()/1024/1024);
    77         text.append(" MB / ");
    78         text.append(Runtime.getRuntime().maxMemory()/1024/1024);
    79         text.append(" MB (");
    80         text.append(Runtime.getRuntime().freeMemory()/1024/1024);
    81         text.append(" MB allocated, but free)");
    82         text.append("\n");
    83         text.append("Java version: " + System.getProperty("java.version") + ", " + System.getProperty("java.vendor") + ", " + System.getProperty("java.vm.name"));
    84         text.append("\n");
     71        text.append(Version.getInstance().getReleaseAttributes())
     72            .append("\n")
     73            .append("Identification: ").append(Version.getInstance().getAgentString())
     74            .append("\n")
     75            .append("Memory Usage: ")
     76            .append(Runtime.getRuntime().totalMemory()/1024/1024)
     77            .append(" MB / ")
     78            .append(Runtime.getRuntime().maxMemory()/1024/1024)
     79            .append(" MB (")
     80            .append(Runtime.getRuntime().freeMemory()/1024/1024)
     81            .append(" MB allocated, but free)")
     82            .append("\n")
     83            .append("Java version: ").append(System.getProperty("java.version")).append(", ")
     84            .append(System.getProperty("java.vendor")).append(", ").append(System.getProperty("java.vm.name"))
     85            .append("\n");
    8586        if (Main.platform.getClass() == PlatformHookUnixoid.class) {
    8687            // Add Java package details
    8788            String packageDetails = ((PlatformHookUnixoid) Main.platform).getJavaPackageDetails();
    8889            if (packageDetails != null) {
    89                 text.append("Java package: ");
    90                 text.append(packageDetails);
    91                 text.append("\n");
     90                text.append("Java package: ")
     91                    .append(packageDetails)
     92                    .append("\n");
    9293            }
    9394            // Add WebStart package details if run from JNLP
     
    9596                String webStartDetails = ((PlatformHookUnixoid) Main.platform).getWebStartPackageDetails();
    9697                if (webStartDetails != null) {
    97                     text.append("WebStart package: ");
    98                     text.append(webStartDetails);
    99                     text.append("\n");
     98                    text.append("WebStart package: ")
     99                        .append(webStartDetails)
     100                        .append("\n");
    100101                }
    101102            }
     
    148149            }
    149150        }
    150         text.append("\n");
    151         text.append(PluginHandler.getBugReportText());
    152         text.append("\n");
     151        text.append("\n")
     152            .append(PluginHandler.getBugReportText())
     153            .append("\n");
    153154
    154155        Collection<String> errorsWarnings = Main.getLastErrorAndWarnings();
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/AbstractDownloadTask.java

    r8195 r8379  
    44import java.util.ArrayList;
    55import java.util.List;
     6
    67import org.openstreetmap.josm.io.XmlWriter;
    78
     
    5051    @Override
    5152    public String acceptsDocumentationSummary() {
    52         StringBuilder buf = new StringBuilder("<tr><td>");
    53         buf.append(getTitle());
    54         buf.append(":</td><td>");
     53        StringBuilder buff = new StringBuilder("<tr><td>");
     54        buff.append(getTitle())
     55            .append(":</td><td>");
    5556        String[] patterns = getPatterns();
    5657        if (patterns.length>0) {
    57             buf.append("<ul>");
     58            buff.append("<ul>");
    5859            for (String pattern: patterns) {
    59                 buf.append("<li>");
    60                 buf.append(XmlWriter.encode(pattern));
    61                 buf.append("</li>");
     60                buff.append("<li>")
     61                    .append(XmlWriter.encode(pattern))
     62                    .append("</li>");
    6263            }
    63             buf.append("</ul>");
     64            buff.append("</ul>");
    6465        }
    65         buf.append("</td></tr>");
    66         return buf.toString();
     66        buff.append("</td></tr>");
     67        return buff.toString();
    6768    }
    6869
     
    8081    /**
    8182     * Check / decide if the task is safe for remotecontrol.
    82      * 
     83     *
    8384     * Keep in mind that a potential attacker has full control over the content
    8485     * of the file that will be downloaded.
    8586     * If it is possible to run arbitrary code or write to the local file
    8687     * system, then the task is (obviously) not save for remote execution.
    87      * 
     88     *
    8889     * The default value is false = unsafe. Override in a subclass to
    8990     * allow running the task via remotecontol.
    90      * 
     91     *
    9192     * @return true if it is safe to download and open any file of the
    9293     * corresponding format, false otherwise
  • trunk/src/org/openstreetmap/josm/actions/search/SearchAction.java

    r8377 r8379  
    678678                result.append('A');
    679679            }
    680             result.append(' ');
    681             result.append(text);
     680            result.append(' ')
     681                  .append(text);
    682682            return result.toString();
    683683        }
  • trunk/src/org/openstreetmap/josm/actions/upload/CyclicUploadDependencyException.java

    r7005 r8379  
    3232    public String getMessage() {
    3333        StringBuilder sb = new StringBuilder();
    34         sb.append(tr("Cyclic dependency between relations:"));
    35         sb.append("[");
     34        sb.append(tr("Cyclic dependency between relations:"))
     35          .append("[");
    3636        for (int i=0; i< cycle.size(); i++) {
    3737            if (i >0 ) {
Note: See TracChangeset for help on using the changeset viewer.