Changeset 8513 in josm for trunk


Ignore:
Timestamp:
2015-06-21T02:25:56+02:00 (9 years ago)
Author:
Don-vip
Message:

checkstyle: blocks

Location:
trunk
Files:
77 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/Main.java

    r8512 r8513  
    934934                    } catch (ClassNotFoundException ex) {
    935935                        // Do nothing
     936                        if (Main.isTraceEnabled()) {
     937                            Main.trace(ex.getMessage());
     938                        }
    936939                    }
    937940                }

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

    r8510 r8513  
    248248        // Search first fixed node
    249249        int startPosition = 0;
    250         for (startPosition = 0; startPosition < nodeCount; startPosition++)
    251             if (fixNodes.contains(nodes.get(startPosition % nodeCount))) break;
     250        for (startPosition = 0; startPosition < nodeCount; startPosition++) {
     251            if (fixNodes.contains(nodes.get(startPosition % nodeCount)))
     252                break;
     253        }
    252254        int i = startPosition; // Start position for current arc
    253255        int j; // End position for current arc
    254256        while (i < startPosition + nodeCount) {
    255             for (j = i + 1; j < startPosition + nodeCount; j++)
    256                 if (fixNodes.contains(nodes.get(j % nodeCount))) break;
     257            for (j = i + 1; j < startPosition + nodeCount; j++) {
     258                if (fixNodes.contains(nodes.get(j % nodeCount)))
     259                    break;
     260            }
    257261            Node first = nodes.get(i % nodeCount);
    258262            PolarCoor pcFirst = new PolarCoor(first.getEastNorth(), center, 0);
    … …  
    289293    private List<Node> collectNodesWithExternReferers(List<Way> ways) {
    290294        List<Node> withReferrers = new ArrayList<>();
    291         for (Way w: ways)
    292             for (Node n: w.getNodes())
    293                 if (n.getReferrers().size() > 1)
     295        for (Way w: ways) {
     296            for (Node n: w.getNodes()) {
     297                if (n.getReferrers().size() > 1) {
    294298                    withReferrers.add(n);
     299                }
     300            }
     301        }
    295302        return withReferrers;
    296303    }
    … …  
    312319                if (way.firstNode() == lastNode) {
    313320                    List<Node> wayNodes = way.getNodes();
    314                     for (int i = 0; i < wayNodes.size() - 1; i++)
     321                    for (int i = 0; i < wayNodes.size() - 1; i++) {
    315322                        nodes.add(wayNodes.get(i));
     323                    }
    316324                    lastNode = way.lastNode();
    317325                    lastWay = way;
    … …  
    320328                if (way.lastNode() == lastNode) {
    321329                    List<Node> wayNodes = way.getNodes();
    322                     for (int i = wayNodes.size() - 1; i > 0; i--)
     330                    for (int i = wayNodes.size() - 1; i > 0; i--) {
    323331                        nodes.add(wayNodes.get(i));
     332                    }
    324333                    lastNode = way.firstNode();
    325334                    lastWay = way;
    … …  
    348357    private boolean actionAllowed(Collection<Node> nodes) {
    349358        boolean outside = false;
    350         for (Node n: nodes)
     359        for (Node n: nodes) {
    351360            if (n.isOutsideDownloadArea()) {
    352361                outside = true;
    353362                break;
    354363            }
     364        }
    355365        if (outside)
    356366            new Notification(

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

    r8510 r8513  
    219219        Collection<Command> cmds = new ArrayList<>(nodes.size());
    220220        Line line = new Line(anchors[0], anchors[1]);
    221         for (Node node: nodes)
     221        for (Node node: nodes) {
    222222            if (node != anchors[0] && node != anchors[1])
    223223                cmds.add(line.projectionCommand(node));
     224        }
    224225        return new SequenceCommand(tr("Align Nodes in Line"), cmds);
    225226    }
    … …  
    245246        for (Node n: nodes) {
    246247            referers.clear();
    247             for (OsmPrimitive o: n.getReferrers())
     248            for (OsmPrimitive o: n.getReferrers()) {
    248249                if (ways.contains(o))
    249250                    referers.add((Way) o);
     251            }
    250252            if (referers.size() == 1) {
    251253                Way way = referers.get(0);
    … …  
    274276            List<Node> nodes = way.getNodes();
    275277            neighbors.clear();
    276             for (int i = 1; i < nodes.size()-1; i++)
     278            for (int i = 1; i < nodes.size()-1; i++) {
    277279                if (nodes.get(i) == node) {
    278280                    neighbors.add(nodes.get(i-1));
    279281                    neighbors.add(nodes.get(i+1));
    280282                }
     283            }
    281284            if (neighbors.isEmpty())
    282285                continue;

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

    r8510 r8513  
    103103
    104104        // TODO: convert this to switch/case and make sure the parsing still works
     105        // CHECKSTYLE.OFF: LeftCurly
     106        // CHECKSTYLE.OFF: RightCurly
    105107        /* leave as single line for shortcut overview parsing! */
    106108        if (mode.equals("data")) { shortcut = KeyEvent.VK_1; }
    … …  
    112114        else if (mode.equals("previous")) { shortcut = KeyEvent.VK_8; }
    113115        else if (mode.equals("next")) { shortcut = KeyEvent.VK_9; }
     116        // CHECKSTYLE.ON: LeftCurly
     117        // CHECKSTYLE.ON: RightCurly
    114118
    115119        return shortcut;

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

    r8510 r8513  
    244244    protected void updateEnabledState(Collection<? extends OsmPrimitive> selection) {
    245245        int numWays = 0;
    246         for (OsmPrimitive osm : selection)
     246        for (OsmPrimitive osm : selection) {
    247247            if (osm instanceof Way) {
    248248                numWays++;
    249249            }
     250        }
    250251        setEnabled(numWays >= 2);
    251252    }

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

    r8510 r8513  
    8787        while (assign < nodesCount) {
    8888            int imax = 0;
    89             for (int i = 1; i < angles.length; i++)
     89            for (int i = 1; i < angles.length; i++) {
    9090                if (remainder[i] > remainder[imax])
    9191                    imax = i;
     92            }
    9293            count[imax]++;
    9394            remainder[imax] = 0;

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

    r8443 r8513  
    369369
    370370        // filter out empty key conflicts - we need second iteration
    371         if (!Main.pref.getBoolean("multipoly.alltags", false))
    372             for (RelationMember m : relation.getMembers())
    373                 if (m.hasRole() && "outer".equals(m.getRole()) && m.isWay())
    374                     for (String key : values.keySet())
    375                         if (!m.getWay().hasKey(key) && !relation.hasKey(key))
     371        if (!Main.pref.getBoolean("multipoly.alltags", false)) {
     372            for (RelationMember m : relation.getMembers()) {
     373                if (m.hasRole() && "outer".equals(m.getRole()) && m.isWay()) {
     374                    for (String key : values.keySet()) {
     375                        if (!m.getWay().hasKey(key) && !relation.hasKey(key)) {
    376376                            conflictingKeys.add(key);
    377 
    378         for (String key : conflictingKeys)
     377                        }
     378                    }
     379                }
     380            }
     381        }
     382
     383        for (String key : conflictingKeys) {
    379384            values.remove(key);
    380 
    381         for (String linearTag : Main.pref.getCollection("multipoly.lineartagstokeep", DEFAULT_LINEAR_TAGS))
     385        }
     386
     387        for (String linearTag : Main.pref.getCollection("multipoly.lineartagstokeep", DEFAULT_LINEAR_TAGS)) {
    382388            values.remove(linearTag);
     389        }
    383390
    384391        if ("coastline".equals(values.get("natural")))

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

    r8512 r8513  
    239239                } catch (NumberFormatException nfe) {
    240240                    // we repaint offset numbers in any case
     241                    if (Main.isTraceEnabled()) {
     242                        Main.trace(nfe.getMessage());
     243                    }
    241244                }
    242245            }

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

    r8510 r8513  
    165165         */
    166166        public void reverse() {
    167             for (WayInPolygon way: ways)
     167            for (WayInPolygon way: ways) {
    168168                way.insideToTheRight = !way.insideToTheRight;
     169            }
    169170            Collections.reverse(ways);
    170171        }
    … …  
    266267            double angle = Math.atan2(en3.getY() - en1.getY(), en3.getX() - en1.getX()) -
    267268                    Math.atan2(en2.getY() - en1.getY(), en2.getX() - en1.getX());
    268             while (angle >= 2*Math.PI)
     269            while (angle >= 2*Math.PI) {
    269270                angle -= 2*Math.PI;
    270             while (angle < 0)
     271            }
     272            while (angle < 0) {
    271273                angle += 2*Math.PI;
     274            }
    272275            return angle;
    273276        }
    … …  
    10701073        // Remove all of these way to make the next work.
    10711074        List<WayInPolygon> cleanMultigonWays = new ArrayList<>();
    1072         for (WayInPolygon way: multigonWays)
     1075        for (WayInPolygon way: multigonWays) {
    10731076            if (way.way.getNodesCount() == 2 && way.way.isClosed())
    10741077                discardedWays.add(way);
    10751078            else
    10761079                cleanMultigonWays.add(way);
     1080        }
    10771081
    10781082        WayTraverser traverser = new WayTraverser(cleanMultigonWays);
    … …  
    11051109                        // Invalid ring (2 nodes) -> remove
    11061110                        traverser.removeWays(path);
    1107                         for (WayInPolygon way: path)
     1111                        for (WayInPolygon way: path) {
    11081112                            discardedResult.add(way.way);
     1113                        }
    11091114                    } else {
    11101115                        // Close ring -> add
    … …  
    14651470            for (RelationRole r : multiouters) {
    14661471                // Add members
    1467                 for (RelationMember rm : r.rel.getMembers())
     1472                for (RelationMember rm : r.rel.getMembers()) {
    14681473                    if (!newRel.getMembers().contains(rm)) {
    14691474                        newRel.addMember(rm);
    14701475                    }
     1476                }
    14711477                // Add tags
    14721478                for (String key : r.rel.keySet()) {

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

    r8510 r8513  
    5151    class OsmURLListener implements DocumentListener {
    5252        @Override
    53         public void changedUpdate(DocumentEvent e) { parseURL(); }
    54 
    55         @Override
    56         public void insertUpdate(DocumentEvent e) { parseURL(); }
    57 
    58         @Override
    59         public void removeUpdate(DocumentEvent e) { parseURL(); }
     53        public void changedUpdate(DocumentEvent e) {
     54            parseURL();
     55        }
     56
     57        @Override
     58        public void insertUpdate(DocumentEvent e) {
     59            parseURL();
     60        }
     61
     62        @Override
     63        public void removeUpdate(DocumentEvent e) {
     64            parseURL();
     65        }
    6066    }
    6167
    6268    class OsmLonLatListener implements DocumentListener {
    6369        @Override
    64         public void changedUpdate(DocumentEvent e) { updateUrl(false); }
    65 
    66         @Override
    67         public void insertUpdate(DocumentEvent e) { updateUrl(false); }
    68 
    69         @Override
    70         public void removeUpdate(DocumentEvent e) { updateUrl(false); }
     70        public void changedUpdate(DocumentEvent e) {
     71            updateUrl(false);
     72        }
     73
     74        @Override
     75        public void insertUpdate(DocumentEvent e) {
     76            updateUrl(false);
     77        }
     78
     79        @Override
     80        public void removeUpdate(DocumentEvent e) {
     81            updateUrl(false);
     82        }
    7183    }
    7284

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

    r8510 r8513  
    206206                    // to exactly one occurrence of the merged target node
    207207                    newNodes.add(targetNode);
    208                 } else {
    209                     // drop the node
    210                 }
     208                }
     209                // else: drop the node
    211210            }
    212211            if (newNodes.size() < 2) {

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

    r8510 r8513  
    7272                    double east = n.getEastNorth().east();
    7373                    double north = n.getEastNorth().north();
    74                     if (east > maxEast) { maxEast = east; }
    75                     if (east < minEast) { minEast = east; }
    76                     if (north > maxNorth) { maxNorth = north; }
    77                     if (north < minNorth) { minNorth = north; }
     74                    if (east > maxEast) {
     75                        maxEast = east;
     76                    }
     77                    if (east < minEast) {
     78                        minEast = east;
     79                    }
     80                    if (north > maxNorth) {
     81                        maxNorth = north;
     82                    }
     83                    if (north < minNorth) {
     84                        minNorth = north;
     85                    }
    7886                }
    7987            }

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

    r8379 r8513  
    8484            return;
    8585        } catch (NumberFormatException ignore) {
     86            if (Main.isTraceEnabled()) {
     87                Main.trace(ignore.getMessage());
     88            }
    8689        }
    8790

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

    r8338 r8513  
    4141            outerNodes = new TreeSet<>();
    4242
    43             for (Way way : ways)
     43            for (Way way : ways) {
    4444                addNodes(way);
     45            }
    4546        }
    4647    }
    … …  
    150151
    151152        selection = new LinkedList<>();
    152         for (OsmPrimitive primitive : currentSelection)
     153        for (OsmPrimitive primitive : currentSelection) {
    153154            selection.add(primitive);
     155        }
    154156
    155157        do {

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

    r8465 r8513  
    129129        } catch (SecurityException e) {
    130130            // Ignore exception
     131            if (Main.isTraceEnabled()) {
     132                Main.trace(e.getMessage());
     133            }
    131134        }
    132135        List<String> commandLineArgs = Main.getCommandLineArgs();

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

    r8510 r8513  
    411411        List<Command> cmds = new LinkedList<>();
    412412        Way way = null;
    413         for (Way w: OsmPrimitive.getFilteredList(selectedNode.getReferrers(), Way.class))
     413        for (Way w: OsmPrimitive.getFilteredList(selectedNode.getReferrers(), Way.class)) {
    414414            if (w.isUsable() && w.getNodesCount() >= 1) {
    415415                way = w;
    416416            }
     417        }
    417418        List<Node> oldNodes = way.getNodes();
    418419        List<Node> newNodes = new ArrayList<>(oldNodes.size());

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

    r8510 r8513  
    141141     */
    142142    protected boolean hasPrimitivesToDelete(Collection<OsmPrimitive> primitives) {
    143         for (OsmPrimitive p: primitives)
     143        for (OsmPrimitive p: primitives) {
    144144            if (p.isDeleted() && p.isModified() && !p.isNew())
    145145                return true;
     146        }
    146147        return false;
    147148    }

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

    r8510 r8513  
    539539                if (way != null) {
    540540                    int nodeCount = 0;
    541                     for (Node p : way.getNodes())
     541                    for (Node p : way.getNodes()) {
    542542                        if (p.equals(n0)) {
    543543                            nodeCount++;
    544544                        }
     545                    }
    545546                    if (nodeCount > 1) {
    546547                        way = null;

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

    r8444 r8513  
    258258            } else if (matchesCurrentModifiers(toggleSelectedModifierCombo)) {
    259259                newCursor = ImageProvider.getCursor("normal", "parallel"); // FIXME
    260             } else {
     260            } else if (Main.isDebugEnabled()) {
    261261                // TODO: set to a cursor indicating an error
     262                Main.debug("TODO: set an error cursor");
    262263            }
    263264            break;

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

    r8509 r8513  
    7979                if (sortedNodesPath.get(i+1).getCoor().equals(n.getCoor())) {
    8080                    removedNodes.add(n);
    81                     for (Way w : ways)
     81                    for (Way w : ways) {
    8282                        w.removeNode(n);
     83                    }
    8384                    continue;
    8485                }

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

    r8510 r8513  
    10671067        } catch (NumberFormatException e) {
    10681068            // fall out
     1069            if (Main.isTraceEnabled()) {
     1070                Main.trace(e.getMessage());
     1071            }
    10691072        }
    10701073        return def;
    … …  
    10821085        } catch (NumberFormatException e) {
    10831086            // fall out
     1087            if (Main.isTraceEnabled()) {
     1088                Main.trace(e.getMessage());
     1089            }
    10841090        }
    10851091        return def;
    … …  
    10951101        } catch (NumberFormatException e) {
    10961102            // fall out
     1103            if (Main.isTraceEnabled()) {
     1104                Main.trace(e.getMessage());
     1105            }
    10971106        }
    10981107        return def;
    … …  
    11081117        } catch (NumberFormatException e) {
    11091118            // fall out
     1119            if (Main.isTraceEnabled()) {
     1120                Main.trace(e.getMessage());
     1121            }
    11101122        }
    11111123        return def;
    … …  
    14721484        } catch (Exception e) {
    14731485            // Ignore all exceptions
     1486            if (Main.isTraceEnabled()) {
     1487                Main.trace(e.getMessage());
     1488            }
    14741489        }
    14751490        // Workaround to fix a Java "feature"

  • trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java

    r8510 r8513  
    441441            } catch (NumberFormatException e) {
    442442                // ignore malformed Cache-Control headers
     443                if (Main.isTraceEnabled()) {
     444                    Main.trace(e.getMessage());
     445                }
    443446            }
    444447        }

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

    r8510 r8513  
    665665                Matcher m = Pattern.compile(".*\\{PROJ\\(([^)}]+)\\)\\}.*").matcher(url.toUpperCase(Locale.ENGLISH));
    666666                if (m.matches()) {
    667                     for (String p : m.group(1).split(","))
     667                    for (String p : m.group(1).split(",")) {
    668668                        serverProjections.add(p);
     669                    }
    669670                }
    670671            } catch (Exception e) {

  • trunk/src/org/openstreetmap/josm/data/oauth/OAuthParameters.java

    r8415 r8513  
    7979            } catch (MalformedURLException e) {
    8080                // Ignored
     81                if (Main.isTraceEnabled()) {
     82                    Main.trace(e.getMessage());
     83                }
    8184            }
    8285        }

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/AllNodesVisitor.java

    r7005 r8513  
    3737    public void visit(Way w) {
    3838        if (w.isIncomplete()) return;
    39         for (Node n : w.getNodes())
     39        for (Node n : w.getNodes()) {
    4040            visit(n);
     41        }
    4142    }
    4243
    … …  
    4849    @Override
    4950    public void visit(Relation e) {
    50         for (RelationMember m : e.getMembers())
     51        for (RelationMember m : e.getMembers()) {
    5152            if (m.isNode()) visit(m.getNode());
     53        }
    5254    }
    5355
    … …  
    5961    public static Collection<Node> getAllNodes(Collection<? extends OsmPrimitive> osms) {
    6062        AllNodesVisitor v = new AllNodesVisitor();
    61         for (OsmPrimitive osm : osms)
     63        for (OsmPrimitive osm : osms) {
    6264            osm.accept(v);
     65        }
    6366        return v.nodes;
    6467    }

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/AbstractMapRenderer.java

    r8395 r8513  
    144144            // On read, it would first check, if the way still has firstIdx+2 nodes, then check if the corresponding way nodes are still the same
    145145            // and report changes in a more controlled manner.
     146            if (Main.isTraceEnabled()) {
     147                Main.trace(e.getMessage());
     148            }
    146149        }
    147150    }

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

    r8510 r8513  
    214214        List<RelationMember> rMembers = r.getMembers();
    215215        Map<String, String> rkeys = r.getKeys();
    216         for (String key : ignoreKeys)
     216        for (String key : ignoreKeys) {
    217217            rkeys.remove(key);
     218        }
    218219        RelationPair rKey = new RelationPair(rMembers, rkeys);
    219220        relations.put(rKey, r);
    … …  
    231232        Set<Relation> relFix = new HashSet<>();
    232233
    233         for (OsmPrimitive osm : sel)
     234        for (OsmPrimitive osm : sel) {
    234235            if (osm instanceof Relation && !osm.isDeleted()) {
    235236                relFix.add((Relation) osm);
    236237            }
     238        }
    237239
    238240        if (relFix.size() < 2)
    … …  
    294296        Set<Relation> relations = new HashSet<>();
    295297
    296         for (OsmPrimitive osm : sel)
     298        for (OsmPrimitive osm : sel) {
    297299            if (osm instanceof Relation) {
    298300                relations.add((Relation) osm);
    299301            }
     302        }
    300303
    301304        if (relations.size() < 2)

  • trunk/src/org/openstreetmap/josm/data/validation/util/Entities.java

    r8510 r8513  
    384384                        if (mapNameToValue == null) {
    385385                            mapNameToValue = new HashMap<>();
    386                             for (String[] pair : ARRAY)
     386                            for (String[] pair : ARRAY) {
    387387                                mapNameToValue.put(pair[0], pair[1]);
     388                            }
    388389                        }
    389390                        String value = mapNameToValue.get(entityContent);

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

    r8509 r8513  
    202202            if (userName != null && !userName.equals(this.userName)) {
    203203                setPartiallyIdentified(userName);
    204             } else {
    205                 // same name in the preferences as JOSM already knows about.
    206                 // keep the state, be it partially or fully identified
    207             }
     204            }
     205            // else: same name in the preferences as JOSM already knows about.
     206            // keep the state, be it partially or fully identified
    208207        }
    209208    }

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

    r8510 r8513  
    727727        } catch (SecurityException e) {
    728728            // Don't care, awtListener probably wasn't registered anyway
     729            if (Main.isTraceEnabled()) {
     730                Main.trace(e.getMessage());
     731            }
    729732        }
    730733        mv.removeMouseMotionListener(mouseMotionListener);

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

    r8511 r8513  
    361361            } catch (NullPointerException ignore) {
    362362                // NullPointerException at javax.swing.text.html.StyleSheet$ListPainter.paint
     363                if (Main.isTraceEnabled()) {
     364                    Main.trace(ignore.getMessage());
     365                }
    363366            }
    364367        }

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

    r8510 r8513  
    5757                setText(item.getTheirTagValue());
    5858                setToolTipText(item.getTheirTagValue());
    59             } else {
    60                 // should not happen
    6159            }
    6260        }
    6361    }
    64 
    6562}

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

    r8291 r8513  
    109109                primitive.put(key, myTagValue);
    110110            }
    111         } else {
    112            // should not happen
    113111        }
    114112    }

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

    r8510 r8513  
    267267                // (only if both primitives are tagged)
    268268                decision.keepOne(values.get(0));
    269             } else {
    270                 // Do not suggest to keep all values in order to reduce the wrong usage of semicolon values, see #9104!
    271             }
     269            }
     270            // else: Do not suggest to keep all values in order to reduce the wrong usage of semicolon values, see #9104!
    272271        }
    273272        rebuild();

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

    r8510 r8513  
    10951095                } catch (ParseException e) {
    10961096                    // Try next format
     1097                    if (Main.isTraceEnabled()) {
     1098                        Main.trace(e.getMessage());
     1099                    }
    10971100                }
    10981101            }

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

    r8510 r8513  
    842842            } catch (AddAbortException ex) {
    843843                // do nothing
     844                if (Main.isTraceEnabled()) {
     845                    Main.trace(ex.getMessage());
     846                }
    844847            }
    845848        }
    … …  
    869872            } catch (AddAbortException ex) {
    870873                // do nothing
     874                if (Main.isTraceEnabled()) {
     875                    Main.trace(ex.getMessage());
     876                }
    871877            }
    872878        }
    … …  
    879885
    880886    class AddSelectedBeforeSelection extends AddFromSelectionAction implements TableModelListener, ListSelectionListener {
     887        /**
     888         * Constructs a new {@code AddSelectedBeforeSelection}.
     889         */
    881890        public AddSelectedBeforeSelection() {
    882891            putValue(SHORT_DESCRIPTION,
    … …  
    899908            } catch (AddAbortException ex) {
    900909                // do nothing
    901             }
    902 
     910                if (Main.isTraceEnabled()) {
     911                    Main.trace(ex.getMessage());
     912                }
     913            }
    903914        }
    904915
    … …  
    935946            } catch (AddAbortException ex) {
    936947                // do nothing
     948                if (Main.isTraceEnabled()) {
     949                    Main.trace(ex.getMessage());
     950                }
    937951            }
    938952        }
    … …  
    950964
    951965    class RemoveSelectedAction extends AbstractAction implements TableModelListener {
     966        /**
     967         * Constructs a new {@code RemoveSelectedAction}.
     968         */
    952969        public RemoveSelectedAction() {
    953970            putValue(SHORT_DESCRIPTION, tr("Remove all members referring to one of the selected objects"));
    … …  
    17771794            } catch (AddAbortException ex) {
    17781795                // Do nothing
     1796                if (Main.isTraceEnabled()) {
     1797                    Main.trace(ex.getMessage());
     1798                }
    17791799            }
    17801800        }

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

    r8510 r8513  
    1010import java.nio.charset.StandardCharsets;
    1111
     12import org.openstreetmap.josm.Main;
    1213import org.openstreetmap.josm.tools.Utils;
    1314import org.openstreetmap.josm.tools.WikiReader;
    … …  
    6263                } catch (IOException e1) {
    6364                    // ignore
     65                    if (Main.isTraceEnabled()) {
     66                        Main.trace(e1.getMessage());
     67                    }
    6468                }
    6569            }

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

    r8510 r8513  
    315315        } catch (NumberFormatException e) {
    316316            // don't save invalid value to preferences
     317            if (Main.isTraceEnabled()) {
     318                Main.trace(e.getMessage());
     319            }
    317320        }
    318321    }

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

    r8510 r8513  
    7878        switch (answer) {
    7979        case 0:
    80             for (Layer layer : layers)
     80            for (Layer layer : layers) {
    8181                Main.pref.putColor("layer "+layer.getName(), c.getColor());
     82            }
    8283            break;
    8384        case 1:
    8485            return;
    8586        case 2:
    86             for (Layer layer : layers)
     87            for (Layer layer : layers) {
    8788                Main.pref.putColor("layer "+layer.getName(), null);
     89            }
    8890            break;
    8991        }

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

    r8510 r8513  
    932932    private boolean isDataSetEmpty() {
    933933        if (data != null) {
    934             for (OsmPrimitive osm : data.allNonDeletedPrimitives())
     934            for (OsmPrimitive osm : data.allNonDeletedPrimitives()) {
    935935                if (!osm.isDeleted() || !osm.isNewOrUndeleted())
    936936                    return false;
     937            }
    937938        }
    938939        return true;

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

    r8512 r8513  
    155155                } catch (IOException e) {
    156156                    // silence exceptions
     157                    if (Main.isTraceEnabled()) {
     158                        Main.trace(e.getMessage());
     159                    }
    157160                }
    158161

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

    r8510 r8513  
    490490    public void visitBoundingBox(BoundingXYVisitor v) {
    491491        for (int x = 0; x < dax; ++x) {
    492             for (int y = 0; y < day; ++y)
     492            for (int y = 0; y < day; ++y) {
    493493                if (images[x][y].getImage() != null) {
    494494                    v.visit(images[x][y].getMin());
    495495                    v.visit(images[x][y].getMax());
    496496                }
     497            }
    497498        }
    498499    }

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

    r8126 r8513  
    6767     *
    6868     */
    69     public void play() { play(0.0); }
     69    public void play() {
     70        play(0.0);
     71    }
    7072
    7173    public void adjustOffset(double adjustment) {

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

    r8509 r8513  
    127127                        rotationAngle = RotationAngle.buildStaticRotation(rotationKW.val);
    128128                    } catch (IllegalArgumentException ignore) {
     129                        if (Main.isTraceEnabled()) {
     130                            Main.trace(ignore.getMessage());
     131                        }
    129132                    }
    130133                }

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

    r8419 r8513  
    1818    }
    1919
    20     public AreaPrototype() { init(); }
     20    public AreaPrototype() {
     21        init();
     22    }
    2123
    2224    public final void init() {

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

    r8419 r8513  
    1818    }
    1919
    20     public IconPrototype() { init(); }
     20    /**
     21     * Constructs a new {@code IconPrototype}.
     22     */
     23    public IconPrototype() {
     24        init();
     25    }
    2126
    2227    public final void init() {

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

    r8395 r8513  
    3030    }
    3131
    32     public LinePrototype() { init(); }
     32    /**
     33     * Constructs a new {@code LinePrototype}.
     34     */
     35    public LinePrototype() {
     36        init();
     37    }
    3338
    3439    public void init() {

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

    r8510 r8513  
    1717    }
    1818
    19     public LinemodPrototype() { init(); }
     19    /**
     20     * Constructs a new {@code LinemodPrototype}.
     21     */
     22    public LinemodPrototype() {
     23        init();
     24    }
    2025
    2126    @Override

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

    r8510 r8513  
    13711371                                } catch (NumberFormatException e) {
    13721372                                    // ignore
     1373                                    if (Main.isTraceEnabled()) {
     1374                                        Main.trace(e.getMessage());
     1375                                    }
    13731376                                }
    13741377                            }

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

    r8512 r8513  
    5454    protected final void registerValidableComponent(AbstractButton component) {
    5555        component.addChangeListener(new ChangeListener() {
    56             @Override public void stateChanged(ChangeEvent e) { notifyListeners(); }
     56            @Override
     57            public void stateChanged(ChangeEvent e) {
     58                notifyListeners();
     59            }
    5760        });
    5861    }

  • trunk/src/org/openstreetmap/josm/gui/preferences/map/TaggingPresetPreference.java

    r8510 r8513  
    7272                        } catch (SAXException e) {
    7373                            // We will handle this in step with validation
     74                            if (Main.isTraceEnabled()) {
     75                                Main.trace(e.getMessage());
     76                            }
    7477                        }
    7578

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

    r8510 r8513  
    196196            } catch (NumberFormatException e) {
    197197                // ignore - load from preference pluginmanager.time-based-update.interval
     198                if (Main.isTraceEnabled()) {
     199                    Main.trace(e.getMessage());
     200                }
    198201            }
    199202            if (days <= 0) {

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

    r8510 r8513  
    6868    @Override
    6969    public Collection<String> getPreferencesFromCode(String code) {
    70         for (int i = 0; i < utmEPSGs.length; i++)
     70        for (int i = 0; i < utmEPSGs.length; i++) {
    7171            if (("EPSG:" + utmEPSGs[i]).equals(code))
    7272                return Collections.singleton(Integer.toString(i+1));
     73        }
    7374        return null;
    7475    }
    75 
    7676}

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

    r8510 r8513  
    657657                    } catch (NumberFormatException ex) {
    658658                        // Ignore - cannot auto-increment if last was non-numeric
     659                        if (Main.isTraceEnabled()) {
     660                            Main.trace(ex.getMessage());
     661                        }
    659662                    }
    660663                } else if (!usage.hadKeys() || PROP_FILL_DEFAULT.get() || "force".equals(use_last_as_default)) {
    … …  
    890893                if (def && !PROP_FILL_DEFAULT.get()) {
    891894                    // default is set and filling default values feature is disabled - check if all primitives are untagged
    892                     for (OsmPrimitive s : sel)
     895                    for (OsmPrimitive s : sel) {
    893896                        if (s.hasKeys()) {
    894897                            def = false;
    895898                        }
     899                    }
    896900                }
    897901

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

    r8510 r8513  
    221221        edSearchText = new JosmTextField();
    222222        edSearchText.getDocument().addDocumentListener(new DocumentListener() {
    223             @Override public void removeUpdate(DocumentEvent e) { filterPresets(); }
    224 
    225             @Override public void insertUpdate(DocumentEvent e) { filterPresets(); }
    226 
    227             @Override public void changedUpdate(DocumentEvent e) { filterPresets(); }
     223            @Override
     224            public void removeUpdate(DocumentEvent e) {
     225                filterPresets();
     226            }
     227
     228            @Override
     229            public void insertUpdate(DocumentEvent e) {
     230                filterPresets();
     231            }
     232
     233            @Override
     234            public void changedUpdate(DocumentEvent e) {
     235                filterPresets();
     236            }
    228237        });
    229238        edSearchText.addKeyListener(new KeyAdapter() {

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

    r8510 r8513  
    9191                } catch (NumberFormatException e) {
    9292                    // either the new text or the current text isn't a number. We continue with autocompletion
     93                    if (Main.isTraceEnabled()) {
     94                        Main.trace(e.getMessage());
     95                    }
    9396                }
    9497            }

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

    r8510 r8513  
    206206    class OsmUrlRefresher implements DocumentListener {
    207207        @Override
    208         public void changedUpdate(DocumentEvent e) { parseURL(); }
    209 
    210         @Override
    211         public void insertUpdate(DocumentEvent e) { parseURL(); }
    212 
    213         @Override
    214         public void removeUpdate(DocumentEvent e) { parseURL(); }
     208        public void changedUpdate(DocumentEvent e) {
     209            parseURL();
     210        }
     211
     212        @Override
     213        public void insertUpdate(DocumentEvent e) {
     214            parseURL();
     215        }
     216
     217        @Override
     218        public void removeUpdate(DocumentEvent e) {
     219            parseURL();
     220        }
    215221    }
    216222}

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

    r8510 r8513  
    55
    66import javax.swing.text.JTextComponent;
     7
     8import org.openstreetmap.josm.Main;
    79
    810/**
    … …  
    2628        return validator.id;
    2729    }
    28    
     30
    2931    /**
    3032     * Reads the changeset id.
    … …  
    3638        return validator.readChangesetId();
    3739    }
    38    
     40
    3941    /**
    4042     * Validator for a changeset ID entered in a {@link JTextComponent}.
    4143     */
    4244    public static class ChangesetIdValidator extends AbstractTextComponentValidator {
    43        
     45
    4446        private int id = 0;
    4547
    … …  
    6567            }
    6668        }
    67        
     69
    6870        /**
    6971         * Reads the changeset id.
    … …  
    8284                } catch (NumberFormatException e) {
    8385                    // Ignored
     86                    if (Main.isTraceEnabled()) {
     87                        Main.trace(e.getMessage());
     88                    }
    8489                }
    8590            }

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

    r8510 r8513  
    5353
    5454    @Override
    55     public void mousePressed(MouseEvent e) { processEvent(e); }
     55    public void mousePressed(MouseEvent e) {
     56        processEvent(e);
     57    }
    5658
    5759    @Override
    58     public void mouseReleased(MouseEvent e) { processEvent(e); }
     60    public void mouseReleased(MouseEvent e) {
     61        processEvent(e);
     62    }
    5963
    6064    private void processEvent(MouseEvent e) {

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

    r8510 r8513  
    182182            }
    183183        }
    184         /** Filter: No one may change the armed/selected/pressed status except us. */
    185         @Override public void setArmed(boolean b) { }
    186 
    187         @Override public void setSelected(boolean b) { }
    188 
    189         @Override public void setPressed(boolean b) { }
     184
     185        // ----------------------------------------------------------------------
     186        // Filter: No one may change the armed/selected/pressed status except us.
     187        // ----------------------------------------------------------------------
     188
     189        @Override
     190        public void setArmed(boolean b) { }
     191
     192        @Override
     193        public void setSelected(boolean b) { }
     194
     195        @Override
     196        public void setPressed(boolean b) { }
    190197
    191198        /** We disable focusing on the component when it is not enabled. */
    192         @Override public void setEnabled(boolean b) {
     199        @Override
     200        public void setEnabled(boolean b) {
    193201            setFocusable(b);
    194202            other.setEnabled(b);
    195203        }
    196         /** All these methods simply delegate to the "other" model
    197          * that is being decorated. */
    198         @Override public boolean isArmed() { return other.isArmed(); }
    199 
    200         @Override public boolean isSelected() { return other.isSelected(); }
    201 
    202         @Override public boolean isEnabled() { return other.isEnabled(); }
    203 
    204         @Override public boolean isPressed() { return other.isPressed(); }
    205 
    206         @Override public boolean isRollover() { return other.isRollover(); }
    207 
    208         @Override public void setRollover(boolean b) { other.setRollover(b); }
    209 
    210         @Override public void setMnemonic(int key) { other.setMnemonic(key); }
    211 
    212         @Override public int getMnemonic() { return other.getMnemonic(); }
    213 
    214         @Override public void setActionCommand(String s) {
     204
     205        // -------------------------------------------------------------------------------
     206        // All these methods simply delegate to the "other" model that is being decorated.
     207        // -------------------------------------------------------------------------------
     208
     209        @Override
     210        public boolean isArmed() {
     211            return other.isArmed();
     212        }
     213
     214        @Override
     215        public boolean isSelected() {
     216            return other.isSelected();
     217        }
     218
     219        @Override
     220        public boolean isEnabled() {
     221            return other.isEnabled();
     222        }
     223
     224        @Override
     225        public boolean isPressed() {
     226            return other.isPressed();
     227        }
     228
     229        @Override
     230        public boolean isRollover() {
     231            return other.isRollover();
     232        }
     233
     234        @Override
     235        public void setRollover(boolean b) {
     236            other.setRollover(b);
     237        }
     238
     239        @Override
     240        public void setMnemonic(int key) {
     241            other.setMnemonic(key);
     242        }
     243
     244        @Override
     245        public int getMnemonic() {
     246            return other.getMnemonic();
     247        }
     248
     249        @Override
     250        public void setActionCommand(String s) {
    215251            other.setActionCommand(s);
    216252        }

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

    r8510 r8513  
    22package org.openstreetmap.josm.io;
    33import static org.openstreetmap.josm.tools.I18n.tr;
     4
     5import org.openstreetmap.josm.Main;
    46
    57/**
    … …  
    144146        } catch (Exception e) {
    145147            // Ignored
     148            if (Main.isTraceEnabled()) {
     149                Main.trace(e.getMessage());
     150            }
    146151        }
    147152        try {
    … …  
    154159        } catch (Exception e) {
    155160            // Ignored
     161            if (Main.isTraceEnabled()) {
     162                Main.trace(e.getMessage());
     163            }
    156164        }
    157165        return sb.toString();

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

    r8510 r8513  
    1010import java.util.List;
    1111
     12import org.openstreetmap.josm.Main;
    1213import org.openstreetmap.josm.data.coor.LatLon;
    1314import org.openstreetmap.josm.gui.layer.GpxLayer;
    … …  
    8788                                // TODO: handle thumbnail loading
    8889                            } catch (NumberFormatException e) {
    89                                 // nothing
     90                                // do nothing
     91                                if (Main.isTraceEnabled()) {
     92                                    Main.trace(e.getMessage());
     93                                }
    9094                            }
    9195                        }

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

    r8510 r8513  
    9393     * if any available.
    9494     */
    95     public PreferenceSetting getPreferenceSetting() { return null; }
     95    public PreferenceSetting getPreferenceSetting() {
     96        return null;
     97    }
    9698
    9799    /**

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

    r8510 r8513  
    11241124     */
    11251125    public static Object getPlugin(String name) {
    1126         for (PluginProxy plugin : pluginList)
     1126        for (PluginProxy plugin : pluginList) {
    11271127            if (plugin.getPluginInformation().name.equals(name))
    11281128                return plugin.plugin;
     1129        }
    11291130        return null;
    11301131    }

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

    r8512 r8513  
    267267                            int k;
    268268
    269                             for (k = 1; xvec[x - k] == yvec[y - k]; k++)
     269                            for (k = 1; xvec[x - k] == yvec[y - k]; k++) {
    270270                                if (k == SNAKE_LIMIT) {
    271271                                    best = v;
    … …  
    273273                                    break;
    274274                                }
     275                            }
    275276                        }
    276277                    }
    … …  
    295296                            int k;
    296297
    297                             for (k = 0; xvec[x + k] == yvec[y + k]; k++)
     298                            for (k = 0; xvec[x + k] == yvec[y + k]; k++) {
    298299                                if (k == SNAKE_LIMIT) {
    299300                                    best = v;
    … …  
    301302                                    break;
    302303                                }
     304                            }
    303305                        }
    304306                    }
    … …  
    723725                        /* Cancel any subrun of MINIMUM or more provisionals
    724726                           within the larger run.  */
    725                         for (j = 0, consec = 0; j < length; j++)
     727                        for (j = 0, consec = 0; j < length; j++) {
    726728                            if (discards[i + j] != 2) {
    727729                                consec = 0;
    … …  
    732734                                discards[i + j] = 0;
    733735                            }
     736                        }
    734737
    735738                        /* Scan from beginning of run
    … …  
    783786            final int end = bufferedLines;
    784787            int j = 0;
    785             for (int i = 0; i < end; ++i)
     788            for (int i = 0; i < end; ++i) {
    786789                if (noDiscards || discards[i] == 0) {
    787790                    undiscarded[j] = equivs[i];
    … …  
    790793                    changedFlag[1+i] = true;
    791794                }
     795            }
    792796            nondiscardedLines = j;
    793797        }

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

    r8510 r8513  
    442442        } catch (MalformedURLException ex) {
    443443            // shouldn't happen
     444            if (Main.isTraceEnabled()) {
     445                Main.trace(e.getMessage());
     446            }
    444447        }
    445448
    … …  
    602605        } catch (MalformedURLException ex) {
    603606            // shouldn't happen
     607            if (Main.isTraceEnabled()) {
     608                Main.trace(e.getMessage());
     609            }
    604610        }
    605611

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

    r8510 r8513  
    482482        } catch (IOException e) {
    483483            // Ignore
     484            if (Main.isTraceEnabled()) {
     485                Main.trace(e.getMessage());
     486            }
    484487        }
    485488    }
    … …  
    511514        } catch (IOException e) {
    512515            // Ignore exception
     516            if (Main.isTraceEnabled()) {
     517                Main.trace(e.getMessage());
     518            }
    513519        }
    514520        return false;

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

    r8510 r8513  
    131131    public void initSystemShortcuts() {
    132132        // TODO: Insert system shortcuts here. See Windows and especially OSX to see how to.
    133         for (int i = KeyEvent.VK_F1; i <= KeyEvent.VK_F12; ++i)
     133        for (int i = KeyEvent.VK_F1; i <= KeyEvent.VK_F12; ++i) {
    134134            Shortcut.registerSystemShortcut("screen:toogle"+i, tr("reserved"),
    135135                    i, KeyEvent.CTRL_DOWN_MASK | KeyEvent.ALT_DOWN_MASK).setAutomatic();
     136        }
    136137        Shortcut.registerSystemShortcut("system:reset", tr("reserved"),
    137138                KeyEvent.VK_DELETE, KeyEvent.CTRL_DOWN_MASK | KeyEvent.ALT_DOWN_MASK).setAutomatic();
    … …  
    374375                    } catch (IOException e) {
    375376                        // Ignore
     377                        if (Main.isTraceEnabled()) {
     378                            Main.trace(e.getMessage());
     379                        }
    376380                    }
    377381                }

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

    r8510 r8513  
    6262            while (true) {
    6363                skipEmpty();
    64                 if (pos == n) { break; }
     64                if (pos == n) {
     65                    break;
     66                }
    6567                k = parseString("\n\r\t= ");
    66                 if (pos == n) { tags.clear();  break; }
     68                if (pos == n) {
     69                    tags.clear();
     70                    break;
     71                }
    6772                skipSign();
    68                 if (pos == n) { tags.clear();  break; }
     73                if (pos == n) {
     74                    tags.clear();
     75                    break;
     76                }
    6977                v = parseString("\n\r\t ");
    7078                tags.put(k, v);

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

    r8512 r8513  
    13011301    public static boolean hasExtension(String filename, String ... extensions) {
    13021302        String name = filename.toLowerCase(Locale.ENGLISH);
    1303         for (String ext : extensions)
     1303        for (String ext : extensions) {
    13041304            if (name.endsWith("."+ext.toLowerCase(Locale.ENGLISH)))
    13051305                return true;
     1306        }
    13061307        return false;
    13071308    }

  • trunk/src/org/openstreetmap/josm/tools/date/FallbackDateParser.java

    r7299 r8513  
    88import java.util.Date;
    99import java.util.List;
     10
     11import org.openstreetmap.josm.Main;
    1012
    1113/**
    … …  
    101103            } catch (ParseException pe) {
    102104                // Ignore parsing errors and try the next pattern.
     105                if (Main.isTraceEnabled()) {
     106                    Main.trace(pe.getMessage());
     107                }
    103108            }
    104109        }

  • trunk/test/unit/org/openstreetmap/josm/data/osm/history/HistoryWayTest.java

    r8510 r8513  
    1010
    1111import org.junit.Test;
     12import org.openstreetmap.josm.Main;
    1213import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    1314import org.openstreetmap.josm.data.osm.User;
    … …  
    7374        } catch (IndexOutOfBoundsException e) {
    7475            // OK
     76            if (Main.isTraceEnabled()) {
     77                Main.trace(e.getMessage());
     78            }
    7579        }
    7680

  • trunk/test/unit/org/openstreetmap/josm/data/projection/SwissGridTest.java

    r8510 r8513  
    7979    @Test
    8080    public void a_latlon2eastNorth_test() {
    81         {
    82             LatLon ll = new LatLon(46.518, 6.567);
    83             EastNorth en = Main.getProjection().latlon2eastNorth(ll);
    84             if (debug) {
    85                 System.out.println(en);
    86             }
    87             assertTrue("Lausanne", Math.abs(en.east() - 533111.69) < 0.1);
    88             assertTrue("Lausanne", Math.abs(en.north() - 152227.85) < 0.1);
    89         }
    90 
    91         {
    92             LatLon ll = new LatLon(47.78, 8.58);
    93             EastNorth en = Main.getProjection().latlon2eastNorth(ll);
    94             if (debug) {
    95                 System.out.println(en);
    96             }
    97             assertTrue("Schafouse", Math.abs(en.east() - 685544.16) < 0.1);
    98             assertTrue("Schafouse", Math.abs(en.north() - 292782.91) < 0.1);
    99         }
    100 
    101         {
    102             LatLon ll = new LatLon(46.58, 10.48);
    103             EastNorth en = Main.getProjection().latlon2eastNorth(ll);
    104             if (debug) {
    105                 System.out.println(en);
    106             }
    107             assertTrue("Grinson", Math.abs(en.east() - 833068.04) < 0.1);
    108             assertTrue("Grinson", Math.abs(en.north() - 163265.39) < 0.1);
    109         }
    110 
    111         {
    112             LatLon ll = new LatLon(46.0 + 57.0 / 60 + 3.89813884505 / 3600, 7.0 + 26.0 / 60 + 19.076595154147 / 3600);
    113             EastNorth en = Main.getProjection().latlon2eastNorth(ll);
    114             if (debug) {
    115                 System.out.println(en);
    116             }
    117             assertTrue("Berne", Math.abs(en.east() - 600000.0) < 0.1);
    118             assertTrue("Berne", Math.abs(en.north() - 200000.0) < 0.1);
    119         }
    120         {
    121             LatLon ll = new LatLon(46.0 + 2.0 / 60 + 38.87 / 3600, 8.0 + 43.0 / 60 + 49.79 / 3600);
    122             EastNorth en = Main.getProjection().latlon2eastNorth(ll);
    123             if (debug) {
    124                 System.out.println(en);
    125             }
    126             assertTrue("Ref", Math.abs(en.east() - 700000.0) < 0.1);
    127             assertTrue("Ref", Math.abs(en.north() - 100000.0) < 0.1);
    128         }
     81        LatLon ll = new LatLon(46.518, 6.567);
     82        EastNorth en = Main.getProjection().latlon2eastNorth(ll);
     83        if (debug) {
     84            System.out.println(en);
     85        }
     86        assertTrue("Lausanne", Math.abs(en.east() - 533111.69) < 0.1);
     87        assertTrue("Lausanne", Math.abs(en.north() - 152227.85) < 0.1);
     88
     89        ll = new LatLon(47.78, 8.58);
     90        en = Main.getProjection().latlon2eastNorth(ll);
     91        if (debug) {
     92            System.out.println(en);
     93        }
     94        assertTrue("Schafouse", Math.abs(en.east() - 685544.16) < 0.1);
     95        assertTrue("Schafouse", Math.abs(en.north() - 292782.91) < 0.1);
     96
     97        ll = new LatLon(46.58, 10.48);
     98        en = Main.getProjection().latlon2eastNorth(ll);
     99        if (debug) {
     100            System.out.println(en);
     101        }
     102        assertTrue("Grinson", Math.abs(en.east() - 833068.04) < 0.1);
     103        assertTrue("Grinson", Math.abs(en.north() - 163265.39) < 0.1);
     104
     105        ll = new LatLon(46.0 + 57.0 / 60 + 3.89813884505 / 3600, 7.0 + 26.0 / 60 + 19.076595154147 / 3600);
     106        en = Main.getProjection().latlon2eastNorth(ll);
     107        if (debug) {
     108            System.out.println(en);
     109        }
     110        assertTrue("Berne", Math.abs(en.east() - 600000.0) < 0.1);
     111        assertTrue("Berne", Math.abs(en.north() - 200000.0) < 0.1);
     112
     113        ll = new LatLon(46.0 + 2.0 / 60 + 38.87 / 3600, 8.0 + 43.0 / 60 + 49.79 / 3600);
     114        en = Main.getProjection().latlon2eastNorth(ll);
     115        if (debug) {
     116            System.out.println(en);
     117        }
     118        assertTrue("Ref", Math.abs(en.east() - 700000.0) < 0.1);
     119        assertTrue("Ref", Math.abs(en.north() - 100000.0) < 0.1);
    129120    }
    130121
    131122    @Test
    132123    public void b_eastNorth2latlon_test() {
    133         {
    134             EastNorth en = new EastNorth(533111.69, 152227.85);
    135             LatLon ll = Main.getProjection().eastNorth2latlon(en);
    136             if (debug) {
    137                 System.out.println(ll);
    138             }
    139             assertTrue("Lausanne", Math.abs(ll.lat() - 46.518) < 0.00001);
    140             assertTrue("Lausanne", Math.abs(ll.lon() - 6.567) < 0.00001);
    141         }
    142 
    143         {
    144             EastNorth en = new EastNorth(685544.16, 292782.91);
    145             LatLon ll = Main.getProjection().eastNorth2latlon(en);
    146             if (debug) {
    147                 System.out.println(ll);
    148             }
    149             assertTrue("Schafouse", Math.abs(ll.lat() - 47.78) < 0.00001);
    150             assertTrue("Schafouse", Math.abs(ll.lon() - 8.58) < 0.00001);
    151         }
    152 
    153         {
    154             EastNorth en = new EastNorth(833068.04, 163265.39);
    155             LatLon ll = Main.getProjection().eastNorth2latlon(en);
    156             if (debug) {
    157                 System.out.println(ll);
    158             }
    159             assertTrue("Grinson", Math.abs(ll.lat() - 46.58) < 0.00001);
    160             assertTrue("Grinson", Math.abs(ll.lon() - 10.48) < 0.00001);
    161         }
    162 
    163         {
    164             EastNorth en = new EastNorth(600000.0, 200000.0);
    165             LatLon ll = Main.getProjection().eastNorth2latlon(en);
    166             if (debug) {
    167                 System.out.println(ll);
    168             }
    169             assertTrue("Berne", Math.abs(ll.lat() - (46.0 + 57.0 / 60 + 3.89813884505 / 3600)) < 0.00001);
    170             assertTrue("Berne", Math.abs(ll.lon() - (7.0 + 26.0 / 60 + 19.076595154147 / 3600)) < 0.00001);
    171         }
    172 
    173         {
    174             EastNorth en = new EastNorth(700000.0, 100000.0);
    175             LatLon ll = Main.getProjection().eastNorth2latlon(en);
    176             if (debug) {
    177                 System.out.println(ll);
    178             }
    179             assertTrue("Ref", Math.abs(ll.lat() - (46.0 + 2.0 / 60 + 38.87 / 3600)) < 0.00001);
    180             assertTrue("Ref", Math.abs(ll.lon() - (8.0 + 43.0 / 60 + 49.79 / 3600)) < 0.00001);
    181         }
    182     }
    183 
     124        EastNorth en = new EastNorth(533111.69, 152227.85);
     125        LatLon ll = Main.getProjection().eastNorth2latlon(en);
     126        if (debug) {
     127            System.out.println(ll);
     128        }
     129        assertTrue("Lausanne", Math.abs(ll.lat() - 46.518) < 0.00001);
     130        assertTrue("Lausanne", Math.abs(ll.lon() - 6.567) < 0.00001);
     131
     132        en = new EastNorth(685544.16, 292782.91);
     133        ll = Main.getProjection().eastNorth2latlon(en);
     134        if (debug) {
     135            System.out.println(ll);
     136        }
     137        assertTrue("Schafouse", Math.abs(ll.lat() - 47.78) < 0.00001);
     138        assertTrue("Schafouse", Math.abs(ll.lon() - 8.58) < 0.00001);
     139
     140        en = new EastNorth(833068.04, 163265.39);
     141        ll = Main.getProjection().eastNorth2latlon(en);
     142        if (debug) {
     143            System.out.println(ll);
     144        }
     145        assertTrue("Grinson", Math.abs(ll.lat() - 46.58) < 0.00001);
     146        assertTrue("Grinson", Math.abs(ll.lon() - 10.48) < 0.00001);
     147
     148        en = new EastNorth(600000.0, 200000.0);
     149        ll = Main.getProjection().eastNorth2latlon(en);
     150        if (debug) {
     151            System.out.println(ll);
     152        }
     153        assertTrue("Berne", Math.abs(ll.lat() - (46.0 + 57.0 / 60 + 3.89813884505 / 3600)) < 0.00001);
     154        assertTrue("Berne", Math.abs(ll.lon() - (7.0 + 26.0 / 60 + 19.076595154147 / 3600)) < 0.00001);
     155
     156        en = new EastNorth(700000.0, 100000.0);
     157        ll = Main.getProjection().eastNorth2latlon(en);
     158        if (debug) {
     159            System.out.println(ll);
     160        }
     161        assertTrue("Ref", Math.abs(ll.lat() - (46.0 + 2.0 / 60 + 38.87 / 3600)) < 0.00001);
     162        assertTrue("Ref", Math.abs(ll.lon() - (8.0 + 43.0 / 60 + 49.79 / 3600)) < 0.00001);
     163    }
    184164
    185165    /**
    … …  
    188168    @Test
    189169    public void c_sendandreturn_test() {
    190         {
    191             EastNorth en = new EastNorth(533111.69, 152227.85);
    192             LatLon ll = Main.getProjection().eastNorth2latlon(en);
    193             EastNorth en2 = Main.getProjection().latlon2eastNorth(ll);
    194             if (debug) {
    195                 System.out.println(en.east() - en2.east());
    196             }
    197             if (debug) {
    198                 System.out.println(en.north() - en2.north());
    199             }
    200             assertTrue("Lausanne", Math.abs(en.east() - en2.east()) < 0.002);
    201             assertTrue("Lausanne", Math.abs(en.north() - en2.north()) < 0.002);
    202         }
    203 
    204         {
    205             EastNorth en = new EastNorth(685544.16, 292782.91);
    206             LatLon ll = Main.getProjection().eastNorth2latlon(en);
    207             EastNorth en2 = Main.getProjection().latlon2eastNorth(ll);
    208             if (debug) {
    209                 System.out.println(en.east() - en2.east());
    210             }
    211             if (debug) {
    212                 System.out.println(en.north() - en2.north());
    213             }
    214             assertTrue("Schafouse", Math.abs(en.east() - en2.east()) < 0.002);
    215             assertTrue("Schafouse", Math.abs(en.north() - en2.north()) < 0.002);
    216         }
    217 
    218         {
    219             EastNorth en = new EastNorth(833068.04, 163265.39);
    220             LatLon ll = Main.getProjection().eastNorth2latlon(en);
    221             EastNorth en2 = Main.getProjection().latlon2eastNorth(ll);
    222             if (debug) {
    223                 System.out.println(en.east() - en2.east());
    224             }
    225             if (debug) {
    226                 System.out.println(en.north() - en2.north());
    227             }
    228             assertTrue("Grinson", Math.abs(en.east() - en2.east()) < 0.002);
    229             assertTrue("Grinson", Math.abs(en.north() - en2.north()) < 0.002);
    230         }
    231 
    232         {
    233             EastNorth en = new EastNorth(600000.0, 200000.0);
    234             LatLon ll = Main.getProjection().eastNorth2latlon(en);
    235             EastNorth en2 = Main.getProjection().latlon2eastNorth(ll);
    236             if (debug) {
    237                 System.out.println(en.east() - en2.east());
    238             }
    239             if (debug) {
    240                 System.out.println(en.north() - en2.north());
    241             }
    242             assertTrue("Berne", Math.abs(en.east() - en2.east()) < 0.002);
    243             assertTrue("Berne", Math.abs(en.north() - en2.north()) < 0.002);
    244         }
    245 
    246         {
    247             EastNorth en = new EastNorth(700000.0, 100000.0);
    248             LatLon ll = Main.getProjection().eastNorth2latlon(en);
    249             EastNorth en2 = Main.getProjection().latlon2eastNorth(ll);
    250             if (debug) {
    251                 System.out.println(en.east() - en2.east());
    252             }
    253             if (debug) {
    254                 System.out.println(en.north() - en2.north());
    255             }
    256             assertTrue("Ref", Math.abs(en.east() - en2.east()) < 0.002);
    257             assertTrue("Ref", Math.abs(en.north() - en2.north()) < 0.002);
    258         }
     170        EastNorth en = new EastNorth(533111.69, 152227.85);
     171        LatLon ll = Main.getProjection().eastNorth2latlon(en);
     172        EastNorth en2 = Main.getProjection().latlon2eastNorth(ll);
     173        if (debug) {
     174            System.out.println(en.east() - en2.east());
     175        }
     176        if (debug) {
     177            System.out.println(en.north() - en2.north());
     178        }
     179        assertTrue("Lausanne", Math.abs(en.east() - en2.east()) < 0.002);
     180        assertTrue("Lausanne", Math.abs(en.north() - en2.north()) < 0.002);
     181
     182        en = new EastNorth(685544.16, 292782.91);
     183        ll = Main.getProjection().eastNorth2latlon(en);
     184        en2 = Main.getProjection().latlon2eastNorth(ll);
     185        if (debug) {
     186            System.out.println(en.east() - en2.east());
     187        }
     188        if (debug) {
     189            System.out.println(en.north() - en2.north());
     190        }
     191        assertTrue("Schafouse", Math.abs(en.east() - en2.east()) < 0.002);
     192        assertTrue("Schafouse", Math.abs(en.north() - en2.north()) < 0.002);
     193
     194        en = new EastNorth(833068.04, 163265.39);
     195        ll = Main.getProjection().eastNorth2latlon(en);
     196        en2 = Main.getProjection().latlon2eastNorth(ll);
     197        if (debug) {
     198            System.out.println(en.east() - en2.east());
     199        }
     200        if (debug) {
     201            System.out.println(en.north() - en2.north());
     202        }
     203        assertTrue("Grinson", Math.abs(en.east() - en2.east()) < 0.002);
     204        assertTrue("Grinson", Math.abs(en.north() - en2.north()) < 0.002);
     205
     206        en = new EastNorth(600000.0, 200000.0);
     207        ll = Main.getProjection().eastNorth2latlon(en);
     208        en2 = Main.getProjection().latlon2eastNorth(ll);
     209        if (debug) {
     210            System.out.println(en.east() - en2.east());
     211        }
     212        if (debug) {
     213            System.out.println(en.north() - en2.north());
     214        }
     215        assertTrue("Berne", Math.abs(en.east() - en2.east()) < 0.002);
     216        assertTrue("Berne", Math.abs(en.north() - en2.north()) < 0.002);
     217
     218        en = new EastNorth(700000.0, 100000.0);
     219        ll = Main.getProjection().eastNorth2latlon(en);
     220        en2 = Main.getProjection().latlon2eastNorth(ll);
     221        if (debug) {
     222            System.out.println(en.east() - en2.east());
     223        }
     224        if (debug) {
     225            System.out.println(en.north() - en2.north());
     226        }
     227        assertTrue("Ref", Math.abs(en.east() - en2.east()) < 0.002);
     228        assertTrue("Ref", Math.abs(en.north() - en2.north()) < 0.002);
    259229    }
    260230}

  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/nodes/NodeListMergeModelTest.java

    r8510 r8513  
    2121import org.junit.Test;
    2222import org.openstreetmap.josm.JOSMFixture;
     23import org.openstreetmap.josm.Main;
    2324import org.openstreetmap.josm.data.osm.DatasetFactory;
    2425import org.openstreetmap.josm.data.osm.Node;
    … …  
    357358        } catch (IllegalArgumentException e) {
    358359            // OK
     360            if (Main.isTraceEnabled()) {
     361                Main.trace(e.getMessage());
     362            }
    359363        }
    360364
    … …  
    364368        } catch (IllegalArgumentException e) {
    365369            // OK
     370            if (Main.isTraceEnabled()) {
     371                Main.trace(e.getMessage());
     372            }
    366373        }
    367374    }

  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/tags/TagMergeItemTest.java

    r8510 r8513  
    99import org.junit.Test;
    1010import org.openstreetmap.josm.JOSMFixture;
     11import org.openstreetmap.josm.Main;
    1112import org.openstreetmap.josm.data.osm.Node;
    1213import org.openstreetmap.josm.gui.conflict.pair.MergeDecisionType;
    … …  
    8788        } catch (IllegalArgumentException e) {
    8889            // OK
     90            if (Main.isTraceEnabled()) {
     91                Main.trace(e.getMessage());
     92            }
    8993        }
    9094    }
    … …  
    133137        } catch (IllegalStateException e) {
    134138            // OK
     139            if (Main.isTraceEnabled()) {
     140                Main.trace(e.getMessage());
     141            }
    135142        }
    136143    }
    … …  
    145152        } catch (IllegalArgumentException e) {
    146153            // OK
     154            if (Main.isTraceEnabled()) {
     155                Main.trace(e.getMessage());
     156            }
    147157        }
    148158    }

  • trunk/test/unit/org/openstreetmap/josm/tools/OsmUrlToBoundsTest.java

    r8509 r8513  
    44import org.junit.Assert;
    55import org.junit.Test;
     6import org.openstreetmap.josm.Main;
    67import org.openstreetmap.josm.data.Bounds;
    78
    … …  
    7172            } catch (IllegalArgumentException e) {
    7273                // Ignore. check if bounds is null after
     74                if (Main.isTraceEnabled()) {
     75                    Main.trace(e.getMessage());
     76                }
    7377            }
    7478            Assert.assertEquals(item.url, item.bounds, bounds);

  • trunk/tools/checkstyle/josm_checks.xml

    r8512 r8513  
    7575    <module name="ModifierOrder"/>
    7676    <module name="RedundantModifier"/>
     77    <module name="AvoidNestedBlocks">
     78      <property name="allowInSwitchCase" value="true"/>
     79    </module>
     80    <module name="EmptyBlock">
     81      <property name="tokens" value="LITERAL_TRY,LITERAL_CATCH,LITERAL_FINALLY,LITERAL_DO,LITERAL_ELSE,LITERAL_FOR,INSTANCE_INIT,STATIC_INIT,LITERAL_SWITCH"/>
     82    </module>
     83    <module name="EmptyCatchBlock"/>
     84    <module name="LeftCurly">
     85      <property name="maxLineLength" value="140"/>
     86    </module>
     87    <module name="RightCurly"/>
     88    <module name="NeedBraces">
     89      <property name="tokens" value="LITERAL_DO,LITERAL_FOR,LITERAL_WHILE"/>
     90      <property name="allowSingleLineStatement" value="true"/>
     91    </module>
    7792  </module>
    7893  <module name="Header">
Note: See TracChangeset for help on using the changeset viewer.