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

Sonar - various performance improvements

Location:
trunk/src/org/openstreetmap/josm/io
Files:
21 edited

Legend:

Unmodified
Added
Removed
  • 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);
Note: See TracChangeset for help on using the changeset viewer.