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

checkstyle: blocks

Location:
trunk/src/org/openstreetmap/josm/actions
Files:
19 edited

Legend:

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