Changeset 8390 in josm


Ignore:
Timestamp:
2015-05-18T02:14:30+02:00 (9 years ago)
Author:
Don-vip
Message:

Sonar - various performance improvements

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/CombineWayAction.java

    r8338 r8390  
    326326        public String toString() {
    327327            return new StringBuilder()
    328             .append("[")
     328            .append('[')
    329329            .append(a.getId())
    330             .append(",")
     330            .append(',')
    331331            .append(b.getId())
    332             .append("]")
     332            .append(']')
    333333            .toString();
    334334        }

  • trunk/src/org/openstreetmap/josm/actions/CopyAction.java

    r8194 r8390  
    6767        StringBuilder idsBuilder = new StringBuilder();
    6868        for (OsmPrimitive p : primitives) {
    69             idsBuilder.append(OsmPrimitiveType.from(p).getAPIName()).append(" ");
    70             idsBuilder.append(p.getId()).append(",");
     69            idsBuilder.append(OsmPrimitiveType.from(p).getAPIName()).append(' ').append(p.getId()).append(',');
    7170        }
    7271        return idsBuilder.substring(0, idsBuilder.length() - 1);

  • trunk/src/org/openstreetmap/josm/actions/CopyCoordinatesAction.java

    r7937 r8390  
    3232            s.append(", ");
    3333            s.append(n.getCoor().lon());
    34             s.append("\n");
     34            s.append('\n');
    3535        }
    3636        Utils.copyToClipboard(s.toString().trim());

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

    r8379 r8390  
    165165                            importer.filter.getDescription()
    166166                    )
    167             ).append("<br>").append("<ul>");
     167            ).append("<br><ul>");
    168168            for (File f: files) {
    169169                msg.append("<li>").append(f.getAbsolutePath()).append("</li>");
    … …  
    189189                            files.size()
    190190                    )
    191             ).append("<br>").append("<ul>");
     191            ).append("<br><ul>");
    192192            for (File f: files) {
    193193                msg.append("<li>").append(f.getAbsolutePath()).append(" (<i>")

  • trunk/src/org/openstreetmap/josm/actions/RestartAction.java

    r8376 r8390  
    116116                StringBuilder sb = new StringBuilder(mainCommand[0]);
    117117                for (int i = 1; i < mainCommand.length && !mainCommand[i-1].endsWith(".jar"); i++) {
    118                     sb.append(" ").append(mainCommand[i]);
     118                    sb.append(' ').append(mainCommand[i]);
    119119                }
    120120                String jarPath = sb.toString();

  • trunk/src/org/openstreetmap/josm/actions/ShowStatusReportAction.java

    r8379 r8390  
    7070        StringBuilder text = new StringBuilder();
    7171        text.append(Version.getInstance().getReleaseAttributes())
    72             .append("\n")
    73             .append("Identification: ").append(Version.getInstance().getAgentString())
    74             .append("\n")
    75             .append("Memory Usage: ")
     72            .append("\nIdentification: ").append(Version.getInstance().getAgentString())
     73            .append("\nMemory Usage: ")
    7674            .append(Runtime.getRuntime().totalMemory()/1024/1024)
    7775            .append(" MB / ")
    … …  
    7977            .append(" MB (")
    8078            .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");
     79            .append(" MB allocated, but free)\nJava version: ")
     80            .append(System.getProperty("java.version")).append(", ")
     81            .append(System.getProperty("java.vendor")).append(", ")
     82            .append(System.getProperty("java.vm.name")).append('\n');
    8683        if (Main.platform.getClass() == PlatformHookUnixoid.class) {
    8784            // Add Java package details
    … …  
    9087                text.append("Java package: ")
    9188                    .append(packageDetails)
    92                     .append("\n");
     89                    .append('\n');
    9390            }
    9491            // Add WebStart package details if run from JNLP
    … …  
    9895                    text.append("WebStart package: ")
    9996                        .append(webStartDetails)
    100                         .append("\n");
     97                        .append('\n');
    10198                }
    10299            }
    … …  
    127124            }
    128125            if (!vmArguments.isEmpty()) {
    129                 text.append("VM arguments: "+ vmArguments.toString().replace("\\\\", "\\"));
    130                 text.append("\n");
     126                text.append("VM arguments: ").append(vmArguments.toString().replace("\\\\", "\\")).append('\n');
    131127            }
    132128        } catch (SecurityException e) {
    … …  
    135131        List<String> commandLineArgs = Main.getCommandLineArgs();
    136132        if (!commandLineArgs.isEmpty()) {
    137             text.append("Program arguments: "+ Arrays.toString(commandLineArgs.toArray()));
    138             text.append("\n");
     133            text.append("Program arguments: ").append(Arrays.toString(commandLineArgs.toArray())).append('\n');
    139134        }
    140135        if (Main.main != null) {
    … …  
    145140                    text.append("Dataset consistency test: No problems found\n");
    146141                } else {
    147                     text.append("\nDataset consistency test:\n"+result+"\n");
    148                 }
    149             }
    150         }
    151         text.append("\n")
    152             .append(PluginHandler.getBugReportText())
    153             .append("\n");
     142                    text.append("\nDataset consistency test:\n").append(result).append('\n');
     143                }
     144            }
     145        }
     146        text.append('\n').append(PluginHandler.getBugReportText()).append('\n');
    154147
    155148        Collection<String> errorsWarnings = Main.getLastErrorAndWarnings();
    … …  
    157150            text.append("Last errors/warnings:\n");
    158151            for (String s : errorsWarnings) {
    159                 text.append("- ").append(s).append("\n");
    160             }
    161             text.append("\n");
     152                text.append("- ").append(s).append('\n');
     153            }
     154            text.append('\n');
    162155        }
    163156
    … …  
    180173            }
    181174            for (Entry<String, Setting<?>> entry : settings.entrySet()) {
    182                 text.append(entry.getKey()).append("=").append(entry.getValue().getValue()).append("\n");
     175                text.append(entry.getKey()).append('=').append(entry.getValue().getValue()).append('\n');
    183176            }
    184177        } catch (Exception x) {

  • trunk/src/org/openstreetmap/josm/actions/ToggleAction.java

    r8308 r8390  
    33
    44import java.awt.event.ActionEvent;
    5 import java.util.ArrayList;
    6 import java.util.List;
     5import java.util.HashSet;
     6import java.util.Set;
    77
    88import javax.swing.ButtonModel;
    … …  
    2323public abstract class ToggleAction extends JosmAction {
    2424
    25     private final transient List<ButtonModel> buttonModels = new ArrayList<>();
     25    private final transient Set<ButtonModel> buttonModels = new HashSet<>();
    2626
    2727    /**

  • trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

    r8384 r8390  
    11921192        if (getCurrentBaseNode() != null && !wayIsFinished) {
    11931193            if (alt) {
    1194                 rv.append(" ").append(tr("Start new way from last node."));
     1194                rv.append(' ').append(tr("Start new way from last node."));
    11951195            } else {
    1196                 rv.append(" ").append(tr("Continue way from last node."));
     1196                rv.append(' ').append(tr("Continue way from last node."));
    11971197            }
    11981198            if (snapHelper.isSnapOn()) {
    1199                 rv.append(" ").append(tr("Angle snapping active."));
     1199                rv.append(' ').append(tr("Angle snapping active."));
    12001200            }
    12011201        }
    … …  
    12201220            for (Node m : w.getNodes()) {
    12211221                if (m.equals(mouseOnExistingNode) || mouseOnExistingWays.contains(w)) {
    1222                     rv.append(" ").append(tr("Finish drawing."));
     1222                    rv.append(' ').append(tr("Finish drawing."));
    12231223                    break;
    12241224                }

  • trunk/src/org/openstreetmap/josm/actions/mapmode/ExtrudeAction.java

    r8345 r8390  
    252252                "Alt-drag to create a new rectangle, double click to add a new node."));
    253253            if (dualAlignEnabled) {
    254                 rv.append(" ").append(tr("Dual alignment active."));
     254                rv.append(' ').append(tr("Dual alignment active."));
    255255                if (dualAlignSegmentCollapsed)
    256                     rv.append(" ").append(tr("Segment collapsed due to its direction reversing."));
     256                    rv.append(' ').append(tr("Segment collapsed due to its direction reversing."));
    257257            }
    258258        } else {
    … …  
    270270            }
    271271            if (dualAlignActive) {
    272                 rv.append(" ").append(tr("Dual alignment active."));
     272                rv.append(' ').append(tr("Dual alignment active."));
    273273                if (dualAlignSegmentCollapsed) {
    274                     rv.append(" ").append(tr("Segment collapsed due to its direction reversing."));
     274                    rv.append(' ').append(tr("Segment collapsed due to its direction reversing."));
    275275                }
    276276            }

  • trunk/src/org/openstreetmap/josm/actions/search/PushbackTokenizer.java

    r7463 r8390  
    177177            // try parsing number
    178178            try {
    179                 currentNumber = Long.parseLong(currentText);
     179                currentNumber = Long.valueOf(currentText);
    180180            } catch (NumberFormatException e) {
    181181                currentNumber = null;
    … …  
    186186            if (isRange) {
    187187                try {
    188                     currentNumber = Long.parseLong(currentText.substring(0, pos));
     188                    currentNumber = Long.valueOf(currentText.substring(0, pos));
    189189                } catch (NumberFormatException e) {
    190190                    currentNumber = null;
    191191                }
    192192                try {
    193                     currentRange = Long.parseLong(currentText.substring(pos + 1));
     193                    currentRange = Long.valueOf(currentText.substring(pos + 1));
    194194                } catch (NumberFormatException e) {
    195195                    currentRange = null;

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

    r8379 r8390  
    2020        StringBuilder sb = new StringBuilder();
    2121        if (r.getName() != null) {
    22             sb.append("'").append(r.getName()).append("'");
     22            sb.append('\'').append(r.getName()).append('\'');
    2323        } else if (!r.isNew()) {
    2424            sb.append(r.getId());
    … …  
    3333        StringBuilder sb = new StringBuilder();
    3434        sb.append(tr("Cyclic dependency between relations:"))
    35           .append("[");
     35          .append('[');
    3636        for (int i=0; i< cycle.size(); i++) {
    3737            if (i >0 ) {
    38                 sb.append(",");
     38                sb.append(',');
    3939            }
    4040            sb.append(formatRelation(cycle.get(i)));
    4141        }
    42         sb.append("]");
     42        sb.append(']');
    4343        return sb.toString();
    4444    }

  • trunk/src/org/openstreetmap/josm/actions/upload/UploadNotesTask.java

    r8380 r8390  
    133133                for (Map.Entry<Note, Exception> entry : failedNotes.entrySet()) {
    134134                    sb.append(tr("Note {0} failed: {1}", entry.getKey().getId(), entry.getValue().getMessage()));
    135                     sb.append("\n");
     135                    sb.append('\n');
    136136                }
    137137                Main.error("Notes failed to upload: " + sb.toString());

  • trunk/src/org/openstreetmap/josm/data/CustomConfigurator.java

    r8338 r8390  
    7575    public static void log(String s) {
    7676        summary.append(s);
    77         summary.append("\n");
     77        summary.append('\n');
    7878    }
    7979

  • trunk/src/org/openstreetmap/josm/data/Preferences.java

    r8357 r8390  
    13701370            f.setAccessible(true);
    13711371            if (f.getType() == Boolean.class || f.getType() == boolean.class) {
    1372                 value = Boolean.parseBoolean(key_value.getValue());
     1372                value = Boolean.valueOf(key_value.getValue());
    13731373            } else if (f.getType() == Integer.class || f.getType() == int.class) {
    13741374                try {
    1375                     value = Integer.parseInt(key_value.getValue());
     1375                    value = Integer.valueOf(key_value.getValue());
    13761376                } catch (NumberFormatException nfe) {
    13771377                    continue;
    … …  
    13791379            } else if (f.getType() == Double.class || f.getType() == double.class) {
    13801380                try {
    1381                     value = Double.parseDouble(key_value.getValue());
     1381                    value = Double.valueOf(key_value.getValue());
    13821382                } catch (NumberFormatException nfe) {
    13831383                    continue;
    … …  
    17011701    public String toXML(boolean nopass) {
    17021702        StringBuilder b = new StringBuilder(
    1703                 "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n" +
    1704                 "<preferences xmlns=\""+Main.getXMLBase()+"/preferences-1.0\" version=\""+
    1705                 Version.getInstance().getVersion() + "\">\n");
     1703                "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<preferences xmlns=\"")
     1704                .append(Main.getXMLBase()).append("/preferences-1.0\" version=\"")
     1705                .append(Version.getInstance().getVersion()).append("\">\n");
    17061706        SettingToXml toXml = new SettingToXml(b, nopass);
    17071707        for (Entry<String, Setting<?>> e : settingsMap.entrySet()) {

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

    r8126 r8390  
    4141            try (BufferedReader in = Utils.openURLReader(resource)) {
    4242                for (String line = in.readLine(); line != null; line = in.readLine()) {
    43                     sb.append(line).append("\n");
     43                    sb.append(line).append('\n');
    4444                }
    4545            }
    … …  
    144144        StringBuilder sb = new StringBuilder();
    145145        for(Entry<String,String> property: properties.entrySet()) {
    146             sb.append(property.getKey()).append(":").append(property.getValue()).append("\n");
     146            sb.append(property.getKey()).append(':').append(property.getValue()).append('\n');
    147147        }
    148148        releaseDescription = sb.toString();

  • trunk/src/org/openstreetmap/josm/data/imagery/ImageryInfo.java

    r8384 r8390  
    262262                for (Shape s : i.bounds.getShapes()) {
    263263                    if (shapesString.length() > 0) {
    264                         shapesString.append(";");
     264                        shapesString.append(';');
    265265                    }
    266266                    shapesString.append(s.encodeAsString(","));
    … …  
    274274                for (String p : i.serverProjections) {
    275275                    if (val.length() > 0) {
    276                         val.append(",");
     276                        val.append(',');
    277277                    }
    278278                    val.append(p);

  • trunk/src/org/openstreetmap/josm/data/osm/AbstractPrimitive.java

    r8291 r8390  
    435435
    436436        if (isIncomplete()) {
    437             builder.append("I");
     437            builder.append('I');
    438438        }
    439439        if (isModified()) {
    440             builder.append("M");
     440            builder.append('M');
    441441        }
    442442        if (isVisible()) {
    443             builder.append("V");
     443            builder.append('V');
    444444        }
    445445        if (isDeleted()) {
    446             builder.append("D");
     446            builder.append('D');
    447447        }
    448448        return builder.toString();

  • trunk/src/org/openstreetmap/josm/data/osm/OsmPrimitive.java

    r8370 r8390  
    12851285        if (isDisabled()) {
    12861286            if (isDisabledAndHidden()) {
    1287                 builder.append("h");
     1287                builder.append('h');
    12881288            } else {
    1289                 builder.append("d");
     1289                builder.append('d');
    12901290            }
    12911291        }
    12921292        if (isTagged()) {
    1293             builder.append("T");
     1293            builder.append('T');
    12941294        }
    12951295        if (hasDirectionKeys()) {
    12961296            if (reversedDirection()) {
    1297                 builder.append("<");
     1297                builder.append('<');
    12981298            } else {
    1299                 builder.append(">");
     1299                builder.append('>');
    13001300            }
    13011301        }

  • trunk/src/org/openstreetmap/josm/data/osm/OsmUtils.java

    r8285 r8390  
    22package org.openstreetmap.josm.data.osm;
    33
    4 import java.util.ArrayList;
    54import java.util.Arrays;
    6 import java.util.List;
     5import java.util.HashSet;
    76import java.util.Locale;
    87import java.util.Map;
     8import java.util.Set;
    99
    1010import org.openstreetmap.josm.data.coor.LatLon;
    … …  
    1414public final class OsmUtils {
    1515
    16     private OsmUtils() {
    17         // Hide default constructor for utils classes
    18     }
    19 
    20     private static final List<String> TRUE_VALUES = new ArrayList<>(Arrays
    21             .asList(new String[] { "true", "yes", "1", "on" }));
    22     private static final List<String> FALSE_VALUES = new ArrayList<>(Arrays
    23             .asList(new String[] { "false", "no", "0", "off" }));
    24     private static final List<String> REVERSE_VALUES = new ArrayList<>(Arrays
    25             .asList(new String[] { "reverse", "-1" }));
     16    private static final Set<String> TRUE_VALUES = new HashSet<>(Arrays
     17            .asList("true", "yes", "1", "on"));
     18    private static final Set<String> FALSE_VALUES = new HashSet<>(Arrays
     19            .asList("false", "no", "0", "off"));
     20    private static final Set<String> REVERSE_VALUES = new HashSet<>(Arrays
     21            .asList("reverse", "-1"));
    2622
    2723    public static final String trueval = "yes";
    2824    public static final String falseval = "no";
    2925    public static final String reverseval = "-1";
     26
     27    private OsmUtils() {
     28        // Hide default constructor for utils classes
     29    }
    3030
    3131    public static Boolean getOsmBoolean(String value) {

  • trunk/src/org/openstreetmap/josm/data/osm/Relation.java

    r8379 r8390  
    274274              .append(" version=")
    275275              .append(getVersion())
    276               .append(" ")
     276              .append(' ')
    277277              .append(getFlagsAsString())
    278278              .append(" [");
    279279        for (RelationMember rm:getMembers()) {
    280280            result.append(OsmPrimitiveType.from(rm.getMember()))
    281                   .append(" ")
     281                  .append(' ')
    282282                  .append(rm.getMember().getUniqueId())
    283283                  .append(", ");
    284284        }
    285285        result.delete(result.length()-2, result.length())
    286               .append("]")
    287               .append("}");
     286              .append("]}");
    288287        return result.toString();
    289288    }

  • trunk/src/org/openstreetmap/josm/data/osm/TigerUtils.java

    r8338 r8390  
    6060        for (Object part : resultSet) {
    6161            if (combined.length() > 0) {
    62                 combined.append(":");
     62                combined.append(':');
    6363            }
    6464            combined.append(part);

  • trunk/src/org/openstreetmap/josm/data/osm/history/History.java

    r8291 r8390  
    341341            result.append(", versions=\n");
    342342            for (HistoryOsmPrimitive v : versions) {
    343                 result.append("\t").append(v).append(",\n");
     343                result.append('\t').append(v).append(",\n");
    344344            }
    345345        }
    346         result.append("]");
     346        result.append(']');
    347347        return result.toString();
    348348    }

  • trunk/src/org/openstreetmap/josm/data/preferences/IntegerProperty.java

    r5464 r8390  
    66/**
    77 * A property containing an {@code Integer} value.
     8 * @since 3246
    89 */
    910public class IntegerProperty extends AbstractProperty<Integer> {
    … …  
    3536     */
    3637    public boolean parseAndPut(String value) {
    37         Integer intVal;
    3838        try {
    39             intVal = Integer.parseInt(value);
     39            return put(Integer.valueOf(value));
    4040        } catch (NumberFormatException ex) {
    4141            return false;
    4242        }
    43         return put(intVal);
    4443    }
    4544}

  • trunk/src/org/openstreetmap/josm/data/projection/CustomProjection.java

    r8384 r8390  
    335335        for (String str : numStr) {
    336336            try {
    337                 towgs84Param.add(Double.parseDouble(str));
     337                towgs84Param.add(Double.valueOf(str));
    338338            } catch (NumberFormatException e) {
    339339                throw new ProjectionConfigurationException(tr("Unable to parse value of parameter ''towgs84'' (''{0}'')", str), e);
    … …  
    496496        if (code != null && code.startsWith("EPSG:")) {
    497497            try {
    498                 return Integer.parseInt(code.substring(5));
     498                return Integer.valueOf(code.substring(5));
    499499            } catch (NumberFormatException e) {
    500500                Main.warn(e);

  • trunk/src/org/openstreetmap/josm/data/validation/TestError.java

    r8380 r8390  
    187187        }
    188188        for (String o : strings) {
    189             ignorestring.append(":").append(o);
     189            ignorestring.append(':').append(o);
    190190        }
    191191        return ignorestring.toString();

  • trunk/src/org/openstreetmap/josm/data/validation/routines/RegexValidator.java

    r8345 r8390  
    182182                    return matcher.group(1);
    183183                }
    184                 StringBuffer buffer = new StringBuffer();
     184                StringBuilder buffer = new StringBuilder();
    185185                for (int j = 0; j < count; j++) {
    186186                    String component = matcher.group(j+1);
    … …  
    201201    @Override
    202202    public String toString() {
    203         StringBuffer buffer = new StringBuffer();
     203        StringBuilder buffer = new StringBuilder();
    204204        buffer.append("RegexValidator{");
    205205        for (int i = 0; i < patterns.length; i++) {
    206206            if (i > 0) {
    207                 buffer.append(",");
     207                buffer.append(',');
    208208            }
    209209            buffer.append(patterns[i].pattern());
    210210        }
    211         buffer.append("}");
     211        buffer.append('}');
    212212        return buffer.toString();
    213213    }
    214 
    215214}

  • trunk/src/org/openstreetmap/josm/data/validation/tests/Highways.java

    r8382 r8390  
    4343     * Classified highways in order of importance
    4444     */
    45     protected static final List<String> CLASSIFIED_HIGHWAYS = Arrays.asList(
     45    private static final Set<String> CLASSIFIED_HIGHWAYS = new HashSet<>(Arrays.asList(
    4646            "motorway",  "motorway_link",
    4747            "trunk",     "trunk_link",
    … …  
    5151            "unclassified",
    5252            "residential",
    53             "living_street");
    54 
    55     protected static final List<String> KNOWN_SOURCE_MAXSPEED_CONTEXTS = Arrays.asList(
    56             "urban", "rural", "zone", "zone30", "zone:30", "nsl_single", "nsl_dual", "motorway", "trunk", "living_street", "bicycle_road");
    57 
    58     protected static final List<String> ISO_COUNTRIES = Arrays.asList(Locale.getISOCountries());
     53            "living_street"));
     54
     55    private static final Set<String> KNOWN_SOURCE_MAXSPEED_CONTEXTS = new HashSet<>(Arrays.asList(
     56            "urban", "rural", "zone", "zone30", "zone:30", "nsl_single", "nsl_dual", "motorway", "trunk", "living_street", "bicycle_road"));
     57
     58    private static final Set<String> ISO_COUNTRIES = new HashSet<>(Arrays.asList(Locale.getISOCountries()));
    5959
    6060    private boolean leftByPedestrians = false;

  • trunk/src/org/openstreetmap/josm/data/validation/tests/MapCSSTagChecker.java

    r8373 r8390  
    246246
    247247        static final String possibleThrows() {
    248             StringBuffer sb = new StringBuffer();
     248            StringBuilder sb = new StringBuilder();
    249249            for (Severity s : Severity.values()) {
    250250                if (sb.length() > 0) {

  • trunk/src/org/openstreetmap/josm/data/validation/tests/SimilarNamedWays.java

    r8378 r8390  
    261261                }
    262262                if (expression.length() > 0) {
    263                     expression.append("|");
     263                    expression.append('|');
    264264                }
    265265                expression.append(Pattern.quote(words[i]));

  • trunk/src/org/openstreetmap/josm/data/validation/tests/UnclosedWays.java

    r8378 r8390  
    99import java.util.Collections;
    1010import java.util.HashSet;
    11 import java.util.List;
    1211import java.util.Set;
    1312
    … …  
    4746        private final String engMessage;
    4847        /** The special values, to be ignored if ignore is set to true; to be considered only if ignore is set to false */
    49         private final List<String> specialValues;
     48        private final Set<String> specialValues;
    5049        /** The boolean indicating if special values must be ignored or considered only */
    5150        private final boolean ignore;
    … …  
    5857         */
    5958        public UnclosedWaysCheck(int code, String key, String engMessage) {
    60             this(code, key, engMessage, Collections.<String>emptyList());
     59            this(code, key, engMessage, Collections.<String>emptySet());
    6160        }
    6261
    … …  
    6867         * @param ignoredValues The ignored values.
    6968         */
    70         public UnclosedWaysCheck(int code, String key, String engMessage, List<String> ignoredValues) {
     69        public UnclosedWaysCheck(int code, String key, String engMessage, Set<String> ignoredValues) {
    7170            this(code, key, engMessage, ignoredValues, true);
    7271        }
    … …  
    8079         * @param ignore indicates if special values must be ignored or considered only
    8180         */
    82         public UnclosedWaysCheck(int code, String key, String engMessage, List<String> specialValues, boolean ignore) {
     81        public UnclosedWaysCheck(int code, String key, String engMessage, Set<String> specialValues, boolean ignore) {
    8382            this.code = code;
    8483            this.key = key;
    … …  
    137136
    138137    private final UnclosedWaysCheck[] checks = {
    139         new UnclosedWaysCheck(1101, "natural",   marktr("natural type {0}"), Arrays.asList("coastline", "cliff", "tree_row", "ridge", "arete", "gorge")),
     138        new UnclosedWaysCheck(1101, "natural",   marktr("natural type {0}"),
     139                new HashSet<>(Arrays.asList("coastline", "cliff", "tree_row", "ridge", "arete", "gorge"))),
    140140        new UnclosedWaysCheck(1102, "landuse",   marktr("landuse type {0}")),
    141141        new UnclosedWaysCheck(1103, "amenities", marktr("amenities type {0}")),
    142         new UnclosedWaysCheck(1104, "sport",     marktr("sport type {0}"), Arrays.asList("water_slide", "climbing")),
    143         new UnclosedWaysCheck(1105, "tourism",   marktr("tourism type {0}"), Arrays.asList("attraction", "artwork")),
     142        new UnclosedWaysCheck(1104, "sport",     marktr("sport type {0}"),
     143                new HashSet<>(Arrays.asList("water_slide", "climbing"))),
     144        new UnclosedWaysCheck(1105, "tourism",   marktr("tourism type {0}"),
     145                new HashSet<>(Arrays.asList("attraction", "artwork"))),
    144146        new UnclosedWaysCheck(1106, "shop",      marktr("shop type {0}")),
    145         new UnclosedWaysCheck(1107, "leisure",   marktr("leisure type {0}"), Arrays.asList("track", "slipway")),
    146         new UnclosedWaysCheck(1108, "waterway",  marktr("waterway type {0}"), Arrays.asList("riverbank"), false),
     147        new UnclosedWaysCheck(1107, "leisure",   marktr("leisure type {0}"),
     148                new HashSet<>(Arrays.asList("track", "slipway"))),
     149        new UnclosedWaysCheck(1108, "waterway",  marktr("waterway type {0}"),
     150                new HashSet<>(Arrays.asList("riverbank")), false),
    147151        new UnclosedWaysCheck(1109, "boundary", marktr("boundary type {0}")),
    148152        new UnclosedWaysBooleanCheck(1120, "building", marktr("building")),

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

    r8379 r8390  
    183183            }
    184184            if (node.getCoor() != null) {
    185                 name.append(" \u200E(").append(node.getCoor().latToString(CoordinateFormat.getDefaultFormat())).append(", ").append(node.getCoor().lonToString(CoordinateFormat.getDefaultFormat())).append(")");
     185                name.append(" \u200E(").append(node.getCoor().latToString(CoordinateFormat.getDefaultFormat())).append(", ")
     186                    .append(node.getCoor().lonToString(CoordinateFormat.getDefaultFormat())).append(')');
    186187            }
    187188        }
    … …  
    287288            /* I18n: count of nodes as parameter */
    288289            String nodes = trn("{0} node", "{0} nodes", nodesNo, nodesNo);
    289             name.append(mark).append(" (").append(nodes).append(")");
     290            name.append(mark).append(" (").append(nodes).append(')');
    290291        }
    291292        decorateNameWithId(name, way);
    … …  
    337338            }
    338339
    339             name.append(")");
     340            name.append(')');
    340341        }
    341342        decorateNameWithId(name, relation);
    … …  
    363364        } else {
    364365            preset.nameTemplate.appendText(result, relation);
    365             result.append("(");
     366            result.append('(');
    366367        }
    367368    }
    … …  
    512513    private String buildDefaultToolTip(long id, Map<String, String> tags) {
    513514        StringBuilder sb = new StringBuilder();
    514         sb.append("<html>")
    515         .append("<strong>id</strong>=")
    516         .append(id)
    517         .append("<br>");
     515        sb.append("<html><strong>id</strong>=")
     516          .append(id)
     517          .append("<br>");
    518518        List<String> keyList = new ArrayList<>(tags.keySet());
    519519        Collections.sort(keyList);
    … …  
    524524            String key = keyList.get(i);
    525525            sb.append("<strong>")
    526             .append(key)
    527             .append("</strong>")
    528             .append("=");
     526              .append(key)
     527              .append("</strong>=");
    529528            String value = tags.get(key);
    530529            while(value.length() != 0) {
    … …  
    583582            .append(", ")
    584583            .append(coord.lonToString(CoordinateFormat.getDefaultFormat()))
    585             .append(")");
     584            .append(')');
    586585        }
    587586        decorateNameWithId(sb, node);
    … …  
    671670
    672671        int mbno = relation.getNumMembers();
    673         sb.append(trn("{0} member", "{0} members", mbno, mbno)).append(")");
     672        sb.append(trn("{0} member", "{0} members", mbno, mbno)).append(')');
    674673
    675674        decorateNameWithId(sb, relation);

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

    r8308 r8390  
    1515import java.util.Arrays;
    1616import java.util.Collections;
     17import java.util.HashSet;
    1718import java.util.List;
     19import java.util.Set;
    1820
    1921import javax.swing.AbstractAction;
    … …  
    8688    private String[] bToolTipTexts;
    8789    private transient Icon[] bIcons;
    88     private transient List<Integer> cancelButtonIdx = Collections.emptyList();
     90    private transient Set<Integer> cancelButtonIdx = Collections.emptySet();
    8991    private int defaultButtonIdx = 1;
    9092    protected JButton defaultButton = null;
    … …  
    561563     */
    562564    public ExtendedDialog setCancelButton(Integer... cancelButtonIdx) {
    563         this.cancelButtonIdx = Arrays.<Integer>asList(cancelButtonIdx);
     565        this.cancelButtonIdx = new HashSet<>(Arrays.<Integer>asList(cancelButtonIdx));
    564566        return this;
    565567    }

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

    r8378 r8390  
    572572         */
    573573        private JLabel popupBuildPrimitiveLabels(final OsmPrimitive osm) {
    574             final StringBuilder text = new StringBuilder();
     574            final StringBuilder text = new StringBuilder(32);
    575575            String name = osm.getDisplayName(DefaultNameFormatter.getInstance());
    576576            if (osm.isNewOrUndeleted() || osm.isModified()) {
    … …  
    583583
    584584            if (!osm.isNew() && !idShown) {
    585                 text.append(" [id="+osm.getId()+"]");
     585                text.append(" [id=").append(osm.getId()).append(']');
    586586            }
    587587
    588588            if(osm.getUser() != null) {
    589                 text.append(" [" + tr("User:") + " " + osm.getUser().getName() + "]");
     589                text.append(" [").append(tr("User:")).append(' ').append(osm.getUser().getName()).append(']');
    590590            }
    591591
    592592            for (String key : osm.keySet()) {
    593                 text.append("<br>" + key + "=" + osm.get(key));
     593                text.append("<br>").append(key).append('=').append(osm.get(key));
    594594            }
    595595
    596596            final JLabel l = new JLabel(
    597                     "<html>" +text.toString() + "</html>",
     597                    "<html>" + text.toString() + "</html>",
    598598                    ImageProvider.get(osm.getDisplayType()),
    599599                    JLabel.HORIZONTAL
    … …  
    601601                // This is necessary so the label updates its colors when the
    602602                // selection is changed from the outside
    603                 @Override public void validate() {
     603                @Override
     604                public void validate() {
    604605                    super.validate();
    605606                    popupSetLabelColors(this, osm);
    … …  
    613614            l.setCursor(Cursor.getPredefinedCursor(Cursor.HAND_CURSOR));
    614615            l.addMouseListener(new MouseAdapter(){
    615                 @Override public void mouseEntered(MouseEvent e) {
     616                @Override
     617                public void mouseEntered(MouseEvent e) {
    616618                    l.setBackground(SystemColor.info);
    617619                    l.setForeground(SystemColor.infoText);
    618620                }
    619                 @Override public void mouseExited(MouseEvent e) {
     621                @Override
     622                public void mouseExited(MouseEvent e) {
    620623                    popupSetLabelColors(l, osm);
    621624                }
    622                 @Override public void mouseClicked(MouseEvent e) {
     625                @Override
     626                public void mouseClicked(MouseEvent e) {
    623627                    DataSet ds = Main.main.getCurrentDataSet();
    624628                    // Let the user toggle the selection

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

    r8378 r8390  
    2626import java.util.Collections;
    2727import java.util.Comparator;
    28 import java.util.LinkedList;
     28import java.util.LinkedHashSet;
    2929import java.util.List;
     30import java.util.Set;
    3031import java.util.concurrent.CopyOnWriteArrayList;
    3132
    … …  
    224225    public MouseEvent lastMEvent = new MouseEvent(this, 0, 0, 0, 0, 0, 0, false); // In case somebody reads it before first mouse move
    225226
    226     private final transient List<MapViewPaintable> temporaryLayers = new LinkedList<>();
     227    private final transient Set<MapViewPaintable> temporaryLayers = new LinkedHashSet<>();
    227228
    228229    private transient BufferedImage nonChangedLayersBuffer;

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

    r8379 r8390  
    651651        @Override
    652652        public String toString() {
    653             StringBuilder sb = new StringBuilder();
    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);
     653            StringBuilder sb = new StringBuilder(24);
     654            sb.append("min=").append(min.x).append(',').append(min.y)
     655              .append(",max=").append(max.x).append(',').append(max.y)
     656              .append(",zoom=").append(zoomLevel);
    657657            return sb.toString();
    658658        }

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

    r8388 r8390  
    1414import java.util.ArrayList;
    1515import java.util.EnumMap;
     16import java.util.HashSet;
    1617import java.util.List;
    1718import java.util.Map;
    1819import java.util.Observable;
     20import java.util.Set;
    1921
    2022import javax.swing.AbstractListModel;
    … …  
    8587    protected EntriesSelectionModel mergedEntriesSelectionModel;
    8688
    87     private final List<PropertyChangeListener> listeners;
     89    private final Set<PropertyChangeListener> listeners;
    8890    private boolean isFrozen = false;
    8991    private final ComparePairListModel comparePairListModel;
    … …  
    199201        mergedEntriesSelectionModel =  new EntriesSelectionModel(entries.get(MERGED_ENTRIES));
    200202
    201         listeners = new ArrayList<>();
     203        listeners = new HashSet<>();
    202204        comparePairListModel = new ComparePairListModel();
    203205
    … …  
    207209    public void addPropertyChangeListener(PropertyChangeListener listener) {
    208210        synchronized(listeners) {
    209             if (listener != null && ! listeners.contains(listener)) {
     211            if (listener != null && !listeners.contains(listener)) {
    210212                listeners.add(listener);
    211213            }

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

    r8308 r8390  
    290290            return tr("(none)");
    291291        StringBuilder sb = new StringBuilder();
    292         sb.append("(")
     292        sb.append('(')
    293293        .append(COORD_FORMATTER.format(coord.lat()))
    294         .append(",")
     294        .append(',')
    295295        .append(COORD_FORMATTER.format(coord.lon()))
    296         .append(")");
     296        .append(')');
    297297        return sb.toString();
    298298    }

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

    r8308 r8390  
    3636
    3737    /** the property change listeners */
    38     private final transient List<PropertyChangeListener> listeners;
     38    private final transient Set<PropertyChangeListener> listeners;
    3939
    4040    private int numUndecidedTags = 0;
    … …  
    4545    public TagMergeModel() {
    4646        tagMergeItems = new ArrayList<>();
    47         listeners = new ArrayList<>();
     47        listeners = new HashSet<>();
    4848    }
    4949

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

    r8308 r8390  
    1313import java.awt.event.MouseAdapter;
    1414import java.awt.event.MouseEvent;
    15 import java.util.ArrayList;
    16 import java.util.List;
     15import java.util.HashSet;
     16import java.util.Set;
    1717
    1818import javax.swing.AbstractAction;
    … …  
    331331     */
    332332    static class AdjustmentSynchronizer implements AdjustmentListener {
    333         private final List<Adjustable> synchronizedAdjustables;
     333        private final Set<Adjustable> synchronizedAdjustables;
    334334
    335335        public AdjustmentSynchronizer() {
    336             synchronizedAdjustables = new ArrayList<>();
     336            synchronizedAdjustables = new HashSet<>();
    337337        }
    338338

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

    r8366 r8390  
    4646    private EastNorth eastNorthCoordinates;
    4747
    48     private static final double ZERO = 0.0;
     48    private static final Double ZERO = 0.0;
    4949    private static final String DEG = "\u00B0";
    5050    private static final String MIN = "\u2032";
    … …  
    349349        if (mXml.matches()) {
    350350            setLatLonObj(latLon,
    351                     Double.parseDouble(mXml.group(1).replace(',', '.')), ZERO, ZERO, "N",
    352                     Double.parseDouble(mXml.group(2).replace(',', '.')), ZERO, ZERO, "E");
     351                    Double.valueOf(mXml.group(1).replace(',', '.')), ZERO, ZERO, "N",
     352                    Double.valueOf(mXml.group(2).replace(',', '.')), ZERO, ZERO, "E");
    353353        } else {
    354354            final Matcher m = P.matcher(coord);
    … …  
    360360                if (m.group(1) != null) {
    361361                    sb.append('R');     // floating point number
    362                     list.add(Double.parseDouble(m.group(1).replace(',', '.')));
     362                    list.add(Double.valueOf(m.group(1).replace(',', '.')));
    363363                } else if (m.group(2) != null) {
    364364                    sb.append('Z');     // integer number
    365                     list.add(Double.parseDouble(m.group(2)));
     365                    list.add(Double.valueOf(m.group(2)));
    366366                } else if (m.group(3) != null) {
    367367                    sb.append('o');     // degree sign
    … …  
    373373                    sb.append(',');     // separator
    374374                } else if (m.group(7) != null) {
    375                     sb.append("x");     // cardinal direction
     375                    sb.append('x');     // cardinal direction
    376376                    String c = m.group(7).toUpperCase();
    377377                    if ("N".equals(c) || "S".equals(c) || "E".equals(c) || "W".equals(c)) {

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

    r7001 r8390  
    4040        } else {
    4141            String comment = cs.get("comment");
    42             sb.append(cs.getId());
    43             sb.append(" - ");
    44             sb.append(cs.isOpen() ? tr("open") : tr("closed"));
     42            sb.append(cs.getId())
     43              .append(" - ")
     44              .append(cs.isOpen() ? tr("open") : tr("closed"));
    4545            if (comment != null) {
    46                 sb.append(" - ").append("'").append(comment).append("'");
     46                sb.append(" - '").append(comment).append('\'');
    4747            }
    4848        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesCellRenderer.java

    r8387 r8390  
    7979                        sb.append(trn(", {0} unset", ", {0} unset", blankCount, blankCount));
    8080                    }
    81                     sb.append(">");
     81                    sb.append('>');
    8282                    str = sb.toString();
    8383                    c.setFont(c.getFont().deriveFont(Font.ITALIC));

  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesDialog.java

    r8345 r8390  
    12821282                    t = "type:relation ";
    12831283                }
    1284                 s.append(sep).append("(").append(t).append("\"").append(
     1284                s.append(sep).append('(').append(t).append('"').append(
    12851285                        org.openstreetmap.josm.actions.search.SearchAction.escapeStringForSearch(key)).append("\"=\"").append(
    12861286                        org.openstreetmap.josm.actions.search.SearchAction.escapeStringForSearch(val)).append("\")");

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/RelationEditor.java

    r8308 r8390  
    88import java.lang.reflect.Constructor;
    99import java.lang.reflect.Method;
    10 import java.util.ArrayList;
    1110import java.util.Collection;
    12 import java.util.List;
     11import java.util.LinkedHashSet;
     12import java.util.Set;
    1313
    1414import org.openstreetmap.josm.Main;
    … …  
    3232
    3333    /** the list of registered relation editor classes */
    34     private static List<Class<RelationEditor>> editors = new ArrayList<>();
     34    private static Set<Class<RelationEditor>> editors = new LinkedHashSet<>();
    3535
    3636    /**

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

    r8342 r8390  
    214214        protected String buildToolTipText(Bookmark b) {
    215215            Bounds area = b.getArea();
    216             StringBuilder sb = new StringBuilder();
     216            StringBuilder sb = new StringBuilder(128);
    217217            sb.append("<html>min[latitude,longitude]=<strong>[")
    218             .append(area.getMinLat()).append(",").append(area.getMinLon()).append("]</strong>")
    219             .append("<br>")
    220             .append("max[latitude,longitude]=<strong>[")
    221             .append(area.getMaxLat()).append(",").append(area.getMaxLon()).append("]</strong>")
    222             .append("</html>");
     218              .append(area.getMinLat()).append(',').append(area.getMinLon()).append("]</strong>")
     219              .append("<br>max[latitude,longitude]=<strong>[")
     220              .append(area.getMaxLat()).append(',').append(area.getMaxLon()).append("]</strong>")
     221              .append("</html>");
    223222            return sb.toString();
    224 
    225223        }
    226224

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

    r8347 r8390  
    539539                    line.append(t);
    540540                } else if (line.length() < 80) {
    541                     line.append(" ").append(t);
     541                    line.append(' ').append(t);
    542542                } else {
    543                     line.append(" ").append(t).append("<br>");
     543                    line.append(' ').append(t).append("<br>");
    544544                    ret.append(line);
    545545                    line = new StringBuilder();

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

    r8308 r8390  
    144144            while ((line = reader.readLine()) != null) {
    145145                css.append(line);
    146                 css.append("\n");
     146                css.append('\n');
    147147            }
    148148        } catch (Exception e) {

  • trunk/src/org/openstreetmap/josm/gui/history/SelectionSynchronizer.java

    r7005 r8390  
    22package org.openstreetmap.josm.gui.history;
    33
    4 import java.util.ArrayList;
    5 import java.util.List;
     4import java.util.HashSet;
     5import java.util.Set;
    66
    77import javax.swing.DefaultListSelectionModel;
    … …  
    1212public class SelectionSynchronizer implements ListSelectionListener {
    1313
    14     private List<ListSelectionModel> participants;
     14    private final Set<ListSelectionModel> participants;
    1515
    1616    /**
    … …  
    1818     */
    1919    public SelectionSynchronizer() {
    20         participants = new ArrayList<>();
     20        participants = new HashSet<>();
    2121    }
    2222

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

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

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

    r8379 r8390  
    3232
    3333    protected String buildChangesetSummary() {
    34         StringBuilder msg = new StringBuilder();
     34        StringBuilder msg = new StringBuilder(96);
    3535        if (selectedChangeset == null || selectedChangeset.isNew()) {
    3636            msg.append(tr("Objects are uploaded to a <strong>new changeset</strong>."));
    … …  
    4343            ));
    4444        }
    45         msg.append(" ");
     45        msg.append(' ');
    4646        if (closeChangesetAfterNextUpload) {
    4747            msg.append(tr("The changeset is going to be <strong>closed</strong> after this upload"));
    … …  
    4949            msg.append(tr("The changeset is <strong>left open</strong> after this upload"));
    5050        }
    51         msg.append(" (<a href=\"urn:changeset-configuration\">" + tr("configure changeset") + "</a>)");
     51        msg.append(" (<a href=\"urn:changeset-configuration\">").append(tr("configure changeset")).append("</a>)");
    5252        return msg.toString();
    5353    }

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

    r8379 r8390  
    120120    @Override
    121121    public Object getInfoComponent() {
    122         StringBuilder info = new StringBuilder();
     122        StringBuilder info = new StringBuilder(48);
    123123
    124124        if (data.attr.containsKey("name")) {
    … …  
    131131
    132132        if (!data.tracks.isEmpty()) {
    133             info.append("<table><thead align='center'><tr><td colspan='5'>"
    134                     + trn("{0} track", "{0} tracks", data.tracks.size(), data.tracks.size())
    135                     + "</td></tr><tr align='center'><td>" + tr("Name") + "</td><td>"
    136                     + tr("Description") + "</td><td>" + tr("Timespan")
    137                     + "</td><td>" + tr("Length") + "</td><td>" + tr("URL")
    138                     + "</td></tr></thead>");
     133            info.append("<table><thead align='center'><tr><td colspan='5'>")
     134                .append(trn("{0} track", "{0} tracks", data.tracks.size(), data.tracks.size()))
     135                .append("</td></tr><tr align='center'><td>").append(tr("Name")).append("</td><td>")
     136                .append(tr("Description")).append("</td><td>").append(tr("Timespan"))
     137                .append("</td><td>").append(tr("Length")).append("</td><td>").append(tr("URL"))
     138                .append("</td></tr></thead>");
    139139
    140140            for (GpxTrack trk : data.tracks) {
    … …  
    145145                info.append("</td><td>");
    146146                if (trk.getAttributes().containsKey(GpxConstants.GPX_DESC)) {
    147                     info.append(" ").append(trk.get(GpxConstants.GPX_DESC));
     147                    info.append(' ').append(trk.get(GpxConstants.GPX_DESC));
    148148                }
    149149                info.append("</td><td>");
    … …  
    209209    @Override
    210210    public String getToolTipText() {
    211         StringBuilder info = new StringBuilder().append("<html>");
     211        StringBuilder info = new StringBuilder(48).append("<html>");
    212212
    213213        if (data.attr.containsKey(GpxConstants.META_NAME)) {
    … …  
    223223            .append(trn("{0} waypoint", "{0} waypoints", data.waypoints.size(), data.waypoints.size())).append("<br>")
    224224            .append(tr("Length: {0}", NavigatableComponent.getSystemOfMeasurement().getDistText(data.length())))
    225             .append("<br>")
    226             .append("</html>");
     225            .append("<br></html>");
    227226        return info.toString();
    228227    }

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

    r8379 r8390  
    113113            StringBuilder sb = new StringBuilder("<html>");
    114114            sb.append(tr("Note"))
    115               .append(" ").append(noteData.getSelectedNote().getId());
     115              .append(' ').append(noteData.getSelectedNote().getId());
    116116            for (NoteComment comment : noteData.getSelectedNote().getComments()) {
    117117                String commentText = comment.getText();
    … …  
    196196        StringBuilder sb = new StringBuilder();
    197197        sb.append(tr("Notes layer"))
    198           .append("\n")
     198          .append('\n')
    199199          .append(tr("Total notes:"))
    200           .append(" ")
     200          .append(' ')
    201201          .append(noteData.getNotes().size())
    202           .append("\n")
     202          .append('\n')
    203203          .append(tr("Changes need uploading?"))
    204           .append(" ")
     204          .append(' ')
    205205          .append(isModified());
    206206        return sb.toString();

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

    r8384 r8390  
    524524                    ed.setIcon(JOptionPane.INFORMATION_MESSAGE);
    525525                    StringBuilder content = new StringBuilder();
    526                     content.append("Tile name: ").append(clickedTile.getKey()).append("\n");
     526                    content.append("Tile name: ").append(clickedTile.getKey()).append('\n');
    527527                    try {
    528                         content.append("Tile url: ").append(clickedTile.getUrl()).append("\n");
     528                        content.append("Tile url: ").append(clickedTile.getUrl()).append('\n');
    529529                    } catch (IOException e) {
    530530                    }
    531                     content.append("Tile size: ").append(clickedTile.getTileSource().getTileSize()).append("x").append(clickedTile.getTileSource().getTileSize()).append("\n");
     531                    content.append("Tile size: ").append(clickedTile.getTileSource().getTileSize()).append('x').append(clickedTile.getTileSource().getTileSize()).append('\n');
    532532                    Rectangle displaySize = tileToRect(clickedTile);
    533                     content.append("Tile display size: ").append(displaySize.width).append("x").append(displaySize.height).append("\n");
     533                    content.append("Tile display size: ").append(displaySize.width).append('x').append(displaySize.height).append('\n');
    534534                    ed.setContent(content.toString());
    535535                    ed.showDialog();

  • trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java

    r8354 r8390  
    120120            if (exts != null && exts.containsKey("offset")) {
    121121                try {
    122                     offset = Double.parseDouble(exts.get("offset"));
     122                    offset = Double.valueOf(exts.get("offset"));
    123123                } catch (NumberFormatException nfe) {
    124124                    Main.warn(nfe);

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

    r8384 r8390  
    127127        if (o instanceof String && !((String) o).isEmpty()) {
    128128            try {
    129                 return Float.parseFloat((String) o);
     129                return Float.valueOf((String) o);
    130130            } catch (NumberFormatException e) {
    131                 Main.debug("'"+o+"' cannot be converted to float");
     131                if (Main.isDebugEnabled()) {
     132                    Main.debug("'"+o+"' cannot be converted to float");
     133                }
    132134            }
    133135        }
    … …  
    217219            res.append("; ");
    218220        }
    219         return res.append("}").toString();
     221        return res.append('}').toString();
    220222    }
    221223

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

    r7509 r8390  
    44import java.awt.event.ActionEvent;
    55import java.util.Arrays;
     6
    67import javax.swing.AbstractAction;
    78import javax.swing.Action;
    89import javax.swing.JCheckBoxMenuItem;
    910import javax.swing.JMenu;
     11
    1012import org.openstreetmap.josm.Main;
    1113
    … …  
    9193            String val = Main.pref.get(prefKey, null);
    9294            if (val == null) return def;
    93             return Boolean.parseBoolean(val);
     95            return Boolean.valueOf(val);
    9496        }
    95 
    9697    }
    9798}

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

    r8379 r8390  
    177177
    178178    protected String toStringImpl() {
    179         StringBuilder sb = new StringBuilder();
    180         sb.append("labelCompositionStrategy=" + labelCompositionStrategy)
    181           .append(" font=" + font);
     179        StringBuilder sb = new StringBuilder(96);
     180        sb.append("labelCompositionStrategy=").append(labelCompositionStrategy)
     181          .append(" font=").append(font);
    182182        if (xOffset != 0) {
    183             sb.append(" xOffset=" + xOffset);
     183            sb.append(" xOffset=").append(xOffset);
    184184        }
    185185        if (yOffset != 0) {
    186             sb.append(" yOffset=" + yOffset);
    187         }
    188         sb.append(" color=" + Utils.toString(color));
     186            sb.append(" yOffset=").append(yOffset);
     187        }
     188        sb.append(" color=").append(Utils.toString(color));
    189189        if (haloRadius != null) {
    190             sb.append(" haloRadius=" + haloRadius)
    191               .append(" haloColor=" + haloColor);
     190            sb.append(" haloRadius=").append(haloRadius)
     191              .append(" haloColor=").append(haloColor);
    192192        }
    193193        return sb.toString();

  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/ExpressionFactory.java

    r8384 r8390  
    11401140        public String toString() {
    11411141            StringBuilder b = new StringBuilder("ParameterFunction~");
    1142             b.append(m.getName()).append("(");
     1142            b.append(m.getName()).append('(');
    11431143            for (int i = 0; i < args.size(); ++i) {
    1144                 if (i > 0) b.append(",");
    1145                 b.append(expectedParameterTypes[i]);
    1146                 b.append(" ").append(args.get(i));
     1144                if (i > 0) b.append(',');
     1145                b.append(expectedParameterTypes[i]).append(' ').append(args.get(i));
    11471146            }
    11481147            b.append(')');
    … …  
    12051204        public String toString() {
    12061205            StringBuilder b = new StringBuilder("ArrayFunction~");
    1207             b.append(m.getName()).append("(");
     1206            b.append(m.getName()).append('(');
    12081207            for (int i = 0; i < args.size(); ++i) {
    1209                 if (i > 0) b.append(",");
    1210                 b.append(arrayComponentType);
    1211                 b.append(" ").append(args.get(i));
     1208                if (i > 0) b.append(',');
     1209                b.append(arrayComponentType).append(' ').append(args.get(i));
    12121210            }
    12131211            b.append(')');

  • trunk/src/org/openstreetmap/josm/gui/mappaint/xml/XmlCondition.java

    r7801 r8390  
    4646            sb.append(",v=").append(value);
    4747
    48         sb.append("]");
     48        sb.append(']');
    4949    }
    5050}

  • trunk/src/org/openstreetmap/josm/gui/mappaint/xml/XmlStyleSourceHandler.java

    r8285 r8390  
    8181            case "width":
    8282                String val = atts.getValue(count);
    83                 if (! (val.startsWith("+") || val.startsWith("-") || val.endsWith("%"))) {
     83                if (!(val.startsWith("+") || val.startsWith("-") || val.endsWith("%"))) {
    8484                    line.setWidth(Integer.parseInt(val));
    8585                }
    … …  
    8989                break;
    9090            case "realwidth":
    91                 line.realWidth = Integer.parseInt(atts.getValue(count));
     91                line.realWidth = Integer.valueOf(atts.getValue(count));
    9292                break;
    9393            case "dashed":
    … …  
    224224                        break;
    225225                    case "annotate":
    226                         rule.icon.annotate = Boolean.parseBoolean (atts.getValue(count));
     226                        rule.icon.annotate = Boolean.valueOf(atts.getValue(count));
    227227                        break;
    228228                    case "priority":

  • trunk/src/org/openstreetmap/josm/gui/oauth/OsmOAuthAuthorizationClient.java

    r8304 r8390  
    186186     */
    187187    public String getAuthoriseUrl(OAuthToken requestToken) {
    188         StringBuilder sb = new StringBuilder();
     188        StringBuilder sb = new StringBuilder(32);
    189189
    190190        // OSM is an OAuth 1.0 provider and JOSM isn't a web app. We just add the oauth request token to
    191191        // the authorisation request, no callback parameter.
    192192        //
    193         sb.append(oauthProviderParameters.getAuthoriseUrl()).append("?")
    194         .append(OAuth.OAUTH_TOKEN).append("=").append(requestToken.getKey());
     193        sb.append(oauthProviderParameters.getAuthoriseUrl()).append('?'+OAuth.OAUTH_TOKEN+'=').append(requestToken.getKey());
    195194        return sb.toString();
    196195    }
    … …  
    249248
    250249    protected String buildPostRequest(Map<String,String> parameters) throws OsmOAuthAuthorizationException {
    251         StringBuilder sb = new StringBuilder();
     250        StringBuilder sb = new StringBuilder(32);
    252251
    253252        for(Iterator<Entry<String,String>> it = parameters.entrySet().iterator(); it.hasNext();) {
    … …  
    255254            String value = entry.getValue();
    256255            value = (value == null) ? "" : value;
    257             sb.append(entry.getKey()).append("=").append(Utils.encodeUrl(value));
     256            sb.append(entry.getKey()).append('=').append(Utils.encodeUrl(value));
    258257            if (it.hasNext()) {
    259                 sb.append("&");
     258                sb.append('&');
    260259            }
    261260        }

  • trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java

    r8377 r8390  
    13331333                            } else if ("min-josm-version".equals(key)) {
    13341334                                try {
    1335                                     last.minJosmVersion = Integer.parseInt(value);
     1335                                    last.minJosmVersion = Integer.valueOf(value);
    13361336                                } catch (NumberFormatException e) {
    13371337                                    // ignore

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

    r8379 r8390  
    6969        String z = sanitize(tmsZoom.getText());
    7070        if (!z.isEmpty()) {
    71             a.append("[").append(z).append("]");
     71            a.append('[').append(z).append(']');
    7272        }
    73         a.append(":").append(getImageryRawUrl());
     73        a.append(':').append(getImageryRawUrl());
    7474        return a.toString();
    7575    }

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

    r8379 r8390  
    6363            sb.append(pi.version);
    6464            if (pi.oldmode) {
    65                 sb.append("*");
     65                sb.append('*');
    6666            }
    6767        }

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

    r8387 r8390  
    8888        Collection<PluginInformation> failed = task.getFailedPlugins();
    8989        StringBuilder sb = new StringBuilder();
    90         if (! downloaded.isEmpty()) {
     90        if (!downloaded.isEmpty()) {
    9191            sb.append(trn(
    9292                    "The following plugin has been downloaded <strong>successfully</strong>:",
    … …  
    9797            sb.append("<ul>");
    9898            for(PluginInformation pi: downloaded) {
    99                 sb.append("<li>").append(pi.name).append(" (").append(pi.version).append(")").append("</li>");
     99                sb.append("<li>").append(pi.name).append(" (").append(pi.version).append(")</li>");
    100100            }
    101101            sb.append("</ul>");
    102102        }
    103         if (! failed.isEmpty()) {
     103        if (!failed.isEmpty()) {
    104104            sb.append(trn(
    105105                    "Downloading the following plugin has <strong>failed</strong>:",

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

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

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

    r7005 r8390  
    117117                sb.append(values.get(i));
    118118                if (i + 1 < values.size()) {
    119                     sb.append(";");
     119                    sb.append(';');
    120120                }
    121121            }

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

    r8379 r8390  
    15501550            return null;
    15511551        try {
    1552             return Integer.parseInt(str);
     1552            return Integer.valueOf(str);
    15531553        } catch (Exception e) {
    15541554            if (Main.isTraceEnabled()) {

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

    r8379 r8390  
    9292          .append("',")
    9393          .append(priority)
    94           .append(">");
     94          .append('>');
    9595        return sb.toString();
    9696    }

  • trunk/src/org/openstreetmap/josm/gui/util/AdjustmentSynchronizer.java

    r8291 r8390  
    99import java.awt.event.ItemEvent;
    1010import java.awt.event.ItemListener;
    11 import java.util.ArrayList;
    1211import java.util.HashMap;
    13 import java.util.List;
     12import java.util.HashSet;
    1413import java.util.Map;
    1514import java.util.Observable;
    1615import java.util.Observer;
     16import java.util.Set;
    1717
    1818import javax.swing.JCheckBox;
    … …  
    2828public class AdjustmentSynchronizer implements AdjustmentListener {
    2929
    30     private final List<Adjustable> synchronizedAdjustables;
     30    private final Set<Adjustable> synchronizedAdjustables;
    3131    private final Map<Adjustable, Boolean> enabledMap;
    3232
    … …  
    3737     */
    3838    public AdjustmentSynchronizer() {
    39         synchronizedAdjustables = new ArrayList<>();
     39        synchronizedAdjustables = new HashSet<>();
    4040        enabledMap = new HashMap<>();
    4141        observable = new Observable();

  • trunk/src/org/openstreetmap/josm/gui/util/CellEditorSupport.java

    r7005 r8390  
    33
    44import java.util.ArrayList;
    5 import java.util.LinkedList;
     5import java.util.LinkedHashSet;
    66import java.util.List;
     7import java.util.Set;
    78
    89import javax.swing.CellEditor;
    … …  
    1718public class CellEditorSupport {
    1819    private final CellEditor editor;
    19     private final List<CellEditorListener> listeners;
     20    private final Set<CellEditorListener> listeners;
    2021
    2122    /**
    … …  
    2526    public CellEditorSupport(CellEditor editor) {
    2627        this.editor = editor;
    27         this.listeners = new LinkedList<>();
     28        this.listeners = new LinkedHashSet<>();
    2829    }
    2930
    … …  
    4041    public final void addCellEditorListener(CellEditorListener l) {
    4142        synchronized (this) {
    42             if (l != null && ! listeners.contains(l)) {
     43            if (l != null && !listeners.contains(l)) {
    4344                listeners.add(l);
    4445            }
    … …  
    5253    public final void removeCellEditorListener(CellEditorListener l) {
    5354        synchronized (this) {
    54             if (l != null &&listeners.contains(l)) {
     55            if (l != null && listeners.contains(l)) {
    5556                listeners.remove(l);
    5657            }

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

    r8379 r8390  
    10201020        public String toString() {
    10211021            int nChildren = getChildren().size();
    1022             StringBuffer sb = new StringBuffer("MultiSplitLayout.Split");
     1022            StringBuilder sb = new StringBuilder("MultiSplitLayout.Split");
    10231023            sb.append(isRowLayout() ? " ROW [" : " COLUMN [")
    10241024              .append(nChildren + ((nChildren == 1) ? " child" : " children"))
    … …  
    10731073        @Override
    10741074        public String toString() {
    1075             StringBuffer sb = new StringBuffer("MultiSplitLayout.Leaf");
     1075            StringBuilder sb = new StringBuilder("MultiSplitLayout.Leaf");
    10761076            sb.append(" \"")
    10771077              .append(getName())

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

    r8308 r8390  
    6262        Long l = 0L;
    6363        try {
    64             l = Long.parseLong(v);
     64            l = Long.valueOf(v);
    6565        } catch(NumberFormatException e) {
    6666            throwException(tr("Illegal value for mandatory attribute ''{0}'' of type long. Got ''{1}''.", name, v), e);

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

    r5131 r8390  
    1313 */
    1414public class AllFormatsImporter extends FileImporter {
     15    /**
     16     * Constructs a new {@code AllFormatsImporter}.
     17     */
    1518    public AllFormatsImporter() {
    1619        super(new ExtensionFileFilter(getAllExtensions(), "", tr("All Formats")
    … …  
    1821    }
    1922
    20     @Override public boolean acceptFile(File pathname) {
     23    @Override
     24    public boolean acceptFile(File pathname) {
    2125        return false;
    2226    }
    … …  
    3438                continue;
    3539            }
    36             ext.append(fi.filter.getExtensions());
    37             ext.append(",");
     40            ext.append(fi.filter.getExtensions()).append(',');
    3841        }
    3942        // remove last comma

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

    r8387 r8390  
    339339        StringBuilder prefKey = new StringBuilder("mirror.");
    340340        if (destDir != null) {
    341             prefKey.append(destDir);
    342             prefKey.append(".");
     341            prefKey.append(destDir).append('.');
    343342        }
    344343        prefKey.append(url.toString());
    … …  
    375374                }
    376375                if (cachingStrategy == CachingStrategy.IfModifiedSince) {
    377                     ifModifiedSince = Long.parseLong(localPathEntry.get(0));
     376                    ifModifiedSince = Long.valueOf(localPathEntry.get(0));
    378377                }
    379378            }

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

    r8347 r8390  
    101101        String s = get(element, attribute);
    102102        if (s == null) return null;
    103         return Double.parseDouble(s);
     103        return Double.valueOf(s);
    104104    }
    105105
    … …  
    115115        String s = get(element, attribute);
    116116        if (s == null) return null;
    117         return Long.parseLong(s);
     117        return Long.valueOf(s);
    118118    }
    119119
    … …  
    197197        if (v != null) {
    198198            try {
    199                 Long n = Long.parseLong(v);
     199                long n = Long.parseLong(v);
    200200                if (n <= 0) {
    201201                    warnIllegalValue("waynodes", "maximum", n);

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

    r8304 r8390  
    253253        StringBuilder sb = new StringBuilder();
    254254        if (uid != null) {
    255             sb.append("user").append("=").append(uid);
     255            sb.append("user=").append(uid);
    256256        } else if (userName != null) {
    257             sb.append("display_name").append("=").append(Utils.encodeUrl(userName));
     257            sb.append("display_name=").append(Utils.encodeUrl(userName));
    258258        }
    259259        if (bounds != null) {
    260260            if (sb.length() > 0) {
    261                 sb.append("&");
     261                sb.append('&');
    262262            }
    263263            sb.append("bbox=").append(bounds.encodeAsString(","));
    … …  
    265265        if (closedAfter != null && createdBefore != null) {
    266266            if (sb.length() > 0) {
    267                 sb.append("&");
     267                sb.append('&');
    268268            }
    269269            DateFormat df = DateUtils.newIsoDateTimeFormat();
    270             sb.append("time").append("=").append(df.format(closedAfter));
    271             sb.append(",").append(df.format(createdBefore));
     270            sb.append("time=").append(df.format(closedAfter));
     271            sb.append(',').append(df.format(createdBefore));
    272272        } else if (closedAfter != null) {
    273273            if (sb.length() > 0) {
    274                 sb.append("&");
     274                sb.append('&');
    275275            }
    276276            DateFormat df = DateUtils.newIsoDateTimeFormat();
    277             sb.append("time").append("=").append(df.format(closedAfter));
     277            sb.append("time=").append(df.format(closedAfter));
    278278        }
    279279
    280280        if (open != null) {
    281281            if (sb.length() > 0) {
    282                 sb.append("&");
     282                sb.append('&');
    283283            }
    284284            sb.append("open=").append(Boolean.toString(open));
    285285        } else if (closed != null) {
    286286            if (sb.length() > 0) {
    287                 sb.append("&");
     287                sb.append('&');
    288288            }
    289289            sb.append("closed=").append(Boolean.toString(closed));
    … …  
    291291            // since 2013-12-05, see https://github.com/openstreetmap/openstreetmap-website/commit/1d1f194d598e54a5d6fb4f38fb569d4138af0dc8
    292292            if (sb.length() > 0) {
    293                 sb.append("&");
     293                sb.append('&');
    294294            }
    295295            sb.append("changesets=").append(Utils.join(",", changesetIds));

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

    r8378 r8390  
    397397                case "pdop":
    398398                    try {
    399                         currentWayPoint.put(localName, Float.parseFloat(accumulator.toString()));
     399                        currentWayPoint.put(localName, Float.valueOf(accumulator.toString()));
    400400                    } catch(Exception e) {
    401401                        currentWayPoint.put(localName, new Float(0));

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

    r8338 r8390  
    256256
    257257    /**
    258      * builds the Multi Get request string for a set of ids and a given
    259      * {@link OsmPrimitiveType}.
     258     * builds the Multi Get request string for a set of ids and a given {@link OsmPrimitiveType}.
    260259     *
    261260     * @param type The primitive type. Must be one of {@link OsmPrimitiveType#NODE NODE}, {@link OsmPrimitiveType#WAY WAY}, {@link OsmPrimitiveType#RELATION RELATION}
    … …  
    272271            sb.append(it.next());
    273272            if (i < idPackage.size()-1) {
    274                 sb.append(",");
     273                sb.append(',');
    275274            }
    276275        }
    … …  
    279278
    280279    /**
    281      * builds the Multi Get request string for a single id and a given
    282      * {@link OsmPrimitiveType}.
     280     * builds the Multi Get request string for a single id and a given {@link OsmPrimitiveType}.
    283281     *
    284282     * @param type The primitive type. Must be one of {@link OsmPrimitiveType#NODE NODE}, {@link OsmPrimitiveType#WAY WAY}, {@link OsmPrimitiveType#RELATION RELATION}

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

    r8379 r8390  
    6969
    7070    private void showNmeaInfobox(boolean success, NmeaReader r) {
    71         final StringBuilder msg = new StringBuilder().append("<html>");
    72         msg.append(tr("Coordinates imported: {0}", r.getNumberOfCoordinates()) + "<br>")
    73            .append(tr("Malformed sentences: {0}", r.getParserMalformed()) + "<br>")
    74            .append(tr("Checksum errors: {0}", r.getParserChecksumErrors()) + "<br>");
     71        final StringBuilder msg = new StringBuilder(160).append("<html>");
     72        msg.append(tr("Coordinates imported: {0}", r.getNumberOfCoordinates())).append("<br>")
     73           .append(tr("Malformed sentences: {0}", r.getParserMalformed())).append("<br>")
     74           .append(tr("Checksum errors: {0}", r.getParserChecksumErrors())).append("<br>");
    7575        if (!success) {
    76             msg.append(tr("Unknown sentences: {0}", r.getParserUnknown()) + "<br>");
     76            msg.append(tr("Unknown sentences: {0}", r.getParserUnknown())).append("<br>");
    7777        }
    7878        msg.append(tr("Zero coordinates: {0}", r.getParserZeroCoordinates()))

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

    r8384 r8390  
    315315                accu=e[GPGGA.HDOP.position];
    316316                if(!accu.isEmpty()) {
    317                     currentwp.put(GpxConstants.PT_HDOP, Float.parseFloat(accu));
     317                    currentwp.put(GpxConstants.PT_HDOP, Float.valueOf(accu));
    318318                }
    319319                // fix
    … …  
    364364                accu=e[GPGSA.VDOP.position];
    365365                if(!accu.isEmpty()) {
    366                     currentwp.put(GpxConstants.PT_VDOP, Float.parseFloat(accu));
     366                    currentwp.put(GpxConstants.PT_VDOP, Float.valueOf(accu));
    367367                }
    368368                // hdop
    369369                accu=e[GPGSA.HDOP.position];
    370370                if(!accu.isEmpty()) {
    371                     currentwp.put(GpxConstants.PT_HDOP, Float.parseFloat(accu));
     371                    currentwp.put(GpxConstants.PT_HDOP, Float.valueOf(accu));
    372372                }
    373373                // pdop
    374374                accu=e[GPGSA.PDOP.position];
    375375                if(!accu.isEmpty()) {
    376                     currentwp.put(GpxConstants.PT_PDOP, Float.parseFloat(accu));
     376                    currentwp.put(GpxConstants.PT_PDOP, Float.valueOf(accu));
    377377                }
    378378            } else if("$GPRMC".equals(e[0]) || "$GNRMC".equals(e[0])) {

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

    r8378 r8390  
    324324        StringBuilder rv = new StringBuilder(serverUrl);
    325325        if (version != null) {
    326             rv.append("/");
    327             rv.append(version);
    328         }
    329         rv.append("/");
     326            rv.append('/').append(version);
     327        }
     328        rv.append('/');
    330329        // this works around a ruby (or lighttpd) bug where two consecutive slashes in
    331330        // an URL will cause a "404 not found" response.
    … …  
    683682                        while((s = in.readLine()) != null) {
    684683                            responseBody.append(s);
    685                             responseBody.append("\n");
     684                            responseBody.append('\n');
    686685                        }
    687686                    }

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

    r8342 r8390  
    136136                sb.append(", Error Header=<")
    137137                .append(eh)
    138                 .append(">");
     138                .append('>');
    139139            }
    140140        } catch (Exception e) {
    … …  
    146146                sb.append(", Error Body=<")
    147147                .append(eb)
    148                 .append(">");
     148                .append('>');
    149149            }
    150150        } catch (Exception e) {

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

    r8291 r8390  
    125125            progressMonitor.subTask(tr("Contacting OSM Server..."));
    126126            StringBuilder sb = new StringBuilder();
    127             sb.append(primitiveType.getAPIName()).append("/").append(id).append(type);
     127            sb.append(primitiveType.getAPIName()).append('/').append(id).append(type);
    128128
    129129            try (InputStream in = getInputStream(sb.toString(), progressMonitor.createSubTaskMonitor(1, true))) {

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

    r8291 r8390  
    6060            StringBuilder sb = new StringBuilder();
    6161            sb.append(primitiveType.getAPIName())
    62             .append("/").append(id).append("/history");
     62            .append('/').append(id).append("/history");
    6363
    6464            try (InputStream in = getInputStream(sb.toString(), progressMonitor.createSubTaskMonitor(1, true))) {

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

    r8379 r8390  
    118118            StringBuilder sb = new StringBuilder();
    119119            sb.append(id.getType().getAPIName())
    120               .append("/")
     120              .append('/')
    121121              .append(id.getUniqueId());
    122122            if (full && ! id.getType().equals(OsmPrimitiveType.NODE)) {
    123123                sb.append("/full");
    124124            } else if (version > 0) {
    125                 sb.append("/").append(version);
     125                sb.append('/').append(version);
    126126            }
    127127

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

    r8342 r8390  
    184184                            while((s = in.readLine()) != null) {
    185185                                errorBody.append(s);
    186                                 errorBody.append("\n");
     186                                errorBody.append('\n');
    187187                            }
    188188                        }

  • trunk/src/org/openstreetmap/josm/io/imagery/ImageryReader.java

    r8349 r8390  
    263263                    Integer val = null;
    264264                    try {
    265                         val = Integer.parseInt(accumulator.toString());
     265                        val = Integer.valueOf(accumulator.toString());
    266266                    } catch(NumberFormatException e) {
    267267                        val = null;
    … …  
    304304                    Integer tileSize = null;
    305305                    try {
    306                         tileSize  = Integer.parseInt(accumulator.toString());
     306                        tileSize = Integer.valueOf(accumulator.toString());
    307307                    } catch(NumberFormatException e) {
    308308                        tileSize = null;

  • trunk/src/org/openstreetmap/josm/io/imagery/WMSImagery.java

    r8379 r8390  
    8181        }
    8282        StringBuilder a = new StringBuilder(serviceUrl.getProtocol());
    83         a.append("://")
    84          .append(serviceUrl.getHost());
     83        a.append("://").append(serviceUrl.getHost());
    8584        if (serviceUrl.getPort() != -1) {
    86             a.append(":")
    87              .append(serviceUrl.getPort());
    88         }
    89         a.append(serviceUrl.getPath())
    90          .append("?");
     85            a.append(':').append(serviceUrl.getPort());
     86        }
     87        a.append(serviceUrl.getPath()).append('?');
    9188        if (serviceUrl.getQuery() != null) {
    9289            a.append(serviceUrl.getQuery());
    9390            if (!serviceUrl.getQuery().isEmpty() && !serviceUrl.getQuery().endsWith("&")) {
    94                 a.append("&");
     91                a.append('&');
    9592            }
    9693        }
    … …  
    150147            while ((line = br.readLine()) != null) {
    151148                ba.append(line);
    152                 ba.append("\n");
     149                ba.append('\n');
    153150            }
    154151        }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RequestProcessor.java

    r8263 r8390  
    369369        StringBuilder r = new StringBuilder();
    370370        boolean first = true;
    371         r.append("[");
     371        r.append('[');
    372372
    373373        for (Entry<String, Class<? extends RequestHandler>> p : handlers.entrySet()) {
    … …  
    379379            r.append(getHandlerInfoAsJSON(p.getKey()));
    380380        }
    381         r.append("]");
     381        r.append(']');
    382382
    383383        return r.toString();
    … …  
    454454     * Reports HTML message with the description of all available commands
    455455     * @return HTML message with the description of all available commands
    456      * @throws IllegalAccessException
    457      * @throws InstantiationException
     456     * @throws IllegalAccessException if one handler class or its nullary constructor is not accessible.
     457     * @throws InstantiationException if one handler class represents an abstract class, an interface, an array class,
     458     * a primitive type, or void; or if the class has no nullary constructor; or if the instantiation fails for some other reason.
    458459     */
    459460    public static String getUsageAsHtml() throws IllegalAccessException, InstantiationException {
    … …  
    464465            String[] optional = sample.getOptionalParams();
    465466            String[] examples = sample.getUsageExamples(handler.getKey().substring(1));
    466             usage.append("<li>");
    467             usage.append(handler.getKey());
     467            usage.append("<li>")
     468                 .append(handler.getKey());
    468469            if (sample.getUsage() != null && !sample.getUsage().isEmpty()) {
    469470                usage.append(" &mdash; <i>").append(sample.getUsage()).append("</i>");
    … …  
    478479                usage.append("<br/>examples: ");
    479480                for (String ex: examples) {
    480                     usage.append("<br/> <a href=\"http://localhost:8111"+ex+"\">"+ex+"</a>");
     481                    usage.append("<br/> <a href=\"http://localhost:8111").append(ex).append("\">").append(ex).append("</a>");
    481482                }
    482483            }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/FeaturesHandler.java

    r7937 r8390  
    22package org.openstreetmap.josm.io.remotecontrol.handler;
    33
    4 import org.openstreetmap.josm.Main;
    54import static org.openstreetmap.josm.tools.I18n.tr;
    65
     6import org.openstreetmap.josm.Main;
    77import org.openstreetmap.josm.io.remotecontrol.PermissionPrefWithDefault;
    88import org.openstreetmap.josm.io.remotecontrol.RequestProcessor;
    … …  
    2525        String q=args.get("q");
    2626        if (q!=null) {
    27             buf.append("[");
     27            buf.append('[');
    2828            boolean first = true;
    2929            for (String s: q.split("[,\\s]+")) {
    … …  
    4040               }
    4141            }
    42             buf.append("]");
     42            buf.append(']');
    4343        } else {
    4444            buf.append(RequestProcessor.getHandlersInfoAsJSON());
    4545        }
    46            
     46
    4747        content = buf.toString();
    4848        contentType = "application/json";
    … …  
    6666        return null;
    6767    }
    68    
     68
    6969    @Override
    7070    public String[] getOptionalParams() {
    … …  
    8484    @Override
    8585    public String[] getUsageExamples() {
    86         return new String[] {"/features", "/features?q=import,add_node"}; 
     86        return new String[] {"/features", "/features?q=import,add_node"};
    8787    }
    8888}

  • trunk/src/org/openstreetmap/josm/io/session/GeoImageSessionImporter.java

    r7937 r8390  
    5555                                    break;
    5656                                case "speed":
    57                                     entry.setSpeed(Double.parseDouble(attrElem.getTextContent()));
     57                                    entry.setSpeed(Double.valueOf(attrElem.getTextContent()));
    5858                                    break;
    5959                                case "elevation":
    60                                     entry.setElevation(Double.parseDouble(attrElem.getTextContent()));
     60                                    entry.setElevation(Double.valueOf(attrElem.getTextContent()));
    6161                                    break;
    6262                                case "gps-time":
    … …  
    6464                                    break;
    6565                                case "exif-orientation":
    66                                     entry.setExifOrientation(Integer.parseInt(attrElem.getTextContent()));
     66                                    entry.setExifOrientation(Integer.valueOf(attrElem.getTextContent()));
    6767                                    break;
    6868                                case "exif-time":

  • trunk/src/org/openstreetmap/josm/io/session/SessionReader.java

    r8332 r8390  
    362362                    Integer idx = null;
    363363                    try {
    364                         idx = Integer.parseInt(e.getAttribute("index"));
     364                        idx = Integer.valueOf(e.getAttribute("index"));
    365365                    } catch (NumberFormatException ex) {
    366366                        Main.warn(ex);
    … …  
    380380                            Integer d = null;
    381381                            try {
    382                                 d = Integer.parseInt(sd);
     382                                d = Integer.valueOf(sd);
    383383                            } catch (NumberFormatException ex) {
    384384                                Main.warn(ex);

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

    r8379 r8390  
    278278            sb.append("<li>").append(depr.name);
    279279            if (depr.reason != null) {
    280                 sb.append(" (").append(depr.reason).append(")");
     280                sb.append(" (").append(depr.reason).append(')');
    281281            }
    282282            sb.append("</li>");
    283283        }
    284         sb.append("</ul>")
    285           .append("</html>");
     284        sb.append("</ul></html>");
    286285        JOptionPane.showMessageDialog(
    287286                parent,
    … …  
    13261325        }
    13271326        for (String s : pl) {
    1328             text.append("- ").append(s).append("\n");
     1327            text.append("- ").append(s).append('\n');
    13291328        }
    13301329        return text.toString();
    … …  
    13511350                        b.append(": ");
    13521351                        b.append(e.getValue());
    1353                         b.append("\n");
     1352                        b.append('\n');
    13541353                    }
    13551354                    JosmTextArea a = new JosmTextArea(10, 40);

  • trunk/src/org/openstreetmap/josm/plugins/PluginInformation.java

    r8379 r8390  
    304304     */
    305305    public String getDescriptionAsHtml() {
    306         StringBuilder sb = new StringBuilder();
     306        StringBuilder sb = new StringBuilder(128);
    307307        sb.append("<html><body>")
    308308          .append(description == null ? tr("no description available") : description);
    … …  
    315315                && !downloadlink.startsWith("http://trac.openstreetmap.org/browser/applications/editors/josm/dist/")
    316316                && !downloadlink.startsWith("https://github.com/JOSM/")) {
    317             sb.append("<p>&nbsp;</p><p>"+tr("<b>Plugin provided by an external source:</b> {0}", downloadlink)+"</p>");
     317            sb.append("<p>&nbsp;</p><p>").append(tr("<b>Plugin provided by an external source:</b> {0}", downloadlink)).append("</p>");
    318318        }
    319319        sb.append("</body></html>");

  • trunk/src/org/openstreetmap/josm/plugins/PluginListParser.java

    r8291 r8390  
    6969                    line = line.substring(1);
    7070                    while (line.length() > 70) {
    71                         manifest.append(line.substring(0, 70)).append("\n");
     71                        manifest.append(line.substring(0, 70)).append('\n');
    7272                        line = " " + line.substring(70);
    7373                    }
    74                     manifest.append(line).append("\n");
     74                    manifest.append(line).append('\n');
    7575                    continue;
    7676                }

  • trunk/src/org/openstreetmap/josm/plugins/ReadRemotePluginInformationTask.java

    r8379 r8390  
    115115            StringBuilder sb = new StringBuilder();
    116116            sb.append("site-")
    117               .append(url.getHost()).append("-");
     117              .append(url.getHost()).append('-');
    118118            if (url.getPort() != -1) {
    119                 sb.append(url.getPort()).append("-");
     119                sb.append(url.getPort()).append('-');
    120120            }
    121121            String path = url.getPath();
    … …  
    125125                    sb.append(c);
    126126                } else {
    127                     sb.append("_");
     127                    sb.append('_');
    128128                }
    129129            }
    … …  
    167167                String line;
    168168                while ((line = in.readLine()) != null) {
    169                     sb.append(line).append("\n");
     169                    sb.append(line).append('\n');
    170170                }
    171171                return sb.toString();
    … …  
    198198                    String line;
    199199                    while ((line = err.readLine()) != null) {
    200                         sb.append(line).append("\n");
     200                        sb.append(line).append('\n');
    201201                    }
    202202                } catch (Exception ex) {
    … …  
    228228                StringBuilder b = new StringBuilder();
    229229                for (String part : msg.split("(?<=\\G.{200})")) {
    230                     b.append(part).append("\n");
     230                    b.append(part).append('\n');
    231231                }
    232232                panel.add(new JLabel("<html><body width=\"500\"><b>"+b.toString().trim()+"</b></body></html>"), GBC.eol().insets(0, 0, 0, 10));

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

    r8375 r8390  
    526526                    w.append(key + "=" + value + "\n");
    527527                }
    528                 w.append("\n");
     528                w.append('\n');
    529529                for (FontEntry entry: extras) {
    530530                    if ("".equals(entry.name) || "".equals(entry.file)) {
    … …  
    539539                    w.append(key + "=" + value + "\n");
    540540                }
    541                 w.append("\n");
     541                w.append('\n');
    542542                String fallback = props.getProperty("sequence.fallback");
    543543                if (fallback != null) {

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

    r8385 r8390  
    931931                    all = new StringBuilder(line);
    932932                } else {
    933                     all.append("\n");
     933                    all.append('\n');
    934934                    all.append(line);
    935935                }
    … …  
    10071007                ++cnt;
    10081008            } else if (cnt == 0) {
    1009                 sb.append(",").append(cur);
     1009                sb.append(',').append(cur);
    10101010            } else {
    1011                 sb.append("-").append(last);
    1012                 sb.append(",").append(cur);
     1011                sb.append('-').append(last);
     1012                sb.append(',').append(cur);
    10131013                cnt = 0;
    10141014            }
    … …  
    10161016        }
    10171017        if (cnt >= 1) {
    1018             sb.append("-").append(last);
     1018            sb.append('-').append(last);
    10191019        }
    10201020        return sb.toString();

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

    r7434 r8390  
    110110        for (String line = in.readLine(); line != null; line = in.readLine()) {
    111111            if (!line.contains("[[TranslatedPages]]")) {
    112                 b.append(line.replaceAll(" />", ">")).append("\n");
     112                b.append(line.replaceAll(" />", ">")).append('\n');
    113113            }
    114114        }

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

    r8365 r8390  
    253253    public void remember(String preferenceKey) {
    254254        StringBuilder value = new StringBuilder();
    255         value.append("x=").append(topLeft.x).append(",")
    256         .append("y=").append(topLeft.y).append(",")
    257         .append("width=").append(extent.width).append(",")
    258         .append("height=").append(extent.height);
     255        value.append("x=").append(topLeft.x).append(",y=").append(topLeft.y)
     256             .append(",width=").append(extent.width).append(",height=").append(extent.height);
    259257        Main.pref.put(preferenceKey, value.toString());
    260258    }

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

    r8287 r8390  
    125125                return parseBoolean(value);
    126126            else if (klass == Integer.TYPE || klass == Long.TYPE)
    127                 return Long.parseLong(value);
     127                return Long.valueOf(value);
    128128            else if (klass == Float.TYPE || klass == Double.TYPE)
    129                 return Double.parseDouble(value);
     129                return Double.valueOf(value);
    130130            return value;
    131131        }

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

    r8376 r8390  
    4747                sb.append(entry);
    4848            } else {
    49                 sb.append("'");
    50                 sb.append(entry);
    51                 sb.append("'");
     49                sb.append('\'').append(entry).append('\'');
    5250            }
    53             sb.append("|");
     51            sb.append('|');
    5452        }
    5553        return sb.toString();

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

    r7937 r8390  
    3535                    first = false;
    3636                }
    37                 result.append(key).append("=").append(dataProvider.getTemplateValue(key, false));
     37                result.append(key).append('=').append(dataProvider.getTemplateValue(key, false));
    3838            }
    3939        } else {
Note: See TracChangeset for help on using the changeset viewer.