Ignore:
Timestamp:
2015-06-20T23:42:21+02:00 (5 years ago)
Author:
Don-vip
Message:

checkstyle: enable relevant whitespace checks and fix them

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

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/io/AbstractParser.java

    r8390 r8510  
    4747        try {
    4848            l = Long.parseLong(v);
    49         } catch(NumberFormatException e) {
     49        } catch (NumberFormatException e) {
    5050            throwException(tr("Illegal value for mandatory attribute ''{0}'' of type long. Got ''{1}''.", name, v), e);
    5151        }
     
    6363        try {
    6464            l = Long.valueOf(v);
    65         } catch(NumberFormatException e) {
     65        } catch (NumberFormatException e) {
    6666            throwException(tr("Illegal value for mandatory attribute ''{0}'' of type long. Got ''{1}''.", name, v), e);
    6767        }
     
    8080        try {
    8181            d = Double.parseDouble(v);
    82         } catch(NumberFormatException e) {
     82        } catch (NumberFormatException e) {
    8383            throwException(tr("Illegal value for attribute ''{0}'' of type double. Got ''{1}''.", name, v), e);
    8484        }
     
    106106
    107107    protected final HistoryOsmPrimitive createPrimitive(Attributes atts, OsmPrimitiveType type) throws SAXException {
    108         long id = getMandatoryAttributeLong(atts,"id");
    109         long version = getMandatoryAttributeLong(atts,"version");
    110         long changesetId = getMandatoryAttributeLong(atts,"changeset");
    111         boolean visible= getMandatoryAttributeBoolean(atts, "visible");
     108        long id = getMandatoryAttributeLong(atts, "id");
     109        long version = getMandatoryAttributeLong(atts, "version");
     110        long changesetId = getMandatoryAttributeLong(atts, "changeset");
     111        boolean visible = getMandatoryAttributeBoolean(atts, "visible");
    112112
    113113        Long uid = getAttributeLong(atts, "uid");
     
    130130            Double lat = getAttributeDouble(atts, "lat");
    131131            Double lon = getAttributeDouble(atts, "lon");
    132             LatLon coor = (lat != null && lon != null) ? new LatLon(lat,lon) : null;
    133             primitive = new HistoryNode(
    134                     id,version,visible,user,changesetId,timestamp,coor
    135             );
     132            LatLon coor = (lat != null && lon != null) ? new LatLon(lat, lon) : null;
     133            primitive = new HistoryNode(id, version, visible, user, changesetId, timestamp, coor);
    136134
    137135        } else if (type.equals(OsmPrimitiveType.WAY)) {
    138             primitive = new HistoryWay(
    139                     id,version,visible,user,changesetId,timestamp
    140             );
     136            primitive = new HistoryWay(id, version, visible, user, changesetId, timestamp);
    141137        } else if (type.equals(OsmPrimitiveType.RELATION)) {
    142             primitive = new HistoryRelation(
    143                     id,version,visible,user,changesetId,timestamp
    144             );
     138            primitive = new HistoryRelation(id, version, visible, user, changesetId, timestamp);
    145139        }
    146140        return primitive;
     
    162156        String key = getMandatoryAttributeString(atts, "k");
    163157        String value = getMandatoryAttributeString(atts, "v");
    164         currentPrimitive.put(key,value);
     158        currentPrimitive.put(key, value);
    165159    }
    166160
    167161    protected final void handleNodeReference(Attributes atts) throws SAXException {
    168162        long ref = getMandatoryAttributeLong(atts, "ref");
    169         ((HistoryWay)currentPrimitive).addNode(ref);
     163        ((HistoryWay) currentPrimitive).addNode(ref);
    170164    }
    171165
     
    176170        try {
    177171            type = OsmPrimitiveType.fromApiTypeName(v);
    178         } catch(IllegalArgumentException e) {
     172        } catch (IllegalArgumentException e) {
    179173            throwException(tr("Illegal value for mandatory attribute ''{0}'' of type OsmPrimitiveType. Got ''{1}''.", "type", v), e);
    180174        }
    181175        String role = getMandatoryAttributeString(atts, "role");
    182         RelationMemberData member = new RelationMemberData(role, type,ref);
    183         ((HistoryRelation)currentPrimitive).addMember(member);
     176        RelationMemberData member = new RelationMemberData(role, type, ref);
     177        ((HistoryRelation) currentPrimitive).addMember(member);
    184178    }
    185179
  • trunk/src/org/openstreetmap/josm/io/AbstractReader.java

    r8419 r8510  
    8282     * @throws IllegalDataException if a data integrity problem is detected
    8383     */
    84     protected void processWaysAfterParsing() throws IllegalDataException{
     84    protected void processWaysAfterParsing() throws IllegalDataException {
    8585        for (Entry<Long, Collection<Long>> entry : ways.entrySet()) {
    8686            Long externalWayId = entry.getKey();
    87             Way w = (Way)externalIdMap.get(new SimplePrimitiveId(externalWayId, OsmPrimitiveType.WAY));
     87            Way w = (Way) externalIdMap.get(new SimplePrimitiveId(externalWayId, OsmPrimitiveType.WAY));
    8888            List<Node> wayNodes = new ArrayList<>();
    8989            for (long id : entry.getValue()) {
    90                 Node n = (Node)externalIdMap.get(new SimplePrimitiveId(id, OsmPrimitiveType.NODE));
     90                Node n = (Node) externalIdMap.get(new SimplePrimitiveId(id, OsmPrimitiveType.NODE));
    9191                if (n == null) {
    9292                    if (id <= 0)
     
    9696                                        id));
    9797                    // create an incomplete node if necessary
    98                     //
    99                     n = (Node)ds.getPrimitiveById(id,OsmPrimitiveType.NODE);
     98                    n = (Node) ds.getPrimitiveById(id, OsmPrimitiveType.NODE);
    10099                    if (n == null) {
    101100                        n = new Node(id);
  • trunk/src/org/openstreetmap/josm/io/AllFormatsImporter.java

    r8390 r8510  
    3333        Iterator<FileImporter> imp = ExtensionFileFilter.importers.iterator();
    3434        StringBuilder ext = new StringBuilder();
    35         while(imp.hasNext()) {
     35        while (imp.hasNext()) {
    3636            FileImporter fi = imp.next();
    37             if(fi instanceof AllFormatsImporter) {
     37            if (fi instanceof AllFormatsImporter) {
    3838                continue;
    3939            }
  • trunk/src/org/openstreetmap/josm/io/BoundingBoxDownloader.java

    r8345 r8510  
    4949        GpxData result = null;
    5050        String url = "trackpoints?bbox="+b.getMinLon()+","+b.getMinLat()+","+b.getMaxLon()+","+b.getMaxLat()+"&page=";
    51         for (int i = 0;!done;++i) {
     51        for (int i = 0; !done; ++i) {
    5252            progressMonitor.subTask(tr("Downloading points {0} to {1}...", i * 5000, (i + 1) * 5000));
    5353            try (InputStream in = getInputStream(url+i, progressMonitor.createSubTaskMonitor(1, true))) {
     
    6363                } else if (currentGpx.hasTrackPoints()) {
    6464                    result.mergeFrom(currentGpx);
    65                 } else{
     65                } else {
    6666                    done = true;
    6767                }
     
    150150            }
    151151            return ds;
    152         } catch(OsmTransferException e) {
     152        } catch (OsmTransferException e) {
    153153            throw e;
    154154        } catch (Exception e) {
     
    188188     * Indicates that the number of fetched notes equals the specified limit. Thus there might be more notes to download.
    189189     */
    190     public static class MoreNotesException extends RuntimeException{
     190    public static class MoreNotesException extends RuntimeException {
    191191        /**
    192192         * The downloaded notes
  • trunk/src/org/openstreetmap/josm/io/CacheCustomContent.java

    r8419 r8510  
    134134        this.data = updateData();
    135135        saveToDisk();
    136         Main.pref.putInteger("cache." + ident, (int)(System.currentTimeMillis()/1000));
     136        Main.pref.putInteger("cache." + ident, (int) (System.currentTimeMillis()/1000));
    137137        return data;
    138138    }
  • trunk/src/org/openstreetmap/josm/io/CacheFiles.java

    r8345 r8510  
    7878            this.dir.mkdirs();
    7979            dir_writeable = true;
    80         } catch(Exception e) {
     80        } catch (Exception e) {
    8181            // We have no access to this directory, so don't do anything
    8282            dir_writeable = false;
     
    8484        this.enabled = dir_writeable;
    8585        this.expire = Main.pref.getLong("cache." + ident + "." + "expire", EXPIRE_DAILY);
    86         if(this.expire < 0) {
     86        if (this.expire < 0) {
    8787            this.expire = CacheFiles.EXPIRE_NEVER;
    8888        }
    8989        this.maxsize = Main.pref.getLong("cache." + ident + "." + "maxsize", 50);
    90         if(this.maxsize < 0) {
     90        if (this.maxsize < 0) {
    9191            this.maxsize = -1;
    9292        }
     
    9999     */
    100100    public byte[] getData(String ident) {
    101         if(!enabled) return null;
     101        if (!enabled) return null;
    102102        try {
    103103            File data = getPath(ident);
    104             if(!data.exists())
     104            if (!data.exists())
    105105                return null;
    106106
    107             if(isExpired(data)) {
     107            if (isExpired(data)) {
    108108                data.delete();
    109109                return null;
     
    111111
    112112            // Update last mod time so we don't expire recently used data
    113             if(updateModTime) {
     113            if (updateModTime) {
    114114                data.setLastModified(System.currentTimeMillis());
    115115            }
     
    132132     */
    133133    public void saveData(String ident, byte[] data) {
    134         if(!enabled) return;
     134        if (!enabled) return;
    135135        try {
    136136            File f = getPath(ident);
     
    156156     */
    157157    public BufferedImage getImg(String ident) {
    158         if(!enabled) return null;
     158        if (!enabled) return null;
    159159        try {
    160160            File img = getPath(ident, "png");
    161             if(!img.exists())
     161            if (!img.exists())
    162162                return null;
    163163
    164             if(isExpired(img)) {
     164            if (isExpired(img)) {
    165165                img.delete();
    166166                return null;
    167167            }
    168168            // Update last mod time so we don't expire recently used images
    169             if(updateModTime) {
     169            if (updateModTime) {
    170170                img.setLastModified(System.currentTimeMillis());
    171171            }
     
    201201    public void setExpire(int amount, boolean force) {
    202202        String key = "cache." + ident + "." + "expire";
    203         if(!Main.pref.get(key).isEmpty() && !force)
     203        if (!Main.pref.get(key).isEmpty() && !force)
    204204            return;
    205205
     
    215215    public void setMaxSize(int amount, boolean force) {
    216216        String key = "cache." + ident + "." + "maxsize";
    217         if(!Main.pref.get(key).isEmpty() && !force)
     217        if (!Main.pref.get(key).isEmpty() && !force)
    218218            return;
    219219
     
    235235     */
    236236    public void checkCleanUp() {
    237         if(this.writes > CLEANUP_INTERVAL) {
     237        if (this.writes > CLEANUP_INTERVAL) {
    238238            cleanUp();
    239239        }
     
    244244     */
    245245    public void cleanUp() {
    246         if(!this.enabled || maxsize == -1) return;
     246        if (!this.enabled || maxsize == -1) return;
    247247
    248248        SortedMap<Long, File> modtime = new TreeMap<>();
     
    251251        File[] files = dir.listFiles();
    252252        if (files != null) {
    253             for(File f : files) {
    254                 if(isExpired(f)) {
     253            for (File f : files) {
     254                if (isExpired(f)) {
    255255                    f.delete();
    256256                } else {
     
    261261        }
    262262
    263         if(dirsize < maxsize*1000*1000) return;
     263        if (dirsize < maxsize*1000*1000) return;
    264264
    265265        Set<Long> keySet = modtime.keySet();
    266266        Iterator<Long> it = keySet.iterator();
    267         int i=0;
     267        int i = 0;
    268268        while (it.hasNext()) {
    269269            i++;
     
    271271
    272272            // Delete a couple of files, then check again
    273             if(i % CLEANUP_TRESHOLD == 0 && getDirSize() < maxsize)
     273            if (i % CLEANUP_TRESHOLD == 0 && getDirSize() < maxsize)
    274274                return;
    275275        }
     
    318318     */
    319319    private long getDirSize() {
    320         if(!enabled) return -1;
     320        if (!enabled) return -1;
    321321        long dirsize = 0;
    322322
    323323        File[] files = dir.listFiles();
    324324        if (files != null) {
    325             for(File f : files) {
     325            for (File f : files) {
    326326                dirsize += f.length();
    327327            }
     
    339339            BigInteger number = new BigInteger(1, md.digest(ident.getBytes(StandardCharsets.UTF_8)));
    340340            return number.toString(16);
    341         } catch(Exception e) {
     341        } catch (Exception e) {
    342342            // Fall back. Remove unsuitable characters and some random ones to shrink down path length.
    343343            // Limit it to 70 characters, that leaves about 190 for the path on Windows/NTFS
     
    373373     */
    374374    private boolean isExpired(File file) {
    375         if(CacheFiles.EXPIRE_NEVER == this.expire)
     375        if (CacheFiles.EXPIRE_NEVER == this.expire)
    376376            return false;
    377377        return file.lastModified() < (System.currentTimeMillis() - expire*1000);
  • trunk/src/org/openstreetmap/josm/io/CachedFile.java

    r8509 r8510  
    5959        IfModifiedSince
    6060    }
     61
    6162    protected String name;
    6263    protected long maxAge;
     
    320321                if (localPath.size() == 2) {
    321322                    File lfile = new File(localPath.get(1));
    322                     if(lfile.exists()) {
     323                    if (lfile.exists()) {
    323324                        lfile.delete();
    324325                    }
     
    342343        }
    343344        prefKey.append(url.toString());
    344         return prefKey.toString().replaceAll("=","_");
     345        return prefKey.toString().replaceAll("=", "_");
    345346    }
    346347
     
    469470
    470471        int numRedirects = 0;
    471         while(true) {
     472        while (true) {
    472473            HttpURLConnection con = Utils.openHttpConnection(downloadUrl);
    473474            if (ifModifiedSince != null) {
     
    475476            }
    476477            con.setInstanceFollowRedirects(false);
    477             con.setConnectTimeout(Main.pref.getInteger("socket.timeout.connect",15)*1000);
    478             con.setReadTimeout(Main.pref.getInteger("socket.timeout.read",30)*1000);
     478            con.setConnectTimeout(Main.pref.getInteger("socket.timeout.connect", 15)*1000);
     479            con.setReadTimeout(Main.pref.getInteger("socket.timeout.read", 30)*1000);
    479480            if (Main.isDebugEnabled()) {
    480481                Main.debug("GET "+downloadString);
  • trunk/src/org/openstreetmap/josm/io/Capabilities.java

    r8443 r8510  
    5151public class Capabilities {
    5252
    53     private final Map<String, Map<String,String>> capabilities;
     53    private final Map<String, Map<String, String>> capabilities;
    5454    private final List<String> imageryBlacklist;
    5555
     
    132132        } else {
    133133            if (!capabilities.containsKey(element))  {
    134                 Map<String,String> h = new HashMap<>();
     134                Map<String, String> h = new HashMap<>();
    135135                capabilities.put(element, h);
    136136            }
     
    248248        @Override
    249249        public void startElement(String namespaceURI, String localName, String qName, Attributes atts) throws SAXException {
    250             for (int i=0; i< atts.getLength(); i++) {
     250            for (int i = 0; i < atts.getLength(); i++) {
    251251                capabilities.put(qName, atts.getQName(i), atts.getValue(i));
    252252            }
  • trunk/src/org/openstreetmap/josm/io/ChangesetClosedException.java

    r7299 r8510  
    7777            try {
    7878                closedOn = DateUtils.newOsmApiDateTimeFormat().parse(m.group(2));
    79             } catch(ParseException ex) {
     79            } catch (ParseException ex) {
    8080                Main.error(tr("Failed to parse date ''{0}'' replied by server.", m.group(2)));
    8181                Main.error(ex);
  • trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java

    r8443 r8510  
    3131     * @throws ChangesetQueryUrlException if query doesn't consist of valid query parameters
    3232     */
    33     public static ChangesetQuery buildFromUrlQuery(String query) throws ChangesetQueryUrlException{
     33    public static ChangesetQuery buildFromUrlQuery(String query) throws ChangesetQueryUrlException {
    3434        return new ChangesetQueryUrlParser().parse(query);
    3535    }
     
    151151        CheckParameterUtil.ensureParameterNotNull(min, "min");
    152152        CheckParameterUtil.ensureParameterNotNull(max, "max");
    153         this.bounds  = new Bounds(min,max);
     153        this.bounds  = new Bounds(min, max);
    154154        return this;
    155155    }
     
    341341                    throw new ChangesetQueryUrlException(
    342342                            tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "uid", value));
    343             } catch(NumberFormatException e) {
     343            } catch (NumberFormatException e) {
    344344                throw new ChangesetQueryUrlException(
    345345                        tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "uid", value), e);
     
    370370            try {
    371371                return formatter.parse(value);
    372             } catch(ParseException e) {
     372            } catch (ParseException e) {
    373373                throw new ChangesetQueryUrlException(
    374374                        tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter, value), e);
     
    456456        }
    457457
    458         protected Map<String,String> createMapFromQueryString(String query) {
    459             Map<String,String> queryParams  = new HashMap<>();
     458        protected Map<String, String> createMapFromQueryString(String query) {
     459            Map<String, String> queryParams  = new HashMap<>();
    460460            String[] keyValuePairs = query.split("&");
    461461            for (String keyValuePair: keyValuePairs) {
     
    488488            if (query.isEmpty())
    489489                return new ChangesetQuery();
    490             Map<String,String> queryParams = createMapFromQueryString(query);
     490            Map<String, String> queryParams = createMapFromQueryString(query);
    491491            return createFromMap(queryParams);
    492492        }
  • trunk/src/org/openstreetmap/josm/io/DefaultProxySelector.java

    r8509 r8510  
    138138        if (proxyPolicy.equals(ProxyPolicy.USE_SOCKS_PROXY)) {
    139139            if (host != null && !host.trim().isEmpty() && port > 0) {
    140                 socksProxySocketAddress = new InetSocketAddress(host,port);
     140                socksProxySocketAddress = new InetSocketAddress(host, port);
    141141            } else {
    142142                Main.warn(tr("Unexpected parameters for SOCKS proxy. Got host ''{0}'' and port ''{1}''.", host, port));
     
    203203        case USE_SYSTEM_SETTINGS:
    204204            if (!JVM_WILL_USE_SYSTEM_PROXIES) {
    205                 Main.warn(tr("The JVM is not configured to lookup proxies from the system settings. The property ''java.net.useSystemProxies'' was missing at startup time.  Will not use a proxy."));
     205                Main.warn(tr("The JVM is not configured to lookup proxies from the system settings. "+
     206                        "The property ''java.net.useSystemProxies'' was missing at startup time.  Will not use a proxy."));
    206207                return NO_PROXY_LIST;
    207208            }
  • trunk/src/org/openstreetmap/josm/io/DiffResultProcessor.java

    r8378 r8510  
    8585            InputSource inputSource = new InputSource(new StringReader(diffUploadResponse));
    8686            Utils.parseSafeSAX(inputSource, new Parser());
    87         } catch(XmlParsingException e) {
     87        } catch (XmlParsingException e) {
    8888            throw e;
    89         } catch(IOException | ParserConfigurationException | SAXException e) {
     89        } catch (IOException | ParserConfigurationException | SAXException e) {
    9090            throw new XmlParsingException(e);
    9191        } finally {
  • trunk/src/org/openstreetmap/josm/io/FileWatcher.java

    r7937 r8510  
    8787            throw new IllegalArgumentException("Resource "+obj+" does not have a parent directory");
    8888        }
    89         synchronized(this) {
     89        synchronized (this) {
    9090            // Register directory. Can be called several times for a same directory without problem
    9191            // (it returns the same key so it should not send events several times)
     
    121121                // The filename is the context of the event.
    122122                @SuppressWarnings("unchecked")
    123                 WatchEvent<Path> ev = (WatchEvent<Path>)event;
     123                WatchEvent<Path> ev = (WatchEvent<Path>) event;
    124124                Path filename = ev.context();
    125125                if (filename == null) {
     
    128128
    129129                // Only way to get full path (http://stackoverflow.com/a/7802029/2257172)
    130                 Path fullPath = ((Path)key.watchable()).resolve(filename);
     130                Path fullPath = ((Path) key.watchable()).resolve(filename);
    131131
    132                 synchronized(this) {
     132                synchronized (this) {
    133133                    StyleSource style = styleMap.get(fullPath);
    134134                    SourceEntry rule = ruleMap.get(fullPath);
  • trunk/src/org/openstreetmap/josm/io/GeoJSONWriter.java

    r7937 r8510  
    9393            if (w instanceof Way) {
    9494                JsonArrayBuilder array = Json.createArrayBuilder();
    95                 for (Node n : ((Way)w).getNodes()) {
     95                for (Node n : ((Way) w).getNodes()) {
    9696                    LatLon ll = n.getCoor();
    9797                    if (ll != null) {
  • trunk/src/org/openstreetmap/josm/io/GpxExporter.java

    r8461 r8510  
    248248            final JLabel warning) {
    249249
    250         ActionListener authorActionListener = new ActionListener(){
     250        ActionListener authorActionListener = new ActionListener() {
    251251            @Override
    252252            public void actionPerformed(ActionEvent e) {
     
    278278        author.addActionListener(authorActionListener);
    279279
    280         KeyAdapter authorNameListener = new KeyAdapter(){
     280        KeyAdapter authorNameListener = new KeyAdapter() {
    281281            @Override public void keyReleased(KeyEvent e) {
    282282                boolean b = !authorName.getText().isEmpty() && author.isSelected();
     
    286286        authorName.addKeyListener(authorNameListener);
    287287
    288         predefined.addActionListener(new ActionListener(){
     288        predefined.addActionListener(new ActionListener() {
    289289            @Override
    290290            public void actionPerformed(ActionEvent e) {
  • trunk/src/org/openstreetmap/josm/io/GpxReader.java

    r8470 r8510  
    4444public class GpxReader implements GpxConstants {
    4545
     46    private enum State { init, gpx, metadata, wpt, rte, trk, ext, author, link, trkseg, copyright}
     47
    4648    private String version;
    47     /**
    48      * The resulting gpx data
    49      */
     49    /** The resulting gpx data */
    5050    private GpxData gpxData;
    51     private enum State { init, gpx, metadata, wpt, rte, trk, ext, author, link, trkseg, copyright}
    5251    private InputSource inputSource;
    5352
     
    252251             */
    253252            if (nokiaSportsTrackerBug) {
    254                 for (int i=0; i<ch.length; ++i) {
     253                for (int i = 0; i < ch.length; ++i) {
    255254                    if (ch[i] == 1) {
    256255                        ch[i] = 32;
     
    398397                    try {
    399398                        currentWayPoint.put(localName, Float.valueOf(accumulator.toString()));
    400                     } catch(Exception e) {
     399                    } catch (Exception e) {
    401400                        currentWayPoint.put(localName, new Float(0));
    402401                    }
     
    506505        public void tryToFinish() throws SAXException {
    507506            List<String> remainingElements = new ArrayList<>(elements);
    508             for (int i=remainingElements.size() - 1; i >= 0; i--) {
     507            for (int i = remainingElements.size() - 1; i >= 0; i--) {
    509508                endElement(null, remainingElements.get(i), remainingElements.get(i));
    510509            }
     
    549548                String message = e.getMessage();
    550549                if (e instanceof SAXParseException) {
    551                     SAXParseException spe = (SAXParseException)e;
     550                    SAXParseException spe = (SAXParseException) e;
    552551                    message += " " + tr("(at line {0}, column {1})", spe.getLineNumber(), spe.getColumnNumber());
    553552                }
  • trunk/src/org/openstreetmap/josm/io/MessageNotifier.java

    r8509 r8510  
    9595        } else if (!isRunning() && interval > 0 && isUserEnoughIdentified()) {
    9696            task = EXECUTOR.scheduleAtFixedRate(WORKER, 0, interval * 60, TimeUnit.SECONDS);
    97             Main.info("Message notifier active (checks every "+interval+" minute"+(interval>1?"s":"")+")");
     97            Main.info("Message notifier active (checks every "+interval+" minute"+(interval > 1 ? "s" : "")+")");
    9898        }
    9999    }
  • trunk/src/org/openstreetmap/josm/io/MissingOAuthAccessTokenException.java

    r3083 r8510  
    33
    44
    5 public class MissingOAuthAccessTokenException extends OsmTransferException{
     5public class MissingOAuthAccessTokenException extends OsmTransferException {
    66    public MissingOAuthAccessTokenException() {
    77        super();
  • trunk/src/org/openstreetmap/josm/io/MultiFetchServerObjectReader.java

    r8509 r8510  
    114114     * @throws NoSuchElementException if ds does not include an {@link OsmPrimitive} with id=<code>id</code>
    115115     */
    116     protected void remember(DataSet ds, long id, OsmPrimitiveType type) throws NoSuchElementException{
     116    protected void remember(DataSet ds, long id, OsmPrimitiveType type) throws NoSuchElementException {
    117117        CheckParameterUtil.ensureParameterNotNull(ds, "ds");
    118118        if (id <= 0) return;
     
    134134     */
    135135    public MultiFetchServerObjectReader append(DataSet ds, long id, OsmPrimitiveType type) {
    136         OsmPrimitive p = ds.getPrimitiveById(id,type);
     136        OsmPrimitive p = ds.getPrimitiveById(id, type);
    137137        switch(type) {
    138138        case NODE:
    139             return appendNode((Node)p);
     139            return appendNode((Node) p);
    140140        case WAY:
    141             return appendWay((Way)p);
     141            return appendWay((Way) p);
    142142        case RELATION:
    143             return appendRelation((Relation)p);
     143            return appendRelation((Relation) p);
    144144        }
    145145        return this;
     
    209209        if (primitive != null) {
    210210            switch (OsmPrimitiveType.from(primitive)) {
    211                 case NODE: return appendNode((Node)primitive);
    212                 case WAY: return appendWay((Way)primitive);
    213                 case RELATION: return appendRelation((Relation)primitive);
     211                case NODE: return appendNode((Node) primitive);
     212                case WAY: return appendWay((Way) primitive);
     213                case RELATION: return appendRelation((Relation) primitive);
    214214            }
    215215        }
     
    246246        if (ids.size() > MAX_IDS_PER_REQUEST) {
    247247            Iterator<Long> it = ids.iterator();
    248             for (int i=0; i<MAX_IDS_PER_REQUEST; i++) {
     248            for (int i = 0; i < MAX_IDS_PER_REQUEST; i++) {
    249249                pkg.add(it.next());
    250250            }
     
    271271
    272272        Iterator<Long> it = idPackage.iterator();
    273         for (int i=0; i<idPackage.size(); i++) {
     273        for (int i = 0; i < idPackage.size(); i++) {
    274274            sb.append(it.next());
    275275            if (i < idPackage.size()-1) {
     
    314314     */
    315315    protected void merge(DataSet from) {
    316         final DataSetMerger visitor = new DataSetMerger(outputDataSet,from);
     316        final DataSetMerger visitor = new DataSetMerger(outputDataSet, from);
    317317        visitor.merge();
    318318    }
     
    395395            missingPrimitives = new HashSet<>();
    396396            if (isCanceled()) return null;
    397             fetchPrimitives(ways,OsmPrimitiveType.WAY, progressMonitor);
     397            fetchPrimitives(ways, OsmPrimitiveType.WAY, progressMonitor);
    398398            if (isCanceled()) return null;
    399             fetchPrimitives(nodes,OsmPrimitiveType.NODE, progressMonitor);
     399            fetchPrimitives(nodes, OsmPrimitiveType.NODE, progressMonitor);
    400400            if (isCanceled()) return null;
    401             fetchPrimitives(relations,OsmPrimitiveType.RELATION, progressMonitor);
     401            fetchPrimitives(relations, OsmPrimitiveType.RELATION, progressMonitor);
    402402            if (outputDataSet != null) {
    403403                outputDataSet.deleteInvisible();
  • trunk/src/org/openstreetmap/josm/io/NmeaReader.java

    r8449 r8510  
    5959    // GPVTG
    6060    public static enum GPVTG {
    61         COURSE(1),COURSE_REF(2), // true course
     61        COURSE(1), COURSE_REF(2), // true course
    6262        COURSE_M(3), COURSE_M_REF(4), // magnetic course
    6363        SPEED_KN(5), SPEED_KN_UNIT(6), // speed in knots
     
    107107        HEIGHT(9), HEIGHT_UNTIS(10), // height above NN (above geoid)
    108108        HEIGHT_2(11), HEIGHT_2_UNTIS(12), // height geoid - height ellipsoid (WGS84)
    109         GPS_AGE(13),// Age of differential GPS data
     109        GPS_AGE(13), // Age of differential GPS data
    110110        REF(14); // REF station
    111111
     
    153153        return ps.unknown;
    154154    }
     155
    155156    public int getParserZeroCoordinates() {
    156157        return ps.zeroCoord;
    157158    }
     159
    158160    public int getParserChecksumErrors() {
    159161        return ps.checksumErrors+ps.noChecksum;
    160162    }
     163
    161164    public int getParserMalformed() {
    162165        return ps.malformed;
    163166    }
     167
    164168    public int getNumberOfCoordinates() {
    165169        return ps.success;
     
    176180            int loopstart_char = rd.read();
    177181            ps = new NMEAParserState();
    178             if(loopstart_char == -1)
     182            if (loopstart_char == -1)
    179183                //TODO tell user about the problem?
    180184                return;
    181             sb.append((char)loopstart_char);
    182             ps.pDate="010100"; // TODO date problem
    183             while(true) {
     185            sb.append((char) loopstart_char);
     186            ps.pDate = "010100"; // TODO date problem
     187            while (true) {
    184188                // don't load unparsable files completely to memory
    185                 if(sb.length()>=1020) {
     189                if (sb.length() >= 1020) {
    186190                    sb.delete(0, sb.length()-1);
    187191                }
    188192                int c = rd.read();
    189                 if(c=='$') {
     193                if (c == '$') {
    190194                    parseNMEASentence(sb.toString(), ps);
    191195                    sb.delete(0, sb.length());
    192196                    sb.append('$');
    193                 } else if(c == -1) {
     197                } else if (c == -1) {
    194198                    // EOF: add last WayPoint if it works out
    195                     parseNMEASentence(sb.toString(),ps);
     199                    parseNMEASentence(sb.toString(), ps);
    196200                    break;
    197201                } else {
    198                     sb.append((char)c);
     202                    sb.append((char) c);
    199203                }
    200204            }
     
    237241            if (chkstrings.length > 1) {
    238242                byte[] chb = chkstrings[0].getBytes(StandardCharsets.UTF_8);
    239                 int chk=0;
     243                int chk = 0;
    240244                for (int i = 1; i < chb.length; i++) {
    241245                    chk ^= chb[i];
    242246                }
    243                 if (Integer.parseInt(chkstrings[1].substring(0,2),16) != chk) {
     247                if (Integer.parseInt(chkstrings[1].substring(0, 2), 16) != chk) {
    244248                    ps.checksumErrors++;
    245                     ps.pWp=null;
     249                    ps.pWp = null;
    246250                    return false;
    247251                }
     
    257261
    258262            // handle the packet content
    259             if("$GPGGA".equals(e[0]) || "$GNGGA".equals(e[0])) {
     263            if ("$GPGGA".equals(e[0]) || "$GNGGA".equals(e[0])) {
    260264                // Position
    261265                LatLon latLon = parseLatLon(
     
    265269                        e[GPGGA.LONGITUDE.position]
    266270                );
    267                 if (latLon==null) {
     271                if (latLon == null) {
    268272                    throw new IllegalDataException("Malformed lat/lon");
    269273                }
     
    278282                Date d = readTime(currentDate+accu);
    279283
    280                 if((ps.pTime==null) || (currentwp==null) || !ps.pTime.equals(accu)) {
     284                if ((ps.pTime == null) || (currentwp == null) || !ps.pTime.equals(accu)) {
    281285                    // this node is newer than the previous, create a new waypoint.
    282                     // no matter if previous WayPoint was null, we got something
    283                     // better now.
    284                     ps.pTime=accu;
     286                    // no matter if previous WayPoint was null, we got something better now.
     287                    ps.pTime = accu;
    285288                    currentwp = new WayPoint(latLon);
    286289                }
    287                 if(!currentwp.attr.containsKey("time")) {
     290                if (!currentwp.attr.containsKey("time")) {
    288291                    // As this sentence has no complete time only use it
    289292                    // if there is no time so far
     
    291294                }
    292295                // elevation
    293                 accu=e[GPGGA.HEIGHT_UNTIS.position];
    294                 if("M".equals(accu)) {
     296                accu = e[GPGGA.HEIGHT_UNTIS.position];
     297                if ("M".equals(accu)) {
    295298                    // Ignore heights that are not in meters for now
    296                     accu=e[GPGGA.HEIGHT.position];
    297                     if(!accu.isEmpty()) {
     299                    accu = e[GPGGA.HEIGHT.position];
     300                    if (!accu.isEmpty()) {
    298301                        Double.parseDouble(accu);
    299302                        // if it throws it's malformed; this should only happen if the
    300303                        // device sends nonstandard data.
    301                         if(!accu.isEmpty()) { // FIX ? same check
     304                        if (!accu.isEmpty()) { // FIX ? same check
    302305                            currentwp.put(GpxConstants.PT_ELE, accu);
    303306                        }
     
    305308                }
    306309                // number of sattelites
    307                 accu=e[GPGGA.SATELLITE_COUNT.position];
     310                accu = e[GPGGA.SATELLITE_COUNT.position];
    308311                int sat = 0;
    309                 if(!accu.isEmpty()) {
     312                if (!accu.isEmpty()) {
    310313                    sat = Integer.parseInt(accu);
    311314                    currentwp.put(GpxConstants.PT_SAT, accu);
    312315                }
    313316                // h-dilution
    314                 accu=e[GPGGA.HDOP.position];
    315                 if(!accu.isEmpty()) {
     317                accu = e[GPGGA.HDOP.position];
     318                if (!accu.isEmpty()) {
    316319                    currentwp.put(GpxConstants.PT_HDOP, Float.valueOf(accu));
    317320                }
    318321                // fix
    319                 accu=e[GPGGA.QUALITY.position];
    320                 if(!accu.isEmpty()) {
     322                accu = e[GPGGA.QUALITY.position];
     323                if (!accu.isEmpty()) {
    321324                    int fixtype = Integer.parseInt(accu);
    322325                    switch(fixtype) {
     
    325328                        break;
    326329                    case 1:
    327                         if(sat < 4) {
     330                        if (sat < 4) {
    328331                            currentwp.put(GpxConstants.PT_FIX, "2d");
    329332                        } else {
     
    338341                    }
    339342                }
    340             } else if("$GPVTG".equals(e[0]) || "$GNVTG".equals(e[0])) {
     343            } else if ("$GPVTG".equals(e[0]) || "$GNVTG".equals(e[0])) {
    341344                // COURSE
    342345                accu = e[GPVTG.COURSE_REF.position];
    343                 if("T".equals(accu)) {
     346                if ("T".equals(accu)) {
    344347                    // other values than (T)rue are ignored
    345348                    accu = e[GPVTG.COURSE.position];
    346                     if(!accu.isEmpty()) {
     349                    if (!accu.isEmpty()) {
    347350                        Double.parseDouble(accu);
    348351                        currentwp.put("course", accu);
     
    351354                // SPEED
    352355                accu = e[GPVTG.SPEED_KMH_UNIT.position];
    353                 if(accu.startsWith("K")) {
     356                if (accu.startsWith("K")) {
    354357                    accu = e[GPVTG.SPEED_KMH.position];
    355                     if(!accu.isEmpty()) {
     358                    if (!accu.isEmpty()) {
    356359                        double speed = Double.parseDouble(accu);
    357360                        speed /= 3.6; // speed in m/s
     
    359362                    }
    360363                }
    361             } else if("$GPGSA".equals(e[0]) || "$GNGSA".equals(e[0])) {
     364            } else if ("$GPGSA".equals(e[0]) || "$GNGSA".equals(e[0])) {
    362365                // vdop
    363                 accu=e[GPGSA.VDOP.position];
    364                 if(!accu.isEmpty()) {
     366                accu = e[GPGSA.VDOP.position];
     367                if (!accu.isEmpty()) {
    365368                    currentwp.put(GpxConstants.PT_VDOP, Float.valueOf(accu));
    366369                }
    367370                // hdop
    368                 accu=e[GPGSA.HDOP.position];
    369                 if(!accu.isEmpty()) {
     371                accu = e[GPGSA.HDOP.position];
     372                if (!accu.isEmpty()) {
    370373                    currentwp.put(GpxConstants.PT_HDOP, Float.valueOf(accu));
    371374                }
    372375                // pdop
    373                 accu=e[GPGSA.PDOP.position];
    374                 if(!accu.isEmpty()) {
     376                accu = e[GPGSA.PDOP.position];
     377                if (!accu.isEmpty()) {
    375378                    currentwp.put(GpxConstants.PT_PDOP, Float.valueOf(accu));
    376379                }
    377             } else if("$GPRMC".equals(e[0]) || "$GNRMC".equals(e[0])) {
     380            } else if ("$GPRMC".equals(e[0]) || "$GNRMC".equals(e[0])) {
    378381                // coordinates
    379382                LatLon latLon = parseLatLon(
     
    393396                Date d = readTime(currentDate+time);
    394397
    395                 if(ps.pTime==null || currentwp==null || !ps.pTime.equals(time)) {
     398                if (ps.pTime == null || currentwp == null || !ps.pTime.equals(time)) {
    396399                    // this node is newer than the previous, create a new waypoint.
    397                     ps.pTime=time;
     400                    ps.pTime = time;
    398401                    currentwp = new WayPoint(latLon);
    399402                }
     
    402405                // speed
    403406                accu = e[GPRMC.SPEED.position];
    404                 if(!accu.isEmpty() && !currentwp.attr.containsKey("speed")) {
     407                if (!accu.isEmpty() && !currentwp.attr.containsKey("speed")) {
    405408                    double speed = Double.parseDouble(accu);
    406409                    speed *= 0.514444444; // to m/s
     
    409412                // course
    410413                accu = e[GPRMC.COURSE.position];
    411                 if(!accu.isEmpty() && !currentwp.attr.containsKey("course")) {
     414                if (!accu.isEmpty() && !currentwp.attr.containsKey("course")) {
    412415                    Double.parseDouble(accu);
    413416                    currentwp.put("course", accu);
     
    426429            }
    427430            ps.pDate = currentDate;
    428             if(ps.pWp != currentwp) {
    429                 if(ps.pWp!=null) {
     431            if (ps.pWp != currentwp) {
     432                if (ps.pWp != null) {
    430433                    ps.pWp.setTime();
    431434                }
     
    440443            // out of bounds and such
    441444            ps.malformed++;
    442             ps.pWp=null;
     445            ps.pWp = null;
    443446            return false;
    444447        }
     
    452455        // return a zero latlon instead of null so it is logged as zero coordinate
    453456        // instead of malformed sentence
    454         if(widthNorth.isEmpty() && lengthEast.isEmpty()) return new LatLon(0.0,0.0);
     457        if (widthNorth.isEmpty() && lengthEast.isEmpty()) return new LatLon(0.0, 0.0);
    455458
    456459        // The format is xxDDLL.LLLL
     
    463466        int latdeg = Integer.parseInt(widthNorth.substring(0, latdegsep));
    464467        double latmin = Double.parseDouble(widthNorth.substring(latdegsep));
    465         if(latdeg < 0) {
     468        if (latdeg < 0) {
    466469            latmin *= -1.0;
    467470        }
     
    476479        int londeg = Integer.parseInt(lengthEast.substring(0, londegsep));
    477480        double lonmin = Double.parseDouble(lengthEast.substring(londegsep));
    478         if(londeg < 0) {
     481        if (londeg < 0) {
    479482            lonmin *= -1.0;
    480483        }
  • trunk/src/org/openstreetmap/josm/io/NoteReader.java

    r8470 r8510  
    7979
    8080            if (parseMode == NoteParseMode.API) {
    81                 if("note".equals(qName)) {
     81                if ("note".equals(qName)) {
    8282                    double lat = Double.parseDouble(attrs.getValue("lat"));
    8383                    double lon = Double.parseDouble(attrs.getValue("lon"));
     
    9797                thisNote.setId(Long.parseLong(attrs.getValue("id")));
    9898                String closedTimeStr = attrs.getValue("closed_at");
    99                 if(closedTimeStr == null) { //no closed_at means the note is still open
     99                if (closedTimeStr == null) { //no closed_at means the note is still open
    100100                    thisNote.setState(Note.State.open);
    101101                } else {
     
    107107            case "comment":
    108108                String uidStr = attrs.getValue("uid");
    109                 if(uidStr == null) {
     109                if (uidStr == null) {
    110110                    commentUid = 0;
    111111                } else {
     
    116116                commentCreateDate = DateUtils.fromString(attrs.getValue("timestamp"));
    117117                String isNew = attrs.getValue("is_new");
    118                 if(isNew == null) {
     118                if (isNew == null) {
    119119                    commentIsNew = false;
    120120                } else {
     
    127127        @Override
    128128        public void endElement(String namespaceURI, String localName, String qName) {
    129             if("note".equals(qName)) {
     129            if ("note".equals(qName)) {
    130130                notes.add(thisNote);
    131131            }
    132             if("comment".equals(qName)) {
     132            if ("comment".equals(qName)) {
    133133                User commentUser = User.createOsmUser(commentUid, commentUsername);
    134134                if (commentUid == 0) {
    135135                    commentUser = User.getAnonymous();
    136136                }
    137                 if(parseMode == NoteParseMode.API) {
     137                if (parseMode == NoteParseMode.API) {
    138138                    commentIsNew = false;
    139139                }
    140                 if(parseMode == NoteParseMode.DUMP) {
     140                if (parseMode == NoteParseMode.DUMP) {
    141141                    commentText = buffer.toString();
    142142                }
     
    148148                commentText = null;
    149149            }
    150             if(parseMode == NoteParseMode.DUMP) {
     150            if (parseMode == NoteParseMode.DUMP) {
    151151                return;
    152152            }
  • trunk/src/org/openstreetmap/josm/io/OsmApi.java

    r8470 r8510  
    9595        if (api == null) {
    9696            api = new OsmApi(serverUrl);
    97             instances.put(serverUrl,api);
     97            instances.put(serverUrl, api);
    9898        }
    9999        return api;
     
    357357            ensureValidChangeset();
    358358            initialize(monitor);
    359             ret = sendRequest("PUT", OsmPrimitiveType.from(osm).getAPIName()+"/create", toXml(osm, true),monitor);
     359            ret = sendRequest("PUT", OsmPrimitiveType.from(osm).getAPIName()+"/create", toXml(osm, true), monitor);
    360360            osm.setOsmId(Long.parseLong(ret.trim()), 1);
    361361            osm.setChangesetId(getChangeset().getId());
    362         } catch(NumberFormatException e){
     362        } catch (NumberFormatException e) {
    363363            throw new OsmTransferException(tr("Unexpected format of ID replied by the server. Got ''{0}''.", ret), e);
    364364        }
     
    382382            osm.setChangesetId(getChangeset().getId());
    383383            osm.setVisible(true);
    384         } catch(NumberFormatException e) {
     384        } catch (NumberFormatException e) {
    385385            throw new OsmTransferException(tr("Unexpected format of new version of modified primitive ''{0}''. Got ''{1}''.",
    386386                    osm.getId(), ret), e);
     
    423423            String ret = "";
    424424            try {
    425                 ret = sendRequest("PUT", "changeset/create", toXml(changeset),progressMonitor);
     425                ret = sendRequest("PUT", "changeset/create", toXml(changeset), progressMonitor);
    426426                changeset.setId(Integer.parseInt(ret.trim()));
    427427                changeset.setOpen(true);
    428             } catch(NumberFormatException e){
     428            } catch (NumberFormatException e) {
    429429                throw new OsmTransferException(tr("Unexpected format of ID replied by the server. Got ''{0}''.", ret), e);
    430430            }
    431             progressMonitor.setCustomText(tr("Successfully opened changeset {0}",changeset.getId()));
     431            progressMonitor.setCustomText(tr("Successfully opened changeset {0}", changeset.getId()));
    432432        } finally {
    433433            progressMonitor.finishTask();
     
    464464                    monitor
    465465            );
    466         } catch(ChangesetClosedException e) {
     466        } catch (ChangesetClosedException e) {
    467467            e.setSource(ChangesetClosedException.Source.UPDATE_CHANGESET);
    468468            throw e;
    469         } catch(OsmApiException e) {
     469        } catch (OsmApiException e) {
    470470            String errorHeader = e.getErrorHeader();
    471471            if (e.getResponseCode() == HttpURLConnection.HTTP_CONFLICT && ChangesetClosedException.errorHeaderMatchesPattern(errorHeader))
     
    536536            monitor.indeterminateSubTask(
    537537                    trn("Uploading {0} object...", "Uploading {0} objects...", list.size(), list.size()));
    538             String diffUploadResponse = sendRequest("POST", "changeset/" + changeset.getId() + "/upload", diffUploadRequest,monitor);
     538            String diffUploadResponse = sendRequest("POST", "changeset/" + changeset.getId() + "/upload", diffUploadRequest, monitor);
    539539
    540540            // Process the response from the server
     
    546546                    monitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false)
    547547            );
    548         } catch(OsmTransferException e) {
     548        } catch (OsmTransferException e) {
    549549            throw e;
    550         } catch(XmlParsingException e) {
     550        } catch (XmlParsingException e) {
    551551            throw new OsmTransferException(e);
    552552        } finally {
     
    557557    private void sleepAndListen(int retry, ProgressMonitor monitor) throws OsmTransferCanceledException {
    558558        Main.info(tr("Waiting 10 seconds ... "));
    559         for (int i=0; i < 10; i++) {
     559        for (int i = 0; i < 10; i++) {
    560560            if (monitor != null) {
    561                 monitor.setCustomText(tr("Starting retry {0} of {1} in {2} seconds ...", getMaxRetries() - retry,getMaxRetries(), 10-i));
     561                monitor.setCustomText(tr("Starting retry {0} of {1} in {2} seconds ...", getMaxRetries() - retry, getMaxRetries(), 10-i));
    562562            }
    563563            if (cancel)
     
    579579    protected int getMaxRetries() {
    580580        int ret = Main.pref.getInteger("osm-server.max-num-retries", DEFAULT_MAX_NUM_RETRIES);
    581         return Math.max(ret,0);
     581        return Math.max(ret, 0);
    582582    }
    583583
     
    591591    }
    592592
    593     protected final String sendRequest(String requestMethod, String urlSuffix,String requestBody, ProgressMonitor monitor)
     593    protected final String sendRequest(String requestMethod, String urlSuffix, String requestBody, ProgressMonitor monitor)
    594594            throws OsmTransferException {
    595595        return sendRequest(requestMethod, urlSuffix, requestBody, monitor, true, false);
     
    615615     *    been exhausted), or rewrapping a Java exception.
    616616     */
    617     protected final String sendRequest(String requestMethod, String urlSuffix,String requestBody, ProgressMonitor monitor,
     617    protected final String sendRequest(String requestMethod, String urlSuffix, String requestBody, ProgressMonitor monitor,
    618618            boolean doAuthenticate, boolean fastFail) throws OsmTransferException {
    619619        StringBuilder responseBody = new StringBuilder();
    620620        int retries = fastFail ? 0 : getMaxRetries();
    621621
    622         while(true) { // the retry loop
     622        while (true) { // the retry loop
    623623            try {
    624624                url = new URL(new URL(getBaseUrl()), urlSuffix);
     
    627627                // fix #5369, see http://www.tikalk.com/java/forums/httpurlconnection-disable-keep-alive
    628628                activeConnection = Utils.openHttpConnection(url, false);
    629                 activeConnection.setConnectTimeout(fastFail ? 1000 : Main.pref.getInteger("socket.timeout.connect",15)*1000);
     629                activeConnection.setConnectTimeout(fastFail ? 1000 : Main.pref.getInteger("socket.timeout.connect", 15)*1000);
    630630                if (fastFail) {
    631631                    activeConnection.setReadTimeout(1000);
     
    662662                    if (retries-- > 0) {
    663663                        sleepAndListen(retries, monitor);
    664                         Main.info(tr("Starting retry {0} of {1}.", getMaxRetries() - retries,getMaxRetries()));
     664                        Main.info(tr("Starting retry {0} of {1}.", getMaxRetries() - retries, getMaxRetries()));
    665665                        continue;
    666666                    }
     
    670670                responseBody.setLength(0);
    671671
    672                 // If the API returned an error code like 403 forbidden, getInputStream
    673                 // will fail with an IOException.
     672                // If the API returned an error code like 403 forbidden, getInputStream will fail with an IOException.
    674673                InputStream i = getConnectionStream();
    675674                if (i != null) {
    676675                    // the input stream can be null if both the input and the error stream
    677                     // are null. Seems to be the case if the OSM server replies a 401
    678                     // Unauthorized, see #3887.
    679                     //
     676                    // are null. Seems to be the case if the OSM server replies a 401 Unauthorized, see #3887.
    680677                    String s;
    681678                    try (BufferedReader in = new BufferedReader(new InputStreamReader(i, StandardCharsets.UTF_8))) {
    682                         while((s = in.readLine()) != null) {
     679                        while ((s = in.readLine()) != null) {
    683680                            responseBody.append(s);
    684681                            responseBody.append('\n');
     
    691688                    errorHeader = activeConnection.getHeaderField("Error");
    692689                    Main.error("Error header: " + errorHeader);
    693                 } else if (retCode != HttpURLConnection.HTTP_OK && responseBody.length()>0) {
     690                } else if (retCode != HttpURLConnection.HTTP_OK && responseBody.length() > 0) {
    694691                    Main.error("Error body: " + responseBody);
    695692                }
     
    700697                }
    701698
    702                 errorHeader = errorHeader == null? null : errorHeader.trim();
    703                 String errorBody = responseBody.length() == 0? null : responseBody.toString().trim();
     699                errorHeader = errorHeader == null ? null : errorHeader.trim();
     700                String errorBody = responseBody.length() == 0 ? null : responseBody.toString().trim();
    704701                switch(retCode) {
    705702                case HttpURLConnection.HTTP_OK:
     
    724721                }
    725722                throw new OsmTransferException(e);
    726             } catch(IOException e) {
     723            } catch (IOException e) {
    727724                throw new OsmTransferException(e);
    728             } catch(OsmTransferException e) {
     725            } catch (OsmTransferException e) {
    729726                throw e;
    730727            }
     
    884881        try {
    885882            List<Note> newNotes = new NoteReader(xml).parse();
    886             if(newNotes.size() == 1) {
     883            if (newNotes.size() == 1) {
    887884                return newNotes.get(0);
    888885            }
    889886            //Shouldn't ever execute. Server will either respond with an error (caught elsewhere) or one note
    890887            throw new OsmTransferException(tr("Note upload failed"));
    891         } catch (SAXException|IOException e) {
     888        } catch (SAXException | IOException e) {
    892889            Main.error(e, true);
    893890            throw new OsmTransferException(tr("Error parsing note response from server"), e);
  • trunk/src/org/openstreetmap/josm/io/OsmApiException.java

    r8509 r8510  
    135135        String eh = "";
    136136        try {
    137             if(errorHeader != null)
     137            if (errorHeader != null)
    138138                eh = tr(errorHeader.trim());
    139139            if (!eh.isEmpty()) {
  • trunk/src/org/openstreetmap/josm/io/OsmApiPrimitiveGoneException.java

    r6889 r8510  
    1313 *
    1414 */
    15 public class OsmApiPrimitiveGoneException extends OsmApiException{
     15public class OsmApiPrimitiveGoneException extends OsmApiException {
    1616    /**
    1717     * The regexp pattern for the error header replied by the OSM API
  • trunk/src/org/openstreetmap/josm/io/OsmChangeBuilder.java

    r8291 r8510  
    5252
    5353    private void switchMode(String newMode) {
    54         if ((newMode != null && !newMode.equals(currentMode))||(newMode == null && currentMode != null)) {
     54        if ((newMode != null && !newMode.equals(currentMode)) || (newMode == null && currentMode != null)) {
    5555            if (currentMode != null) {
    5656                writer.print("</");
     
    7878        writer.print(apiVersion);
    7979        writer.println("\" generator=\"JOSM\">");
    80         prologWritten=true;
     80        prologWritten = true;
    8181    }
    8282
  • trunk/src/org/openstreetmap/josm/io/OsmChangesetContentParser.java

    r8347 r8510  
    152152            progressMonitor.indeterminateSubTask(tr("Parsing changeset content ..."));
    153153            Utils.parseSafeSAX(source, new Parser());
    154         } catch(XmlParsingException e) {
     154        } catch (XmlParsingException e) {
    155155            throw e;
    156156        } catch (ParserConfigurationException | SAXException | IOException e) {
  • trunk/src/org/openstreetmap/josm/io/OsmChangesetParser.java

    r8509 r8510  
    127127                try {
    128128                    minLon = Double.parseDouble(min_lon);
    129                 } catch(NumberFormatException e) {
     129                } catch (NumberFormatException e) {
    130130                    throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "min_lon", min_lon));
    131131                }
     
    133133                try {
    134134                    minLat = Double.parseDouble(min_lat);
    135                 } catch(NumberFormatException e) {
     135                } catch (NumberFormatException e) {
    136136                    throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "min_lat", min_lat));
    137137                }
     
    143143                try {
    144144                    maxLon = Double.parseDouble(max_lon);
    145                 } catch(NumberFormatException e) {
     145                } catch (NumberFormatException e) {
    146146                    throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "max_lon", max_lon));
    147147                }
     
    149149                try {
    150150                    maxLat = Double.parseDouble(max_lat);
    151                 } catch(NumberFormatException e) {
     151                } catch (NumberFormatException e) {
    152152                    throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "max_lat", max_lat));
    153153                }
     
    177177            try {
    178178                att = Integer.parseInt(value);
    179             } catch(NumberFormatException e) {
     179            } catch (NumberFormatException e) {
    180180                throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "id", value));
    181181            }
     
    256256                long id = Long.parseLong(uid);
    257257                return User.createOsmUser(id, name);
    258             } catch(NumberFormatException e) {
     258            } catch (NumberFormatException e) {
    259259                throwException(MessageFormat.format("Illegal value for attribute ''uid''. Got ''{0}''.", uid));
    260260            }
     
    281281            Utils.parseSafeSAX(inputSource, parser.new Parser());
    282282            return parser.getChangesets();
    283         } catch(ParserConfigurationException | SAXException e) {
     283        } catch (ParserConfigurationException | SAXException e) {
    284284            throw new IllegalDataException(e.getMessage(), e);
    285         } catch(Exception e) {
     285        } catch (Exception e) {
    286286            throw new IllegalDataException(e);
    287287        } finally {
  • trunk/src/org/openstreetmap/josm/io/OsmConnection.java

    r8443 r8510  
    9393            return;
    9494        } else {
    95             String username= response.getUsername() == null ? "" : response.getUsername();
     95            String username = response.getUsername() == null ? "" : response.getUsername();
    9696            String password = response.getPassword() == null ? "" : String.valueOf(response.getPassword());
    9797            token = username + ":" + password;
     
    9999                ByteBuffer bytes = encoder.encode(CharBuffer.wrap(token));
    100100                con.addRequestProperty("Authorization", "Basic "+Base64.encode(bytes));
    101             } catch(CharacterCodingException e) {
     101            } catch (CharacterCodingException e) {
    102102                throw new OsmTransferException(e);
    103103            }
     
    124124        try {
    125125            consumer.sign(connection);
    126         } catch(OAuthException e) {
     126        } catch (OAuthException e) {
    127127            throw new OsmTransferException(tr("Failed to sign a HTTP connection with an OAuth Authentication header"), e);
    128128        }
  • trunk/src/org/openstreetmap/josm/io/OsmReader.java

    r8509 r8510  
    189189            ds.dataSources.add(src);
    190190        } else {
    191             throwException(tr(
    192                     "Missing mandatory attributes on element ''bounds''. Got minlon=''{0}'',minlat=''{1}'',maxlon=''{3}'',maxlat=''{4}'', origin=''{5}''.",
     191            throwException(tr("Missing mandatory attributes on element ''bounds''. " +
     192                    "Got minlon=''{0}'',minlat=''{1}'',maxlon=''{3}'',maxlat=''{4}'', origin=''{5}''.",
    193193                    minlon, minlat, maxlon, maxlat, origin
    194194            ));
     
    312312        String value = parser.getAttributeValue(null, "ref");
    313313        if (value == null) {
    314             throwException(tr("Missing attribute ''ref'' on member in relation {0}.",r.getUniqueId()));
     314            throwException(tr("Missing attribute ''ref'' on member in relation {0}.", r.getUniqueId()));
    315315        }
    316316        try {
    317317            id = Long.parseLong(value);
    318         } catch(NumberFormatException e) {
    319             throwException(tr("Illegal value for attribute ''ref'' on member in relation {0}. Got {1}", Long.toString(r.getUniqueId()),value), e);
     318        } catch (NumberFormatException e) {
     319            throwException(tr("Illegal value for attribute ''ref'' on member in relation {0}. Got {1}", Long.toString(r.getUniqueId()), value), e);
    320320        }
    321321        value = parser.getAttributeValue(null, "type");
     
    325325        try {
    326326            type = OsmPrimitiveType.fromApiTypeName(value);
    327         } catch(IllegalArgumentException e) {
     327        } catch (IllegalArgumentException e) {
    328328            throwException(tr("Illegal value for attribute ''type'' on member {0} in relation {1}. Got {2}.",
    329329                    Long.toString(id), Long.toString(r.getUniqueId()), value), e);
     
    421421            long id = Long.parseLong(uid);
    422422            return User.createOsmUser(id, name);
    423         } catch(NumberFormatException e) {
     423        } catch (NumberFormatException e) {
    424424            throwException(MessageFormat.format("Illegal value for attribute ''uid''. Got ''{0}''.", uid), e);
    425425        }
     
    455455            try {
    456456                version = Integer.parseInt(versionString);
    457             } catch(NumberFormatException e) {
     457            } catch (NumberFormatException e) {
    458458                throwException(tr("Illegal value for attribute ''version'' on OSM primitive with ID {0}. Got {1}.",
    459459                        Long.toString(current.getUniqueId()), versionString), e);
     
    529529        String value = parser.getAttributeValue(null, name);
    530530        if (value == null) {
    531             throwException(tr("Missing required attribute ''{0}''.",name));
     531            throwException(tr("Missing required attribute ''{0}''.", name));
    532532        }
    533533        try {
    534534            return Long.parseLong(value);
    535         } catch(NumberFormatException e) {
    536             throwException(tr("Illegal long value for attribute ''{0}''. Got ''{1}''.",name, value), e);
     535        } catch (NumberFormatException e) {
     536            throwException(tr("Illegal long value for attribute ''{0}''. Got ''{1}''.", name, value), e);
    537537        }
    538538        return 0; // should not happen
     
    616616            }
    617617            return getDataSet();
    618         } catch(IllegalDataException e) {
     618        } catch (IllegalDataException e) {
    619619            throw e;
    620         } catch(OsmParsingException e) {
     620        } catch (OsmParsingException e) {
    621621            throw new IllegalDataException(e.getMessage(), e);
    622         } catch(XMLStreamException e) {
     622        } catch (XMLStreamException e) {
    623623            String msg = e.getMessage();
    624624            Pattern p = Pattern.compile("Message: (.+)");
     
    632632            else
    633633                throw new IllegalDataException(msg, e);
    634         } catch(Exception e) {
     634        } catch (Exception e) {
    635635            throw new IllegalDataException(e);
    636636        } finally {
  • trunk/src/org/openstreetmap/josm/io/OsmServerBackreferenceReader.java

    r8509 r8510  
    133133                return OsmReader.parseDataSet(in, progressMonitor.createSubTaskMonitor(1, true));
    134134            }
    135         } catch(OsmTransferException e) {
     135        } catch (OsmTransferException e) {
    136136            throw e;
    137137        } catch (Exception e) {
     
    189189        try {
    190190            Collection<Way> waysToCheck = new ArrayList<>(ds.getWays());
    191             if (isReadFull() ||primitiveType.equals(OsmPrimitiveType.NODE)) {
     191            if (isReadFull() || primitiveType.equals(OsmPrimitiveType.NODE)) {
    192192                for (Way way: waysToCheck) {
    193193                    if (!way.isNew() && way.hasIncompleteNodes()) {
     
    234234            if (primitiveType.equals(OsmPrimitiveType.NODE)) {
    235235                DataSet ds = getReferringWays(progressMonitor.createSubTaskMonitor(1, false));
    236                 DataSetMerger visitor = new DataSetMerger(ret,ds);
     236                DataSetMerger visitor = new DataSetMerger(ret, ds);
    237237                visitor.merge();
    238238                ret = visitor.getTargetDataSet();
    239239            }
    240240            DataSet ds = getReferringRelations(progressMonitor.createSubTaskMonitor(1, false));
    241             DataSetMerger visitor = new DataSetMerger(ret,ds);
     241            DataSetMerger visitor = new DataSetMerger(ret, ds);
    242242            visitor.merge();
    243243            ret = visitor.getTargetDataSet();
  • trunk/src/org/openstreetmap/josm/io/OsmServerChangesetReader.java

    r8291 r8510  
    7979                Main.warn(e);
    8080            }
    81         } catch(OsmTransferException e) {
     81        } catch (OsmTransferException e) {
    8282            throw e;
    83         } catch(IllegalDataException e) {
     83        } catch (IllegalDataException e) {
    8484            throw new OsmTransferException(e);
    8585        } finally {
     
    108108        Changeset result = null;
    109109        try {
    110             monitor.beginTask(tr("Reading changeset {0} ...",id));
     110            monitor.beginTask(tr("Reading changeset {0} ...", id));
    111111            try (InputStream in = getChangesetInputStream(id, includeDiscussion, monitor)) {
    112112                if (in == null)
     
    120120                Main.warn(e);
    121121            }
    122         } catch(OsmTransferException e) {
     122        } catch (OsmTransferException e) {
    123123            throw e;
    124         } catch(IllegalDataException e) {
     124        } catch (IllegalDataException e) {
    125125            throw new OsmTransferException(e);
    126126        } finally {
     
    148148        }
    149149        try {
    150             monitor.beginTask(trn("Downloading {0} changeset ...", "Downloading {0} changesets ...",ids.size(),ids.size()));
     150            monitor.beginTask(trn("Downloading {0} changeset ...", "Downloading {0} changesets ...", ids.size(), ids.size()));
    151151            monitor.setTicksCount(ids.size());
    152152            List<Changeset> ret = new ArrayList<>();
    153             int i=0;
     153            int i = 0;
    154154            for (int id : ids) {
    155155                if (id <= 0) {
     
    172172            }
    173173            return ret;
    174         } catch(OsmTransferException e) {
     174        } catch (OsmTransferException e) {
    175175            throw e;
    176         } catch(IllegalDataException e) {
     176        } catch (IllegalDataException e) {
    177177            throw new OsmTransferException(e);
    178178        } finally {
     
    210210                Main.warn(e);
    211211            }
    212         } catch(XmlParsingException e) {
     212        } catch (XmlParsingException e) {
    213213            throw new OsmTransferException(e);
    214214        } finally {
  • trunk/src/org/openstreetmap/josm/io/OsmServerHistoryReader.java

    r8390 r8510  
    6969                return reader.parse(progressMonitor.createSubTaskMonitor(1, true));
    7070            }
    71         } catch(OsmTransferException e) {
     71        } catch (OsmTransferException e) {
    7272            throw e;
    7373        } catch (Exception e) {
  • trunk/src/org/openstreetmap/josm/io/OsmServerLocationReader.java

    r8378 r8510  
    5050        try {
    5151            return parser.parse();
    52         } catch(OsmTransferException e) {
     52        } catch (OsmTransferException e) {
    5353            throw e;
    5454        } catch (Exception e) {
  • trunk/src/org/openstreetmap/josm/io/OsmServerObjectReader.java

    r8470 r8510  
    131131                return OsmReader.parseDataSet(in, progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false));
    132132            }
    133         } catch(OsmTransferException e) {
     133        } catch (OsmTransferException e) {
    134134            if (cancel) return null;
    135135            throw e;
  • trunk/src/org/openstreetmap/josm/io/OsmServerReader.java

    r8509 r8510  
    126126            try {
    127127                url = new URL(urlStr.replace(" ", "%20"));
    128             } catch(MalformedURLException e) {
     128            } catch (MalformedURLException e) {
    129129                throw new OsmTransferException(e);
    130130            }
     
    132132                // fix #7640, see http://www.tikalk.com/java/forums/httpurlconnection-disable-keep-alive
    133133                activeConnection = Utils.openHttpConnection(url, false);
    134             } catch(Exception e) {
     134            } catch (Exception e) {
    135135                throw new OsmTransferException(tr("Failed to open connection to API {0}.", url.toExternalForm()), e);
    136136            }
     
    149149            }
    150150
    151             activeConnection.setConnectTimeout(Main.pref.getInteger("socket.timeout.connect",15)*1000);
     151            activeConnection.setConnectTimeout(Main.pref.getInteger("socket.timeout.connect", 15)*1000);
    152152
    153153            try {
     
    170170                }
    171171                if (activeConnection.getResponseCode() == HttpURLConnection.HTTP_UNAUTHORIZED)
    172                     throw new OsmApiException(HttpURLConnection.HTTP_UNAUTHORIZED,null,null);
     172                    throw new OsmApiException(HttpURLConnection.HTTP_UNAUTHORIZED, null, null);
    173173
    174174                if (activeConnection.getResponseCode() == HttpURLConnection.HTTP_PROXY_AUTH)
     
    184184                            BufferedReader in = new BufferedReader(new InputStreamReader(i, StandardCharsets.UTF_8));
    185185                            String s;
    186                             while((s = in.readLine()) != null) {
     186                            while ((s = in.readLine()) != null) {
    187187                                errorBody.append(s);
    188188                                errorBody.append('\n');
    189189                            }
    190190                        }
    191                     } catch(Exception e) {
     191                    } catch (Exception e) {
    192192                        errorBody.append(tr("Reading error text failed."));
    193193                    }
  • trunk/src/org/openstreetmap/josm/io/OsmServerUserInfoReader.java

    r8443 r8510  
    4141            XPath xpath = factory.newXPath();
    4242            UserInfo userInfo = new UserInfo();
    43             Node xmlNode = (Node)xpath.compile("/osm/user[1]").evaluate(document, XPathConstants.NODE);
     43            Node xmlNode = (Node) xpath.compile("/osm/user[1]").evaluate(document, XPathConstants.NODE);
    4444            if (xmlNode == null)
    4545                throw new XmlParsingException(tr("XML tag <user> is missing."));
     
    5151            try {
    5252                userInfo.setId(Integer.parseInt(v));
    53             } catch(NumberFormatException e) {
     53            } catch (NumberFormatException e) {
    5454                throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.", "id", "user", v), e);
    5555            }
     
    5959            // -- account_created
    6060            v = getAttribute(xmlNode, "account_created");
    61             if (v!=null) {
     61            if (v != null) {
    6262                userInfo.setAccountCreated(DateUtils.fromString(v));
    6363            }
    6464            // -- description
    65             xmlNode = (Node)xpath.compile("/osm/user[1]/description[1]/text()").evaluate(document, XPathConstants.NODE);
     65            xmlNode = (Node) xpath.compile("/osm/user[1]/description[1]/text()").evaluate(document, XPathConstants.NODE);
    6666            if (xmlNode != null) {
    6767                userInfo.setDescription(xmlNode.getNodeValue());
    6868            }
    6969            // -- home
    70             xmlNode = (Node)xpath.compile("/osm/user[1]/home").evaluate(document, XPathConstants.NODE);
     70            xmlNode = (Node) xpath.compile("/osm/user[1]/home").evaluate(document, XPathConstants.NODE);
    7171            if (xmlNode != null) {
    7272                v = getAttribute(xmlNode, "lat");
     
    7676                try {
    7777                    lat = Double.parseDouble(v);
    78                 } catch(NumberFormatException e) {
     78                } catch (NumberFormatException e) {
    7979                    throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.",
    8080                            "lat", "home", v), e);
     
    8787                try {
    8888                    lon = Double.parseDouble(v);
    89                 } catch(NumberFormatException e) {
     89                } catch (NumberFormatException e) {
    9090                    throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.",
    9191                            "lon", "home", v), e);
     
    9898                try {
    9999                    zoom = Integer.parseInt(v);
    100                 } catch(NumberFormatException e) {
     100                } catch (NumberFormatException e) {
    101101                    throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.",
    102102                            "zoom", "home", v), e);
    103103                }
    104                 userInfo.setHome(new LatLon(lat,lon));
     104                userInfo.setHome(new LatLon(lat, lon));
    105105                userInfo.setHomeZoom(zoom);
    106106            }
    107107
    108108            // -- language list
    109             NodeList xmlNodeList = (NodeList)xpath.compile("/osm/user[1]/languages[1]/lang/text()").evaluate(document, XPathConstants.NODESET);
     109            NodeList xmlNodeList = (NodeList) xpath.compile("/osm/user[1]/languages[1]/lang/text()").evaluate(document, XPathConstants.NODESET);
    110110            if (xmlNodeList != null) {
    111111                List<String> languages = new LinkedList<>();
    112                 for (int i=0; i < xmlNodeList.getLength(); i++) {
     112                for (int i = 0; i < xmlNodeList.getLength(); i++) {
    113113                    languages.add(xmlNodeList.item(i).getNodeValue());
    114114                }
     
    117117
    118118            // -- messages
    119             xmlNode = (Node)xpath.compile("/osm/user[1]/messages/received").evaluate(document, XPathConstants.NODE);
     119            xmlNode = (Node) xpath.compile("/osm/user[1]/messages/received").evaluate(document, XPathConstants.NODE);
    120120            if (xmlNode != null) {
    121121                v = getAttribute(xmlNode, "unread");
     
    124124                try {
    125125                    userInfo.setUnreadMessages(Integer.parseInt(v));
    126                 } catch(NumberFormatException e) {
     126                } catch (NumberFormatException e) {
    127127                    throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.", "unread", "received", v), e);
    128128                }
     
    130130
    131131            return userInfo;
    132         } catch(XPathException e) {
     132        } catch (XPathException e) {
    133133            throw new XmlParsingException(e);
    134134        }
     
    175175                );
    176176            }
    177         } catch(OsmTransferException e) {
     177        } catch (OsmTransferException e) {
    178178            throw e;
    179         } catch(Exception e) {
     179        } catch (Exception e) {
    180180            throw new OsmTransferException(e);
    181181        } finally {
  • trunk/src/org/openstreetmap/josm/io/OsmServerWriter.java

    r8509 r8510  
    4444        postprocessors.add(pp);
    4545    }
     46
    4647    public static void unregisterPostprocessor(OsmServerWritePostprocessor pp) {
    4748        if (postprocessors != null) {
     
    6566            elapsed = 1;
    6667        }
    67         double uploads_per_ms = (double)progress / elapsed;
     68        double uploads_per_ms = (double) progress / elapsed;
    6869        double uploads_left = list_size - progress;
    69         long ms_left = (long)(uploads_left / uploads_per_ms);
     70        long ms_left = (long) (uploads_left / uploads_per_ms);
    7071        long minutes_left = ms_left / MSECS_PER_MINUTE;
    7172        long seconds_left = (ms_left / MSECS_PER_SECOND) % SECONDS_PER_MINUTE;
     
    107108                                osm.getName() == null ? osm.getId() : osm.getName(),
    108109                                        osm.getId()));
    109                 makeApiRequest(osm,progressMonitor);
     110                makeApiRequest(osm, progressMonitor);
    110111                processed.add(osm);
    111112                progressMonitor.worked(1);
    112113            }
    113         } catch(OsmTransferException e) {
    114             throw e;
    115         } catch(Exception e) {
     114        } catch (OsmTransferException e) {
     115            throw e;
     116        } catch (Exception e) {
    116117            throw new OsmTransferException(e);
    117118        } finally {
     
    132133            progressMonitor.beginTask(tr("Starting to upload in one request ..."));
    133134            processed.addAll(api.uploadDiff(primitives, progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false)));
    134         } catch(OsmTransferException e) {
     135        } catch (OsmTransferException e) {
    135136            throw e;
    136137        } finally {
     
    150151    protected void uploadChangesInChunks(Collection<? extends OsmPrimitive> primitives, ProgressMonitor progressMonitor, int chunkSize)
    151152            throws OsmTransferException, IllegalArgumentException {
    152         if (chunkSize <=0)
     153        if (chunkSize <= 0)
    153154            throw new IllegalArgumentException(tr("Value >0 expected for parameter ''{0}'', got {1}", "chunkSize", chunkSize));
    154155        try {
     
    156157            List<OsmPrimitive> chunk = new ArrayList<>(chunkSize);
    157158            Iterator<? extends OsmPrimitive> it = primitives.iterator();
    158             int numChunks = (int)Math.ceil((double)primitives.size() / (double)chunkSize);
    159             int i= 0;
    160             while(it.hasNext()) {
     159            int numChunks = (int) Math.ceil((double) primitives.size() / (double) chunkSize);
     160            int i = 0;
     161            while (it.hasNext()) {
    161162                i++;
    162163                if (canceled) return;
    163164                int j = 0;
    164165                chunk.clear();
    165                 while(it.hasNext() && j < chunkSize) {
     166                while (it.hasNext() && j < chunkSize) {
    166167                    if (canceled) return;
    167168                    j++;
     
    174175                processed.addAll(api.uploadDiff(chunk, progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false)));
    175176            }
    176         } catch(OsmTransferException e) {
     177        } catch (OsmTransferException e) {
    177178            throw e;
    178179        } finally {
     
    192193     * @throws OsmTransferException if something goes wrong
    193194     */
    194     public void uploadOsm(UploadStrategySpecification strategy, Collection<? extends OsmPrimitive> primitives, Changeset changeset, ProgressMonitor monitor) throws OsmTransferException {
     195    public void uploadOsm(UploadStrategySpecification strategy, Collection<? extends OsmPrimitive> primitives,
     196            Changeset changeset, ProgressMonitor monitor) throws OsmTransferException {
    195197        CheckParameterUtil.ensureParameterNotNull(changeset, "changeset");
    196198        processed = new LinkedList<>();
     
    201203            // check whether we can use diff upload
    202204            if (changeset.getId() == 0) {
    203                 api.openChangeset(changeset,monitor.createSubTaskMonitor(0, false));
     205                api.openChangeset(changeset, monitor.createSubTaskMonitor(0, false));
    204206            } else {
    205                 api.updateChangeset(changeset,monitor.createSubTaskMonitor(0, false));
     207                api.updateChangeset(changeset, monitor.createSubTaskMonitor(0, false));
    206208            }
    207209            api.setChangeset(changeset);
    208210            switch(strategy.getStrategy()) {
    209211            case SINGLE_REQUEST_STRATEGY:
    210                 uploadChangesAsDiffUpload(primitives,monitor.createSubTaskMonitor(0,false));
     212                uploadChangesAsDiffUpload(primitives, monitor.createSubTaskMonitor(0, false));
    211213                break;
    212214            case INDIVIDUAL_OBJECTS_STRATEGY:
    213                 uploadChangesIndividually(primitives,monitor.createSubTaskMonitor(0,false));
     215                uploadChangesIndividually(primitives, monitor.createSubTaskMonitor(0, false));
    214216                break;
    215217            case CHUNKED_DATASET_STRATEGY:
    216                 uploadChangesInChunks(primitives,monitor.createSubTaskMonitor(0,false), strategy.getChunkSize());
     218                uploadChangesInChunks(primitives, monitor.createSubTaskMonitor(0, false), strategy.getChunkSize());
    217219                break;
    218220            }
    219         } catch(OsmTransferException e) {
     221        } catch (OsmTransferException e) {
    220222            throw e;
    221223        } finally {
  • trunk/src/org/openstreetmap/josm/io/OsmWriter.java

    r8443 r8510  
    6565        this.changeset = cs;
    6666    }
     67
    6768    public void setVersion(String v) {
    6869        this.version = v;
     
    9091    protected static final Comparator<OsmPrimitive> byIdComparator = new Comparator<OsmPrimitive>() {
    9192        @Override public int compare(OsmPrimitive o1, OsmPrimitive o2) {
    92             return o1.getUniqueId()<o2.getUniqueId() ? -1 : (o1.getUniqueId()==o2.getUniqueId() ? 0 : 1);
     93            return o1.getUniqueId() < o2.getUniqueId() ? -1 : (o1.getUniqueId() == o2.getUniqueId() ? 0 : 1);
    9394        }
    9495    };
     
    198199        } else {
    199200            out.println(">");
    200             for (int i=0; i<w.getNodesCount(); ++i) {
     201            for (int i = 0; i < w.getNodesCount(); ++i) {
    201202                out.println("    <nd ref='"+w.getNodeId(i) +"' />");
    202203            }
     
    213214        } else {
    214215            out.println(">");
    215             for (int i=0; i<e.getMembersCount(); ++i) {
     216            for (int i = 0; i < e.getMembersCount(); ++i) {
    216217                out.print("    <member type='");
    217218                out.print(e.getMemberType(i).getAPIName());
     
    249250    }
    250251
    251     protected static final Comparator<Entry<String, String>> byKeyComparator = new Comparator<Entry<String,String>>() {
    252         @Override public int compare(Entry<String, String> o1, Entry<String, String> o2) {
     252    protected static final Comparator<Entry<String, String>> byKeyComparator = new Comparator<Entry<String, String>>() {
     253        @Override
     254        public int compare(Entry<String, String> o1, Entry<String, String> o2) {
    253255            return o1.getKey().compareTo(o2.getKey());
    254256        }
     
    301303            // user and visible added with 0.4 API
    302304            if (osm.getUser() != null) {
    303                 if(osm.getUser().isLocalUser()) {
     305                if (osm.getUser().isLocalUser()) {
    304306                    out.print(" user='"+XmlWriter.encode(osm.getUser().getName())+"'");
    305307                } else if (osm.getUser().isOsmUser()) {
  • trunk/src/org/openstreetmap/josm/io/OsmWriterFactory.java

    r8126 r8510  
    2727        return theFactory.createOsmWriterImpl(out, osmConform, version);
    2828    }
     29
    2930    protected OsmWriter createOsmWriterImpl(PrintWriter out, boolean osmConform, String version) {
    3031        return new OsmWriter(out, osmConform, version);
  • trunk/src/org/openstreetmap/josm/io/UTFInputStreamReader.java

    r8378 r8510  
    6969        super(in);
    7070    }
     71
    7172    private UTFInputStreamReader(InputStream in, String cs) throws UnsupportedEncodingException {
    7273        super(in, cs);
  • trunk/src/org/openstreetmap/josm/io/WMSLayerExporter.java

    r7033 r8510  
    3131        if (layer instanceof WMSLayer) {
    3232            try (ObjectOutputStream oos = new ObjectOutputStream(new FileOutputStream(file))) {
    33                 ((WMSLayer)layer).writeExternal(oos);
     33                ((WMSLayer) layer).writeExternal(oos);
    3434            }
    3535        }
  • trunk/src/org/openstreetmap/josm/io/XmlWriter.java

    r7005 r8510  
    5353            if (encS != null) {
    5454                if (buffer == null) {
    55                     buffer = new StringBuilder(unencoded.substring(0,i));
     55                    buffer = new StringBuilder(unencoded.substring(0, i));
    5656                }
    5757                buffer.append(encS);
  • trunk/src/org/openstreetmap/josm/io/auth/CredentialsAgentException.java

    r4245 r8510  
    33
    44public class CredentialsAgentException extends Exception {
    5     public CredentialsAgentException() {super();}
    6     public CredentialsAgentException(String message, Throwable cause) {super(message, cause);}
    7     public CredentialsAgentException(String message) {super(message);}
    8     public CredentialsAgentException(Throwable cause) {super(cause);}
    95
     6    /**
     7     * Constructs a new {@code CredentialsAgentException}.
     8     * @param message the detail message (which is saved for later retrieval by the {@link #getMessage()} method).
     9     * @param cause the cause (which is saved for later retrieval by the {@link #getCause()} method).
     10     */
     11    public CredentialsAgentException(String message, Throwable cause) {
     12        super(message, cause);
     13    }
     14
     15    /**
     16     * Constructs a new {@code CredentialsAgentException}.
     17     * @param message the detail message (which is saved for later retrieval by the {@link #getMessage()} method).
     18     */
     19    public CredentialsAgentException(String message) {
     20        super(message);
     21    }
     22
     23    /**
     24     * Constructs a new {@code CredentialsAgentException}.
     25     * @param cause the cause (which is saved for later retrieval by the {@link #getCause()} method).
     26     */
     27    public CredentialsAgentException(Throwable cause) {
     28        super(cause);
     29    }
    1030}
  • trunk/src/org/openstreetmap/josm/io/auth/CredentialsManager.java

    r8509 r8510  
    118118        if (requestorType == RequestorType.SERVER && Objects.equals(OsmApi.getOsmApi().getHost(), host)) {
    119119            String username = credentials.getUserName();
    120             if(username != null && !username.trim().isEmpty()) {
     120            if (username != null && !username.trim().isEmpty()) {
    121121                JosmUserIdentityManager.getInstance().setPartiallyIdentified(username);
    122122            }
  • trunk/src/org/openstreetmap/josm/io/auth/DefaultAuthenticator.java

    r8388 r8510  
    5757            credentialsTried.put(getRequestorType(), true);
    5858            return new PasswordAuthentication(response.getUsername(), response.getPassword());
    59         } catch(CredentialsAgentException e) {
     59        } catch (CredentialsAgentException e) {
    6060            Main.error(e);
    6161            return null;
  • trunk/src/org/openstreetmap/josm/io/auth/JosmPreferencesCredentialAgent.java

    r8509 r8510  
    2828     */
    2929    @Override
    30     public PasswordAuthentication lookup(RequestorType requestorType, String host) throws CredentialsAgentException{
     30    public PasswordAuthentication lookup(RequestorType requestorType, String host) throws CredentialsAgentException {
    3131        if (requestorType == null)
    3232            return null;
     
    3838                user = Main.pref.get("osm-server.username", null);
    3939                password = Main.pref.get("osm-server.password", null);
    40             } else if(host != null) {
     40            } else if (host != null) {
    4141                user = Main.pref.get("server.username."+host, null);
    4242                password = Main.pref.get("server.password."+host, null);
     
    7474                    Main.pref.put("osm-server.password", String.valueOf(credentials.getPassword()));
    7575                }
    76             } else if(host != null) {
     76            } else if (host != null) {
    7777                Main.pref.put("server.username."+host, credentials.getUserName());
    7878                if (credentials.getPassword() == null) {
     
    130130    public Component getPreferencesDecorationPanel() {
    131131        HtmlPanel pnlMessage = new HtmlPanel();
    132         HTMLEditorKit kit = (HTMLEditorKit)pnlMessage.getEditorPane().getEditorKit();
     132        HTMLEditorKit kit = (HTMLEditorKit) pnlMessage.getEditorPane().getEditorKit();
    133133        kit.getStyleSheet().addRule(
    134                 ".warning-body {background-color:rgb(253,255,221);padding: 10pt; border-color:rgb(128,128,128);border-style: solid;border-width: 1px;}");
    135         pnlMessage.setText(
    136                 tr(
     134                ".warning-body {background-color:rgb(253,255,221);padding: 10pt; " +
     135                "border-color:rgb(128,128,128);border-style: solid;border-width: 1px;}");
     136        pnlMessage.setText(tr(
    137137                        "<html><body>"
    138138                        + "<p class=\"warning-body\">"
  • trunk/src/org/openstreetmap/josm/io/imagery/HTMLGrabber.java

    r8444 r8510  
    3030        String urlstring = url.toExternalForm();
    3131
    32         Main.info("Grabbing HTML " + (attempt > 1? "(attempt " + attempt + ") ":"") + url);
     32        Main.info("Grabbing HTML " + (attempt > 1 ? "(attempt " + attempt + ") " : "") + url);
    3333
    3434        List<String> cmdParams = new ArrayList<>();
     
    5252        BufferedImage img = layer.normalizeImage(ImageProvider.read(bais, true, WMSLayer.PROP_ALPHA_CHANNEL.get()));
    5353        bais.reset();
    54         layer.cache.saveToCache(layer.isOverlapEnabled()?img:null, bais, Main.getProjection(), request.getPixelPerDegree(), b.minEast, b.minNorth);
     54        layer.cache.saveToCache(layer.isOverlapEnabled() ? img : null,
     55                bais, Main.getProjection(), request.getPixelPerDegree(), b.minEast, b.minNorth);
    5556
    5657        return img;
  • trunk/src/org/openstreetmap/josm/io/imagery/ImageryReader.java

    r8418 r8510  
    274274                    try {
    275275                        val = Integer.valueOf(accumulator.toString());
    276                     } catch(NumberFormatException e) {
     276                    } catch (NumberFormatException e) {
    277277                        val = null;
    278278                    }
     
    315315                    try {
    316316                        tileSize = Integer.valueOf(accumulator.toString());
    317                     } catch(NumberFormatException e) {
     317                    } catch (NumberFormatException e) {
    318318                        tileSize = null;
    319319                    }
  • trunk/src/org/openstreetmap/josm/io/imagery/WMSGrabber.java

    r8444 r8510  
    8383        Matcher matcher = pattern.matcher(this.baseURL);
    8484        while (matcher.find()) {
    85             props.put(matcher.group(1),matcher.group(2));
     85            props.put(matcher.group(1), matcher.group(2));
    8686            matcher.appendReplacement(output, "");
    8787        }
     
    113113                    Main.debug("Ignoring "+request+" (precache only + exact match)");
    114114                }
    115             } else if (!loadFromCache(request)){
     115            } else if (!loadFromCache(request)) {
    116116                attempt(request);
    117117            } else if (Main.isDebugEnabled()) {
     
    122122    }
    123123
    124     protected void attempt(WMSRequest request){ // try to fetch the image
     124    protected void attempt(WMSRequest request) { // try to fetch the image
    125125        int maxTries = 5; // n tries for every image
    126126        for (int i = 1; i <= maxTries; i++) {
     
    153153
    154154    public static int random(int min, int max) {
    155         return (int)(Math.random() * ((max+1)-min)) + min;
     155        return (int) (Math.random() * ((max+1)-min)) + min;
    156156    }
    157157
     
    177177    public static final NumberFormat latLonFormat = new DecimalFormat("###0.0000000", new DecimalFormatSymbols(Locale.US));
    178178
    179     protected URL getURL(double w, double s,double e,double n,
     179    protected URL getURL(double w, double s, double e, double n,
    180180            int wi, int ht) throws MalformedURLException {
    181181        String myProj = Main.getProjection().toCode();
     
    247247        }
    248248
    249         if (!request.isReal() && !layer.hasAutoDownload()){
     249        if (!request.isReal() && !layer.hasAutoDownload()) {
    250250            request.finish(State.NOT_IN_CACHE, null, null);
    251251            return true;
     
    256256
    257257    protected BufferedImage grab(WMSRequest request, URL url, int attempt) throws WMSException, IOException, OsmTransferException {
    258         Main.info("Grabbing WMS " + (attempt > 1? "(attempt " + attempt + ") ":"") + url);
     258        Main.info("Grabbing WMS " + (attempt > 1 ? "(attempt " + attempt + ") " : "") + url);
    259259
    260260        HttpURLConnection conn = Utils.openHttpConnection(url);
     
    263263            conn.setRequestProperty(e.getKey(), e.getValue());
    264264        }
    265         conn.setConnectTimeout(Main.pref.getInteger("socket.timeout.connect",15) * 1000);
     265        conn.setConnectTimeout(Main.pref.getInteger("socket.timeout.connect", 15) * 1000);
    266266        conn.setReadTimeout(Main.pref.getInteger("socket.timeout.read", 30) * 1000);
    267267
     
    293293        BufferedImage img = layer.normalizeImage(ImageProvider.read(bais, true, WMSLayer.PROP_ALPHA_CHANNEL.get()));
    294294        bais.reset();
    295         layer.cache.saveToCache(layer.isOverlapEnabled()?img:null, bais, Main.getProjection(), request.getPixelPerDegree(), b.minEast, b.minNorth);
     295        layer.cache.saveToCache(layer.isOverlapEnabled() ? img : null,
     296                bais, Main.getProjection(), request.getPixelPerDegree(), b.minEast, b.minNorth);
    296297        return img;
    297298    }
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/AddTagsDialog.java

    r8509 r8510  
    5858    static class DeleteTagMarker {
    5959        private int num;
     60
    6061        public DeleteTagMarker(int num) {
    6162            this.num = num;
    6263        }
     64
    6365        @Override
    6466        public String toString() {
     
    7173     */
    7274    static class ExistingValues {
    73         private String tag;
    74         private Map<String, Integer> valueCount;
     75        private final String tag;
     76        private final Map<String, Integer> valueCount;
     77
    7578        public ExistingValues(String tag) {
    76             this.tag=tag; valueCount=new HashMap<>();
     79            this.tag = tag;
     80            this.valueCount = new HashMap<>();
    7781        }
    7882
    7983        int addValue(String val) {
    8084            Integer c = valueCount.get(val);
    81             int r = c==null? 1 : (c.intValue()+1);
     85            int r = c == null ? 1 : (c.intValue()+1);
    8286            valueCount.put(val, r);
    8387            return r;
     
    8690        @Override
    8791        public String toString() {
    88             StringBuilder sb=new StringBuilder();
     92            StringBuilder sb = new StringBuilder();
    8993            for (String k: valueCount.keySet()) {
    90                 if (sb.length()>0) sb.append(", ");
     94                if (sb.length() > 0) sb.append(", ");
    9195                sb.append(k);
    9296            }
     
    135139        count = new int[tags.length];
    136140
    137         for (int i = 0; i<tags.length; i++) {
     141        for (int i = 0; i < tags.length; i++) {
    138142            count[i] = 0;
    139143            String key = tags[i][0];
     
    143147            for (OsmPrimitive osm : sel) {
    144148                oldValue  = osm.get(key);
    145                 if (oldValue!=null) {
     149                if (oldValue != null) {
    146150                    old.addValue(oldValue);
    147151                    if (!oldValue.equals(value)) {
     
    162166            public Component prepareRenderer(TableCellRenderer renderer, int row, int column) {
    163167                Component c = super.prepareRenderer(renderer, row, column);
    164                 if (count[row]>0) {
     168                if (count[row] > 0) {
    165169                    c.setFont(c.getFont().deriveFont(Font.ITALIC));
    166170                    c.setForeground(new Color(100, 100, 100));
     
    174178            @Override
    175179            public TableCellEditor getCellEditor(int row, int column) {
    176                 Object value = getValueAt(row,column);
     180                Object value = getValueAt(row, column);
    177181                if (value instanceof DeleteTagMarker) return null;
    178182                if (value instanceof ExistingValues) return null;
     
    185189                int c = columnAtPoint(event.getPoint());
    186190                Object o = getValueAt(r, c);
    187                 if (c==1 || c==2) return o.toString();
    188                 if (c==3) return ((ExistingValues)o).getToolTip();
     191                if (c == 1 || c == 2) return o.toString();
     192                if (c == 3) return ((ExistingValues) o).getToolTip();
    189193                return tr("Enable the checkbox to accept the value");
    190194            }
     
    221225                }
    222226            });
    223             tablePanel.add(c , GBC.eol().insets(20,10,0,0));
     227            tablePanel.add(c , GBC.eol().insets(20, 10, 0, 0));
    224228        }
    225229        setContent(tablePanel);
     
    237241        if (Main.main.getCurrentDataSet() != null  && buttonIndex != 2) {
    238242            TableModel tm = propertyTable.getModel();
    239             for (int i=0; i<tm.getRowCount(); i++) {
    240                 if (buttonIndex==1 || (Boolean)tm.getValueAt(i, 0)) {
    241                     String key =(String)tm.getValueAt(i, 1);
     243            for (int i = 0; i < tm.getRowCount(); i++) {
     244                if (buttonIndex == 1 || (Boolean) tm.getValueAt(i, 0)) {
     245                    String key = (String) tm.getValueAt(i, 1);
    242246                    Object value = tm.getValueAt(i, 2);
    243247                    Main.main.undoRedo.add(new ChangePropertyCommand(sel,
     
    275279                        for (String tag : tagSet) {
    276280                            // support a  =   b===c as "a"="b===c"
    277                             String[] pair = tag.split("\\s*=\\s*",2);
     281                            String[] pair = tag.split("\\s*=\\s*", 2);
    278282                            keyValue[i][0] = pair[0];
    279                             keyValue[i][1] = pair.length<2 ? "": pair[1];
     283                            keyValue[i][1] = pair.length < 2 ? "" : pair[1];
    280284                            i++;
    281285                        }
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/DNSName.java

    r8444 r8510  
    8080        //startIndex is the index of the first character of a component
    8181        //endIndex is the index of the last character of a component plus 1
    82         for (int endIndex,startIndex=0; startIndex < name.length(); startIndex = endIndex+1) {
     82        for (int endIndex, startIndex = 0; startIndex < name.length(); startIndex = endIndex+1) {
    8383            endIndex = name.indexOf('.', startIndex);
    8484            if (endIndex < 0) {
     
    8989
    9090            //nonStartIndex: index for characters in the component beyond the first one
    91             for (int nonStartIndex=startIndex+1; nonStartIndex < endIndex; nonStartIndex++) {
     91            for (int nonStartIndex = startIndex+1; nonStartIndex < endIndex; nonStartIndex++) {
    9292                char x = name.charAt(nonStartIndex);
    9393                if ((alphaDigitsAndHyphen).indexOf(x) < 0)
     
    147147            return false;
    148148
    149         DNSName other = (DNSName)obj;
     149        DNSName other = (DNSName) obj;
    150150
    151151        // RFC2459 mandates that these names are
     
    203203        else {
    204204            String inName =
    205                 (((DNSName)inputName).getName()).toLowerCase(Locale.ENGLISH);
     205                (((DNSName) inputName).getName()).toLowerCase(Locale.ENGLISH);
    206206            String thisName = name.toLowerCase(Locale.ENGLISH);
    207207            if (inName.equals(thisName))
     
    236236    @Override
    237237    public int subtreeDepth() {
    238         String subtree=name;
    239         int i=1;
     238        String subtree = name;
     239        int i = 1;
    240240
    241241        /* count dots */
    242242        for (; subtree.lastIndexOf('.') >= 0; i++) {
    243             subtree=subtree.substring(0,subtree.lastIndexOf('.'));
     243            subtree = subtree.substring(0, subtree.lastIndexOf('.'));
    244244        }
    245245
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/PermissionPrefWithDefault.java

    r5085 r8510  
    22package org.openstreetmap.josm.io.remotecontrol;
    33
     4import static org.openstreetmap.josm.tools.I18n.tr;
     5
    46import java.util.Arrays;
    57import java.util.List;
     8
    69import org.openstreetmap.josm.Main;
    7 import static org.openstreetmap.josm.tools.I18n.tr;
    810
    911/**
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RemoteControl.java

    r8465 r8510  
    8686     */
    8787    public static InetAddress getInet6Address() throws UnknownHostException {
    88         for(InetAddress a : InetAddress.getAllByName(Main.pref.get("remote.control.host.ipv6", "::1"))) {
    89             if(a instanceof Inet6Address) {
     88        for (InetAddress a : InetAddress.getAllByName(Main.pref.get("remote.control.host.ipv6", "::1"))) {
     89            if (a instanceof Inet6Address) {
    9090                return a;
    9191            }
     
    102102    public static InetAddress getInet4Address() throws UnknownHostException {
    103103        // Return an address to the loopback interface by default
    104         for(InetAddress a : InetAddress.getAllByName(Main.pref.get("remote.control.host.ipv4", "127.0.0.1"))) {
    105             if(a instanceof Inet4Address) {
     104        for (InetAddress a : InetAddress.getAllByName(Main.pref.get("remote.control.host.ipv4", "127.0.0.1"))) {
     105            if (a instanceof Inet4Address) {
    106106                return a;
    107107            }
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RemoteControlHttpServer.java

    r8470 r8510  
    4545        } catch (Exception ex) {
    4646            /* only show error when we also have no IPv4 */
    47             if(instance4 == null) {
     47            if (instance4 == null) {
    4848                Main.warn(marktr("Cannot start IPv6 remotecontrol server on port {0}: {1}"),
    4949                    Integer.toString(port), ex.getLocalizedMessage());
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RemoteControlHttpsServer.java

    r8470 r8510  
    177177            String[] ps = san.split(",");
    178178            GeneralNames gnames = new GeneralNames();
    179             for(String item: ps) {
     179            for (String item: ps) {
    180180                colonpos = item.indexOf(':');
    181181                if (colonpos < 0) {
     
    197197
    198198        // Update the algorithm, and resign.
    199         algo = (AlgorithmId)cert.get(X509CertImpl.SIG_ALG);
     199        algo = (AlgorithmId) cert.get(X509CertImpl.SIG_ALG);
    200200        info.set(CertificateAlgorithmId.NAME + "." + CertificateAlgorithmId.ALGORITHM, algo);
    201201        cert = new X509CertImpl(info);
     
    337337            } catch (Exception ex) {
    338338                /* only show error when we also have no IPv4 */
    339                 if(instance4 == null) {
     339                if (instance4 == null) {
    340340                    Main.warn(marktr("Cannot start IPv6 remotecontrol https server on port {0}: {1}"),
    341341                        Integer.toString(port), ex.getLocalizedMessage());
     
    392392
    393393        if (Main.isTraceEnabled()) {
    394             if(server instanceof SSLServerSocket) {
     394            if (server instanceof SSLServerSocket) {
    395395                SSLServerSocket sslServer = (SSLServerSocket) server;
    396396                Main.trace("SSL server - Enabled Cipher suites: "+Arrays.toString(sslServer.getEnabledCipherSuites()));
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RequestProcessor.java

    r8465 r8510  
    106106    private static void addRequestHandlerClass(String command,
    107107                Class<? extends RequestHandler> handler, boolean silent) {
    108         if(command.charAt(0) == '/') {
     108        if (command.charAt(0) == '/') {
    109109            command = command.substring(1);
    110110        }
     
    176176            String command = questionPos < 0 ? url : url.substring(0, questionPos);
    177177
    178             Map<String,String> headers = new HashMap<>();
    179             int k=0, MAX_HEADERS=20;
    180             while (k<MAX_HEADERS) {
    181                 get=in.readLine();
    182                 if (get==null) break;
     178            Map<String, String> headers = new HashMap<>();
     179            int k = 0, MAX_HEADERS = 20;
     180            while (k < MAX_HEADERS) {
     181                get = in.readLine();
     182                if (get == null) break;
    183183                k++;
    184184                String[] h = get.split(": ", 2);
    185                 if (h.length==2) {
     185                if (h.length == 2) {
    186186                    headers.put(h[0], h[1]);
    187187                } else break;
     
    198198                String ref = headers.get("Referer");
    199199                Pattern r = Pattern.compile("(https?://)?([^/]*)");
    200                 if (ref!=null) {
     200                if (ref != null) {
    201201                    Matcher m = r.matcher(ref);
    202202                    if (m.find()) {
     
    390390            try {
    391391                Class<?> c = handlers.get(cmd);
    392                 if (c==null) return null;
     392                if (c == null) return null;
    393393                handler = handlers.get(cmd).newInstance();
    394394            } catch (InstantiationException | IllegalAccessException ex) {
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/AddNodeHandler.java

    r8470 r8510  
    7878     * @param args request arguments
    7979     */
    80     private void addNode(Map<String, String> args){
     80    private void addNode(Map<String, String> args) {
    8181
    8282        // Parse the arguments
     
    9191            Point p = Main.map.mapView.getPoint(ll);
    9292            node = Main.map.mapView.getNearestNode(p, OsmPrimitive.isUsablePredicate);
    93             if (node!=null && node.getCoor().greatCircleDistance(ll) > Main.pref.getDouble("remotecontrol.tolerance", 0.1)) {
     93            if (node != null && node.getCoor().greatCircleDistance(ll) > Main.pref.getDouble("remotecontrol.tolerance", 0.1)) {
    9494                node = null; // node is too far
    9595            }
    9696        }
    9797
    98         if (node==null) {
     98        if (node == null) {
    9999            node = new Node(ll);
    100100            // Now execute the commands to add this node.
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/AddWayHandler.java

    r8444 r8510  
    127127            Point p = Main.map.mapView.getPoint(ll);
    128128            nd = Main.map.mapView.getNearestNode(p, OsmPrimitive.isUsablePredicate);
    129             if (nd!=null && nd.getCoor().greatCircleDistance(ll) > Main.pref.getDouble("remote.tolerance", 0.1)) {
     129            if (nd != null && nd.getCoor().greatCircleDistance(ll) > Main.pref.getDouble("remote.tolerance", 0.1)) {
    130130                nd = null; // node is too far
    131131            }
     
    141141        }
    142142
    143         if (prev!=null) {
     143        if (prev != null) {
    144144            nd = prev;
    145         } else if (nd==null) {
     145        } else if (nd == null) {
    146146            nd = new Node(ll);
    147147            // Now execute the commands to add this node.
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/FeaturesHandler.java

    r8390 r8510  
    2323            RequestHandlerBadRequestException {
    2424        StringBuilder buf = new StringBuilder();
    25         String q=args.get("q");
    26         if (q!=null) {
     25        String q = args.get("q");
     26        if (q != null) {
    2727            buf.append('[');
    2828            boolean first = true;
     
    3434               }
    3535               String info = RequestProcessor.getHandlerInfoAsJSON("/"+s);
    36                if (info!=null) {
     36               if (info != null) {
    3737                   buf.append(info);
    3838               } else {
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/ImageryHandler.java

    r8509 r8510  
    140140            }
    141141        }));
    142         return new String[] {"/imagery?title=osm&type=tms&url=https://a.tile.openstreetmap.org/%7Bzoom%7D/%7Bx%7D/%7By%7D.png",
    143             "/imagery?title=landsat&type=wms&url=http://irs.gis-lab.info/?layers=landsat&SRS=%7Bproj%7D&WIDTH=%7Bwidth%7D&HEIGHT=%7Bheight%7D&BBOX=%7Bbbox%7D",
    144             "/imagery?title=...&type={"+types+"}&url=....[&cookies=...][&min_zoom=...][&max_zoom=...]"};
     142        return new String[] {
     143            "/imagery?title=osm&type=tms&url=https://a.tile.openstreetmap.org/%7Bzoom%7D/%7Bx%7D/%7By%7D.png",
     144            "/imagery?title=landsat&type=wms&url=http://irs.gis-lab.info/?" +
     145                    "layers=landsat&SRS=%7Bproj%7D&WIDTH=%7Bwidth%7D&HEIGHT=%7Bheight%7D&BBOX=%7Bbbox%7D",
     146            "/imagery?title=...&type={"+types+"}&url=....[&cookies=...][&min_zoom=...][&max_zoom=...]"
     147            };
    145148    }
    146149}
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadAndZoomHandler.java

    r8509 r8510  
    9090        if (command.equals(cmd)) {
    9191            return new String[] {
    92                     "/load_and_zoom?addtags=wikipedia:de=Wei%C3%9Fe_Gasse|maxspeed=5&select=way23071688,way23076176,way23076177,&left=13.740&right=13.741&top=51.05&bottom=51.049",
     92                    "/load_and_zoom?addtags=wikipedia:de=Wei%C3%9Fe_Gasse|maxspeed=5&select=way23071688,way23076176,way23076177," +
     93                            "&left=13.740&right=13.741&top=51.05&bottom=51.049",
    9394                    "/load_and_zoom?left=8.19&right=8.20&top=48.605&bottom=48.590&select=node413602999&new_layer=true"};
    9495        } else {
     
    123124                        }
    124125                        if (present != null && !present.isEmpty()) {
    125                             toDownload = new Area(new Rectangle2D.Double(minlon,minlat,maxlon-minlon,maxlat-minlat));
     126                            toDownload = new Area(new Rectangle2D.Double(minlon, minlat, maxlon-minlon, maxlat-minlat));
    126127                            toDownload.subtract(present);
    127128                            if (!toDownload.isEmpty()) {
     
    138139                        Main.info("RemoteControl: no download necessary");
    139140                    } else {
    140                         Future<?> future = osmTask.download(newLayer, new Bounds(minlat,minlon,maxlat,maxlon),
     141                        Future<?> future = osmTask.download(newLayer, new Bounds(minlat, minlon, maxlat, maxlon),
    141142                                null /* let the task manage the progress monitor */);
    142143                        Main.worker.submit(new PostDownloadHandler(osmTask, future));
     
    158159                public void run() {
    159160                    DataSet ds = Main.main.getCurrentDataSet();
    160                     if(ds == null) // e.g. download failed
     161                    if (ds == null) // e.g. download failed
    161162                        return;
    162163                    ds.clearSelection();
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadObjectHandler.java

    r8308 r8510  
    7373                    final List<PrimitiveId> downloaded = task.getDownloadedId();
    7474                    final DataSet ds = Main.main.getCurrentDataSet();
    75                     if(downloaded != null) {
     75                    if (downloaded != null) {
    7676                        GuiHelper.runInEDT(new Runnable() {
    7777                            @Override
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/RequestHandler.java

    r8470 r8510  
    3535
    3636    /** The GET request arguments */
    37     protected Map<String,String> args;
     37    protected Map<String, String> args;
    3838
    3939    /** The request URL without "GET". */
     
    220220        List<String> missingKeys = new LinkedList<>();
    221221        boolean error = false;
    222         if(mandatory != null) for (String key : mandatory) {
     222        if (mandatory != null) for (String key : mandatory) {
    223223            String value = args.get(key);
    224224            if (value == null || value.isEmpty()) {
     
    312312            super(message);
    313313        }
     314
    314315        public RequestHandlerBadRequestException(String message, Throwable cause) {
    315316            super(message, cause);
  • trunk/src/org/openstreetmap/josm/io/session/GeoImageSessionExporter.java

    r8426 r8510  
    4848        p.add(export, GBC.std());
    4949        p.add(lbl, GBC.std());
    50         p.add(GBC.glue(1,0), GBC.std().fill(GBC.HORIZONTAL));
     50        p.add(GBC.glue(1, 0), GBC.std().fill(GBC.HORIZONTAL));
    5151        return p;
    5252    }
  • trunk/src/org/openstreetmap/josm/io/session/GeoImageSessionImporter.java

    r8390 r8510  
    3333        NodeList imgNodes = elem.getChildNodes();
    3434        boolean useThumbs = false;
    35         for (int i=0; i<imgNodes.getLength(); ++i) {
     35        for (int i = 0; i < imgNodes.getLength(); ++i) {
    3636            Node imgNode = imgNodes.item(i);
    3737            if (imgNode.getNodeType() == Node.ELEMENT_NODE) {
     
    4040                    ImageEntry entry = new ImageEntry();
    4141                    NodeList attrNodes = imgElem.getChildNodes();
    42                     for (int j=0; j<attrNodes.getLength(); ++j) {
     42                    for (int j = 0; j < attrNodes.getLength(); ++j) {
    4343                        Node attrNode = attrNodes.item(j);
    4444                        if (attrNode.getNodeType() == Node.ELEMENT_NODE) {
  • trunk/src/org/openstreetmap/josm/io/session/GpxTracksSessionExporter.java

    r8426 r8510  
    121121        topRow.add(export, GBC.std());
    122122        topRow.add(lbl, GBC.std());
    123         topRow.add(GBC.glue(1,0), GBC.std().fill(GBC.HORIZONTAL));
     123        topRow.add(GBC.glue(1, 0), GBC.std().fill(GBC.HORIZONTAL));
    124124        p.add(topRow, GBC.eol().fill(GBC.HORIZONTAL));
    125         p.add(lblData, GBC.std().insets(10,0,0,0));
     125        p.add(lblData, GBC.std().insets(10, 0, 0, 0));
    126126        p.add(link, GBC.std());
    127127        p.add(include, GBC.eol());
    128         p.add(cards, GBC.eol().insets(15,0,3,3));
     128        p.add(cards, GBC.eol().insets(15, 0, 3, 3));
    129129
    130130        export.addItemListener(new ItemListener() {
  • trunk/src/org/openstreetmap/josm/io/session/ImagerySessionExporter.java

    r8426 r8510  
    6060        p.add(export, GBC.std());
    6161        p.add(lbl, GBC.std());
    62         p.add(GBC.glue(1,0), GBC.std().fill(GBC.HORIZONTAL));
     62        p.add(GBC.glue(1, 0), GBC.std().fill(GBC.HORIZONTAL));
    6363        return p;
    6464    }
     
    8080        layerElem.setAttribute("version", "0.1");
    8181        ImageryPreferenceEntry e = new ImageryPreferenceEntry(layer.getInfo());
    82         Map<String,String> data = new LinkedHashMap<>(Preferences.serializeStruct(e, ImageryPreferenceEntry.class));
     82        Map<String, String> data = new LinkedHashMap<>(Preferences.serializeStruct(e, ImageryPreferenceEntry.class));
    8383        if (layer instanceof WMSLayer) {
    8484            WMSLayer wms = (WMSLayer) layer;
     
    8686            data.put("automatically-change-resolution", Boolean.toString(wms.isAutoResolution()));
    8787        }
    88         for (Map.Entry<String,String> entry : data.entrySet()) {
     88        for (Map.Entry<String, String> entry : data.entrySet()) {
    8989            Element attrElem = support.createElement(entry.getKey());
    9090            layerElem.appendChild(attrElem);
  • trunk/src/org/openstreetmap/josm/io/session/ImagerySessionImporter.java

    r7937 r8510  
    3636        NodeList nodes = elem.getChildNodes();
    3737
    38         for (int i=0; i<nodes.getLength(); ++i) {
     38        for (int i = 0; i < nodes.getLength(); ++i) {
    3939            Node node = nodes.item(i);
    4040            if (node.getNodeType() == Node.ELEMENT_NODE) {
  • trunk/src/org/openstreetmap/josm/io/session/MarkerSessionExporter.java

    r8426 r8510  
    5555        p.add(export, GBC.std());
    5656        p.add(lbl, GBC.std());
    57         p.add(GBC.glue(1,0), GBC.std().fill(GBC.HORIZONTAL));
     57        p.add(GBC.glue(1, 0), GBC.std().fill(GBC.HORIZONTAL));
    5858        return p;
    5959    }
  • trunk/src/org/openstreetmap/josm/io/session/OsmDataSessionExporter.java

    r8426 r8510  
    110110            tf.setEditable(false);
    111111            cardLink.add(tf, GBC.std());
    112             save.setMargin(new Insets(0,0,0,0));
    113             cardLink.add(save, GBC.eol().insets(2,0,0,0));
     112            save.setMargin(new Insets(0, 0, 0, 0));
     113            cardLink.add(save, GBC.eol().insets(2, 0, 0, 0));
    114114        } else {
    115115            cardLink.add(new JLabel(tr("No file association")), GBC.eol());
     
    150150        topRow.add(export, GBC.std());
    151151        topRow.add(lbl, GBC.std());
    152         topRow.add(GBC.glue(1,0), GBC.std().fill(GBC.HORIZONTAL));
     152        topRow.add(GBC.glue(1, 0), GBC.std().fill(GBC.HORIZONTAL));
    153153        p.add(topRow, GBC.eol().fill(GBC.HORIZONTAL));
    154         p.add(lblData, GBC.std().insets(10,0,0,0));
     154        p.add(lblData, GBC.std().insets(10, 0, 0, 0));
    155155        p.add(link, GBC.std());
    156156        p.add(include, GBC.eol());
    157         p.add(cards, GBC.eol().insets(15,0,3,3));
     157        p.add(cards, GBC.eol().insets(15, 0, 3, 3));
    158158
    159159        export.addItemListener(new ItemListener() {
  • trunk/src/org/openstreetmap/josm/io/session/SessionReader.java

    r8509 r8510  
    5858public class SessionReader {
    5959
    60     private static Map<String, Class<? extends SessionLayerImporter>> sessionLayerImporters = new HashMap<>();
     60    private static final Map<String, Class<? extends SessionLayerImporter>> sessionLayerImporters = new HashMap<>();
     61
    6162    static {
    6263        registerSessionLayerImporter("osm-data", OsmDataSessionImporter.class);
     
    352353        NodeList nodes = layersEl.getChildNodes();
    353354
    354         for (int i=0; i<nodes.getLength(); ++i) {
     355        for (int i = 0; i < nodes.getLength(); ++i) {
    355356            Node node = nodes.item(i);
    356357            if (node.getNodeType() == Node.ELEMENT_NODE) {
  • trunk/src/org/openstreetmap/josm/io/session/SessionWriter.java

    r8384 r8510  
    4949
    5050    private static Map<Class<? extends Layer>, Class<? extends SessionLayerExporter>> sessionLayerExporters = new HashMap<>();
     51
    5152    static {
    5253        registerSessionLayerExporter(OsmDataLayer.class , OsmDataSessionExporter.class);
     
    196197        root.appendChild(layersEl);
    197198
    198         for (int index=0; index<layers.size(); ++index) {
     199        for (int index = 0; index < layers.size(); ++index) {
    199200            Layer layer = layers.get(index);
    200201            SessionLayerExporter exporter = exporters.get(layer);
Note: See TracChangeset for help on using the changeset viewer.