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

Sonar - various performance improvements

Location:
trunk/src/org/openstreetmap/josm/tools
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • 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.