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

checkstyle: enable relevant whitespace checks and fix them

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

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/AboutAction.java

    r8442 r8510  
    8787                "</html>");
    8888        info.add(label, GBC.eol().fill(GBC.HORIZONTAL).insets(10, 0, 0, 0));
    89         info.add(new JLabel(tr("Homepage")), GBC.std().insets(10,0,10,0));
    90         info.add(new UrlLabel(Main.getJOSMWebsite(),2), GBC.eol().fill(GBC.HORIZONTAL));
    91         info.add(GBC.glue(0,5), GBC.eol());
    92         info.add(new JLabel(tr("Bug Reports")), GBC.std().insets(10,0,10,0));
     89        info.add(new JLabel(tr("Homepage")), GBC.std().insets(10, 0, 10, 0));
     90        info.add(new UrlLabel(Main.getJOSMWebsite(), 2), GBC.eol().fill(GBC.HORIZONTAL));
     91        info.add(GBC.glue(0, 5), GBC.eol());
     92        info.add(new JLabel(tr("Bug Reports")), GBC.std().insets(10, 0, 10, 0));
    9393        info.add(BugReportExceptionHandler.getBugReportUrlLabel(Utils.strip(ShowStatusReportAction.getReportHeader())),
    9494                GBC.eol().fill(GBC.HORIZONTAL));
     
    112112
    113113    private JScrollPane createScrollPane(JosmTextArea area) {
    114         area.setBorder(BorderFactory.createEmptyBorder(5,5,5,5));
     114        area.setBorder(BorderFactory.createEmptyBorder(5, 5, 5, 5));
    115115        area.setOpaque(false);
    116116        JScrollPane sp = new JScrollPane(area);
  • trunk/src/org/openstreetmap/josm/actions/AbstractInfoAction.java

    r8509 r8510  
    8282        //
    8383        Iterator<OsmPrimitive> it = primitivesToShow.iterator();
    84         while(it.hasNext()) {
     84        while (it.hasNext()) {
    8585            if (it.next().isNew()) {
    8686                it.remove();
  • trunk/src/org/openstreetmap/josm/actions/AddImageryLayerAction.java

    r8509 r8510  
    4343     */
    4444    public AddImageryLayerAction(ImageryInfo info) {
    45         super(info.getMenuName(), /* ICON */"imagery_menu", tr("Add imagery layer {0}",info.getName()), null, false, false);
     45        super(info.getMenuName(), /* ICON */"imagery_menu", tr("Add imagery layer {0}", info.getName()), null, false, false);
    4646        putValue("toolbar", "imagery_" + info.getToolbarName());
    4747        putValue("help", ht("/Preferences/Imagery"));
     
    105105            formats.setToolTipText(tr("Select image format for WMS layer"));
    106106
    107             if (1 != new ExtendedDialog(Main.parent, tr("Select WMS layers"), new String[]{tr("Add layers"), tr("Cancel")}) {{
     107            if (1 != new ExtendedDialog(Main.parent, tr("Select WMS layers"), new String[]{tr("Add layers"), tr("Cancel")}) { {
    108108                final JScrollPane scrollPane = new JScrollPane(tree.getLayerTree());
    109109                scrollPane.setPreferredSize(new Dimension(400, 400));
     
    112112                panel.add(formats, GBC.eol().fill(GBC.HORIZONTAL));
    113113                setContent(panel);
    114             }}.showDialog().getValue()) {
     114            } }.showDialog().getValue()) {
    115115                return null;
    116116            }
  • trunk/src/org/openstreetmap/josm/actions/AlignInCircleAction.java

    r8460 r8510  
    165165            List<Node> outside = new ArrayList<>();
    166166
    167             for(Node n: nodes) {
     167            for (Node n: nodes) {
    168168                boolean isInside = false;
    169                 for(Way w: ways) {
    170                     if(w.getNodes().contains(n)) {
     169                for (Way w: ways) {
     170                    if (w.getNodes().contains(n)) {
    171171                        isInside = true;
    172172                        break;
    173173                    }
    174174                }
    175                 if(isInside)
     175                if (isInside)
    176176                    inside.add(n);
    177177                else
     
    179179            }
    180180
    181             if(outside.size() == 1 && inside.isEmpty()) {
     181            if (outside.size() == 1 && inside.isEmpty()) {
    182182                center = outside.get(0).getEastNorth();
    183             } else if(outside.size() == 1 && inside.size() == 1) {
     183            } else if (outside.size() == 1 && inside.size() == 1) {
    184184                center = outside.get(0).getEastNorth();
    185185                radius = distance(center, inside.get(0).getEastNorth());
    186             } else if(inside.size() == 2 && outside.isEmpty()) {
     186            } else if (inside.size() == 2 && outside.isEmpty()) {
    187187                // 2 nodes inside, define diameter
    188188                EastNorth en0 = inside.get(0).getEastNorth();
     
    239239        }
    240240
    241         if(!actionAllowed(nodes)) return;
     241        if (!actionAllowed(nodes)) return;
    242242
    243243        Collection<Command> cmds = new LinkedList<>();
     
    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))) break;
    252252        int i = startPosition; // Start position for current arc
    253253        int j; // End position for current arc
    254         while(i < startPosition + nodeCount) {
    255             for(j = i + 1; j < startPosition + nodeCount; j++)
    256                 if(fixNodes.contains(nodes.get(j % nodeCount))) break;
     254        while (i < startPosition + nodeCount) {
     255            for (j = i + 1; j < startPosition + nodeCount; j++)
     256                if (fixNodes.contains(nodes.get(j % nodeCount))) break;
    257257            Node first = nodes.get(i % nodeCount);
    258258            PolarCoor pcFirst = new PolarCoor(first.getEastNorth(), center, 0);
    259259            pcFirst.radius = radius;
    260260            cmds.add(pcFirst.createMoveCommand(first));
    261             if(j > i + 1) {
     261            if (j > i + 1) {
    262262                double delta;
    263                 if(j == i + nodeCount) {
     263                if (j == i + nodeCount) {
    264264                    delta = 2 * Math.PI / nodeCount;
    265265                } else {
    266266                    PolarCoor pcLast = new PolarCoor(nodes.get(j % nodeCount).getEastNorth(), center, 0);
    267267                    delta = pcLast.angle - pcFirst.angle;
    268                     if(delta < 0) // Assume each PolarCoor.angle is in range ]-pi; pi]
     268                    if (delta < 0) // Assume each PolarCoor.angle is in range ]-pi; pi]
    269269                        delta +=  2*Math.PI;
    270270                    delta /= j - i;
    271271                }
    272                 for(int k = i+1; k < j; k++) {
     272                for (int k = i+1; k < j; k++) {
    273273                    PolarCoor p = new PolarCoor(radius, pcFirst.angle + (k-i)*delta, center, 0);
    274274                    cmds.add(p.createMoveCommand(nodes.get(k % nodeCount)));
     
    289289    private List<Node> collectNodesWithExternReferers(List<Way> ways) {
    290290        List<Node> withReferrers = new ArrayList<>();
    291         for(Way w: ways)
    292             for(Node n: w.getNodes())
    293                 if(n.getReferrers().size() > 1)
     291        for (Way w: ways)
     292            for (Node n: w.getNodes())
     293                if (n.getReferrers().size() > 1)
    294294                    withReferrers.add(n);
    295295        return withReferrers;
     
    306306        Node lastNode = null;
    307307        Way lastWay = null;
    308         while(firstNode != lastNode) {
    309             if(lastNode == null) lastNode = firstNode;
    310             for(Way way: ways) {
    311                 if(way == lastWay) continue;
    312                 if(way.firstNode() == lastNode) {
     308        while (firstNode != lastNode) {
     309            if (lastNode == null) lastNode = firstNode;
     310            for (Way way: ways) {
     311                if (way == lastWay) continue;
     312                if (way.firstNode() == lastNode) {
    313313                    List<Node> wayNodes = way.getNodes();
    314                     for(int i = 0; i < wayNodes.size() - 1; i++)
     314                    for (int i = 0; i < wayNodes.size() - 1; i++)
    315315                        nodes.add(wayNodes.get(i));
    316316                    lastNode = way.lastNode();
     
    318318                    break;
    319319                }
    320                 if(way.lastNode() == lastNode) {
     320                if (way.lastNode() == lastNode) {
    321321                    List<Node> wayNodes = way.getNodes();
    322                     for(int i = wayNodes.size() - 1; i > 0; i--)
     322                    for (int i = wayNodes.size() - 1; i > 0; i--)
    323323                        nodes.add(wayNodes.get(i));
    324324                    lastNode = way.firstNode();
     
    331331        int nc = nodes.size();
    332332        double area = 0;
    333         for(int i = 0; i < nc; i++) {
     333        for (int i = 0; i < nc; i++) {
    334334            EastNorth p1 = nodes.get(i).getEastNorth();
    335335            EastNorth p2 = nodes.get((i+1) % nc).getEastNorth();
    336336            area += p1.east()*p2.north() - p2.east()*p1.north();
    337337        }
    338         if(area < 0)
     338        if (area < 0)
    339339            Collections.reverse(nodes);
    340340        return nodes;
     
    348348    private boolean actionAllowed(Collection<Node> nodes) {
    349349        boolean outside = false;
    350         for(Node n: nodes)
    351             if(n.isOutsideDownloadArea()) {
     350        for (Node n: nodes)
     351            if (n.isOutsideDownloadArea()) {
    352352                outside = true;
    353353                break;
    354354            }
    355         if(outside)
     355        if (outside)
    356356            new Notification(
    357357                    tr("One or more nodes involved in this action is outside of the downloaded area."))
  • trunk/src/org/openstreetmap/josm/actions/AlignInLineAction.java

    r8460 r8510  
    8787        // Get ways passing though all selected nodes.
    8888        Set<Way> waysRef = null;
    89         for(Node n: nodes) {
     89        for (Node n: nodes) {
    9090            Collection<Way> ref = OsmPrimitive.getFilteredList(n.getReferrers(), Way.class);
    91             if(waysRef == null)
     91            if (waysRef == null)
    9292                waysRef = new HashSet<>(ref);
    9393            else
     
    171171            // Decide what to align based on selection:
    172172
    173             if(selectedNodes.isEmpty() && !selectedWays.isEmpty()) {
     173            if (selectedNodes.isEmpty() && !selectedWays.isEmpty()) {
    174174                // Only ways selected -> For each way align their nodes taking care of intersection
    175175                cmd = alignMultiWay(selectedWays);
    176             } else if(selectedNodes.size() == 1) {
     176            } else if (selectedNodes.size() == 1) {
    177177                // Only 1 node selected -> align this node relative to referers way
    178178                Node selectedNode = selectedNodes.get(0);
    179179                List<Way> involvedWays = null;
    180                 if(selectedWays.isEmpty())
     180                if (selectedWays.isEmpty())
    181181                    // No selected way, all way containing this node are used
    182182                    involvedWays = OsmPrimitive.getFilteredList(selectedNode.getReferrers(), Way.class);
     
    185185                    involvedWays = selectedWays;
    186186                List<Line> lines = getInvolvedLines(selectedNode, involvedWays);
    187                 if(lines.size() > 2 || lines.isEmpty())
     187                if (lines.size() > 2 || lines.isEmpty())
    188188                    throw new InvalidSelection();
    189189                cmd = alignSingleNode(selectedNodes.get(0), lines);
    190             } else if(selectedNodes.size() >= 3) {
     190            } else if (selectedNodes.size() >= 3) {
    191191                // More than 3 nodes and way(s) selected -> align selected nodes. Don't care of way(s).
    192192                cmd = alignOnlyNodes(selectedNodes);
     
    219219        Collection<Command> cmds = new ArrayList<>(nodes.size());
    220220        Line line = new Line(anchors[0], anchors[1]);
    221         for(Node node: nodes)
    222             if(node != anchors[0] && node != anchors[1])
     221        for (Node node: nodes)
     222            if (node != anchors[0] && node != anchors[1])
    223223                cmds.add(line.projectionCommand(node));
    224224        return new SequenceCommand(tr("Align Nodes in Line"), cmds);
     
    271271        List<Line> lines = new ArrayList<>();
    272272        List<Node> neighbors = new ArrayList<>();
    273         for(Way way: refWays) {
     273        for (Way way: refWays) {
    274274            List<Node> nodes = way.getNodes();
    275275            neighbors.clear();
    276             for(int i = 1; i < nodes.size()-1; i++)
    277                 if(nodes.get(i) == node) {
     276            for (int i = 1; i < nodes.size()-1; i++)
     277                if (nodes.get(i) == node) {
    278278                    neighbors.add(nodes.get(i-1));
    279279                    neighbors.add(nodes.get(i+1));
    280280                }
    281             if(neighbors.isEmpty())
     281            if (neighbors.isEmpty())
    282282                continue;
    283             else if(neighbors.size() == 2)
     283            else if (neighbors.size() == 2)
    284284                // Non self crossing
    285285                lines.add(new Line(neighbors.get(0), neighbors.get(1)));
    286             else if(neighbors.size() == 4) {
     286            else if (neighbors.size() == 4) {
    287287                // Self crossing, have to make 2 lines with 4 neighbors
    288288                // see #9081 comment 6
    289289                EastNorth c = node.getEastNorth();
    290290                double[] angle = new double[4];
    291                 for(int i = 0; i < 4; i++) {
     291                for (int i = 0; i < 4; i++) {
    292292                    EastNorth p = neighbors.get(i).getEastNorth();
    293293                    angle[i] = Math.atan2(p.north() - c.north(), p.east() - c.east());
    294294                }
    295295                double[] deltaAngle = new double[3];
    296                 for(int i = 0; i < 3; i++) {
     296                for (int i = 0; i < 3; i++) {
    297297                    deltaAngle[i] = angle[i+1] - angle[0];
    298                     if(deltaAngle[i] < 0)
     298                    if (deltaAngle[i] < 0)
    299299                        deltaAngle[i] += 2*Math.PI;
    300300                }
    301301                int nb = 0;
    302                 if(deltaAngle[1] < deltaAngle[0]) nb++;
    303                 if(deltaAngle[2] < deltaAngle[0]) nb++;
    304                 if(nb == 1) {
     302                if (deltaAngle[1] < deltaAngle[0]) nb++;
     303                if (deltaAngle[2] < deltaAngle[0]) nb++;
     304                if (nb == 1) {
    305305                    // Align along [neighbors[0], neighbors[1]] and [neighbors[0], neighbors[2]]
    306306                    lines.add(new Line(neighbors.get(0), neighbors.get(1)));
     
    325325     */
    326326    private Command alignSingleNode(Node node, List<Line> lines) throws InvalidSelection {
    327         if(lines.size() == 1)
     327        if (lines.size() == 1)
    328328            return lines.get(0).projectionCommand(node);
    329         else if(lines.size() == 2)
     329        else if (lines.size() == 2)
    330330            return lines.get(0).intersectionCommand(node,  lines.get(1));
    331331        throw new InvalidSelection();
     
    396396        public Command intersectionCommand(Node n, Line other) throws InvalidSelection {
    397397            double d = this.a * other.b - other.a * this.b;
    398             if(Math.abs(d) < 10e-6)
     398            if (Math.abs(d) < 10e-6)
    399399                // parallels lines
    400400                throw new InvalidSelection(tr("Two parallels ways found. Abort."));
  • trunk/src/org/openstreetmap/josm/actions/AutoScaleAction.java

    r8345 r8510  
    270270            }
    271271            DataSet dataset = Main.main.getCurrentDataSet();
    272             if(dataset != null) {
     272            if (dataset != null) {
    273273                List<DataSource> dataSources = new ArrayList<>(dataset.getDataSources());
    274274                int s = dataSources.size();
    275                 if(s > 0) {
    276                     if(lastZoomTime == -1 || lastZoomArea == -1 || lastZoomArea > s) {
     275                if (s > 0) {
     276                    if (lastZoomTime == -1 || lastZoomArea == -1 || lastZoomArea > s) {
    277277                        lastZoomArea = s-1;
    278278                        v.visit(dataSources.get(lastZoomArea).bounds);
    279                     } else if(lastZoomArea > 0) {
     279                    } else if (lastZoomArea > 0) {
    280280                        lastZoomArea -= 1;
    281281                        v.visit(dataSources.get(lastZoomArea).bounds);
  • trunk/src/org/openstreetmap/josm/actions/CloseChangesetAction.java

    r8419 r8510  
    4848
    4949    }
     50
    5051    @Override
    5152    public void actionPerformed(ActionEvent e) {
     
    136137                        getProgressMonitor().createSubTaskMonitor(1, false /* not internal */)
    137138                );
    138             } catch(Exception e) {
     139            } catch (Exception e) {
    139140                if (canceled)
    140141                    return;
  • trunk/src/org/openstreetmap/josm/actions/CombineWayAction.java

    r8459 r8510  
    220220        final Way selectedWay = combineResult.a;
    221221        Main.main.undoRedo.add(combineResult.b);
    222         if(selectedWay != null) {
     222        if (selectedWay != null) {
    223223            Runnable guiTask = new Runnable() {
    224224                @Override
     
    272272         * @param pair An existing {@code Pair} of nodes
    273273         */
    274         public NodePair(Pair<Node,Node> pair) {
     274        public NodePair(Pair<Node, Node> pair) {
    275275            this(pair.a, pair.b);
    276276        }
     
    317317
    318318        public NodePair swap() {
    319             return new NodePair(b,a);
     319            return new NodePair(b, a);
    320320        }
    321321
     
    375375        public static List<NodePair> buildNodePairs(Way way, boolean directed) {
    376376            List<NodePair> pairs = new ArrayList<>();
    377             for (Pair<Node,Node> pair: way.getNodePairs(false /* don't sort */)) {
     377            for (Pair<Node, Node> pair: way.getNodePairs(false /* don't sort */)) {
    378378                pairs.add(new NodePair(pair));
    379379                if (!directed) {
     
    394394        public static List<NodePair> eliminateDuplicateNodePairs(List<NodePair> pairs) {
    395395            List<NodePair> cleaned = new ArrayList<>();
    396             for(NodePair p: pairs) {
     396            for (NodePair p: pairs) {
    397397                if (!cleaned.contains(p) && !cleaned.contains(p.swap())) {
    398398                    cleaned.add(p);
     
    451451            NodeGraph graph = new NodeGraph();
    452452            for (Way w: ways) {
    453                 if(!w.isNew()) {
     453                if (!w.isNew()) {
    454454                    /* let the first non-new way give the direction (see #5880) */
    455455                    graph.add(buildNodePairs(w, dir));
     
    540540            Set<Node> nodes = getNodes();
    541541            for (Node n: nodes) {
    542                 if (successors.get(n) != null && successors.get(n).size() ==1)
     542                if (successors.get(n) != null && successors.get(n).size() == 1)
    543543                    return n;
    544544            }
     
    612612            Stack<NodePair> nextPairs  = new Stack<>();
    613613            nextPairs.addAll(getOutboundPairs(startNode));
    614             while(!nextPairs.isEmpty()) {
    615                 NodePair cur= nextPairs.pop();
     614            while (!nextPairs.isEmpty()) {
     615                NodePair cur = nextPairs.pop();
    616616                if (!path.contains(cur) && !path.contains(cur.swap())) {
    617                     while(!path.isEmpty() && !path.peek().isPredecessorOf(cur)) {
     617                    while (!path.isEmpty() && !path.peek().isPredecessorOf(cur)) {
    618618                        path.pop();
    619619                    }
  • trunk/src/org/openstreetmap/josm/actions/CopyAction.java

    r8390 r8510  
    4343    @Override
    4444    public void actionPerformed(ActionEvent e) {
    45         if(isEmptySelection()) return;
     45        if (isEmptySelection()) return;
    4646        Collection<OsmPrimitive> selection = getCurrentDataSet().getSelected();
    4747
  • trunk/src/org/openstreetmap/josm/actions/CreateCircleAction.java

    r8419 r8510  
    7373        double[] width = new double[angles.length];
    7474        double[] remainder = new double[angles.length];
    75         for(int i = 0; i < angles.length; i++) {
     75        for (int i = 0; i < angles.length; i++) {
    7676            width[i] = angles[(i+1) % angles.length].a - angles[i].a;
    77             if(width[i] < 0)
     77            if (width[i] < 0)
    7878                width[i] += 2*Math.PI;
    7979        }
    8080        int assign = 0;
    81         for(int i = 0; i < angles.length; i++) {
     81        for (int i = 0; i < angles.length; i++) {
    8282            double part = width[i] / 2.0 / Math.PI * nodesCount;
    8383            count[i] = (int) Math.floor(part);
     
    8585            assign += count[i];
    8686        }
    87         while(assign < nodesCount) {
     87        while (assign < nodesCount) {
    8888            int imax = 0;
    89             for(int i = 1; i < angles.length; i++)
    90                 if(remainder[i] > remainder[imax])
     89            for (int i = 1; i < angles.length; i++)
     90                if (remainder[i] > remainder[imax])
    9191                    imax = i;
    9292            count[imax]++;
     
    116116    private static class PolarNodeComparator implements Comparator<PolarNode>, Serializable {
    117117        private static final long serialVersionUID = 1L;
     118
    118119        @Override
    119120        public int compare(PolarNode pc1, PolarNode pc2) {
     
    145146            existingWay = ways.get(0);
    146147            for (Node n : existingWay.getNodes()) {
    147                 if(!nodes.contains(n)) {
     148                if (!nodes.contains(n)) {
    148149                    nodes.add(n);
    149150                }
     
    188189        // calculate the radius (r)
    189190        EastNorth n1 = nodes.get(0).getEastNorth();
    190         double r = Math.sqrt(Math.pow(center.east()-n1.east(),2) +
    191                 Math.pow(center.north()-n1.north(),2));
     191        double r = Math.sqrt(Math.pow(center.east()-n1.east(), 2) +
     192                Math.pow(center.north()-n1.north(), 2));
    192193
    193194        // Order nodes by angle
    194195        PolarNode[] angles = new PolarNode[nodes.size()];
    195         for(int i = 0; i < nodes.size(); i++) {
     196        for (int i = 0; i < nodes.size(); i++) {
    196197            angles[i] = new PolarNode(center, nodes.get(i));
    197198        }
     
    202203        // build a way for the circle
    203204        List<Node> nodesToAdd = new ArrayList<>();
    204         for(int i = 0; i < nodes.size(); i++) {
     205        for (int i = 0; i < nodes.size(); i++) {
    205206            nodesToAdd.add(angles[i].node);
    206207            double delta = angles[(i+1) % nodes.size()].a - angles[i].a;
    207             if(delta < 0)
     208            if (delta < 0)
    208209                delta += 2*Math.PI;
    209             for(int j = 0; j < count[i]; j++) {
     210            for (int j = 0; j < count[i]; j++) {
    210211                double alpha = angles[i].a + (j+1)*delta/(count[i]+1);
    211212                double x = center.east() + r*Math.cos(alpha);
    212213                double y = center.north() + r*Math.sin(alpha);
    213                 LatLon ll = Main.getProjection().eastNorth2latlon(new EastNorth(x,y));
     214                LatLon ll = Main.getProjection().eastNorth2latlon(new EastNorth(x, y));
    214215                if (ll.isOutSideWorld()) {
    215216                    notifyNodesNotOnCircle();
  • trunk/src/org/openstreetmap/josm/actions/DialogsToggleAction.java

    r7937 r8510  
    2626              (ImageProvider) null, /* no icon */
    2727              tr("Toggle dialogs panel, maximize mapview"),
    28               Shortcut.registerShortcut("menu:view:dialogspanel", tr("Toggle dialogs panel"),KeyEvent.VK_TAB, Shortcut.DIRECT),
     28              Shortcut.registerShortcut("menu:view:dialogspanel", tr("Toggle dialogs panel"), KeyEvent.VK_TAB, Shortcut.DIRECT),
    2929              true, "dialogspanel", /* register in toolbar */
    3030              false
  • trunk/src/org/openstreetmap/josm/actions/DistributeAction.java

    r8467 r8510  
    6666        Collection<Way> ways = new LinkedList<>();
    6767        Collection<Node> nodes = new HashSet<>();
    68         for(OsmPrimitive osm : selected) {
    69             if(osm instanceof Node) {
     68        for (OsmPrimitive osm : selected) {
     69            if (osm instanceof Node) {
    7070                nodes.add((Node) osm);
    71             } else if(osm instanceof Way) {
     71            } else if (osm instanceof Way) {
    7272                ways.add((Way) osm);
    7373            }
     
    8282        // Switch between algorithms
    8383        Collection<Command> cmds;
    84         if(checkDistributeWay(ways, nodes)) {
     84        if (checkDistributeWay(ways, nodes)) {
    8585            cmds = distributeWay(ways, nodes);
    86         } else if(checkDistributeNodes(ways, nodes)) {
     86        } else if (checkDistributeNodes(ways, nodes)) {
    8787            cmds = distributeNodes(nodes);
    8888        } else {
     
    9797        }
    9898
    99         if(cmds.isEmpty()) {
     99        if (cmds.isEmpty()) {
    100100            return;
    101101        }
     
    113113     */
    114114    private boolean checkDistributeWay(Collection<Way> ways, Collection<Node> nodes) {
    115         if(ways.size() == 1 && nodes.size() <= 2) {
     115        if (ways.size() == 1 && nodes.size() <= 2) {
    116116            Way w = ways.iterator().next();
    117117            Set<Node> unduplicated = new HashSet<>(w.getNodes());
    118             if(unduplicated.size() != w.getNodesCount()) {
     118            if (unduplicated.size() != w.getNodesCount()) {
    119119                // No self crossing way
    120120                return false;
    121121            }
    122             for(Node node: nodes) {
    123                 if(!w.containsNode(node)) {
     122            for (Node node: nodes) {
     123                if (!w.containsNode(node)) {
    124124                    return false;
    125125                }
     
    142142        Collection<Command> cmds = new LinkedList<>();
    143143
    144         if(w.getNodesCount() == nodes.size() || w.getNodesCount() <= 2) {
     144        if (w.getNodesCount() == nodes.size() || w.getNodesCount() <= 2) {
    145145            // Nothing to do
    146146            return cmds;
     
    149149        double xa, ya; // Start point
    150150        double dx, dy; // Segment increment
    151         if(nodes.isEmpty()) {
     151        if (nodes.isEmpty()) {
    152152            Node na = w.firstNode();
    153153            nodes.add(na);
     
    158158            dx = (nb.getEastNorth().east() - xa) / (w.getNodesCount() - 1);
    159159            dy = (nb.getEastNorth().north() - ya) / (w.getNodesCount() - 1);
    160         } else if(nodes.size() == 1) {
     160        } else if (nodes.size() == 1) {
    161161            Node n = nodes.iterator().next();
    162162            int nIdx = w.getNodes().indexOf(n);
     
    182182        }
    183183
    184         for(int i = 0; i < w.getNodesCount(); i++) {
     184        for (int i = 0; i < w.getNodesCount(); i++) {
    185185            Node n = w.getNode(i);
    186186            if (!n.isLatLonKnown() || nodes.contains(n)) {
  • trunk/src/org/openstreetmap/josm/actions/DownloadAction.java

    r8443 r8510  
    5252            if (dialog.isDownloadGpxData()) {
    5353                DownloadGpsTask task = new DownloadGpsTask();
    54                 Future<?> future = task.download(dialog.isNewLayerRequired(),area, null);
     54                Future<?> future = task.download(dialog.isNewLayerRequired(), area, null);
    5555                Main.worker.submit(new PostDownloadHandler(task, future));
    5656            }
  • trunk/src/org/openstreetmap/josm/actions/DownloadPrimitiveAction.java

    r8509 r8510  
    5757            public void run() {
    5858                final List<PrimitiveId> downloaded = task.getDownloadedId();
    59                 if(downloaded != null) {
     59                if (downloaded != null) {
    6060                    GuiHelper.runInEDT(new Runnable() {
    6161                        @Override
  • trunk/src/org/openstreetmap/josm/actions/DownloadReferrersAction.java

    r8291 r8510  
    9090            return;
    9191        Collection<OsmPrimitive> primitives = layer.data.getSelected();
    92         downloadReferrers(layer,primitives);
     92        downloadReferrers(layer, primitives);
    9393    }
    9494
  • trunk/src/org/openstreetmap/josm/actions/ExtensionFileFilter.java

    r8404 r8510  
    104104        for (String classname : exporterNames) {
    105105            try {
    106                 FileExporter exporter = (FileExporter)Class.forName(classname).newInstance();
     106                FileExporter exporter = (FileExporter) Class.forName(classname).newInstance();
    107107                exporters.add(exporter);
    108108                MapView.addLayerChangeListener(exporter);
     
    148148     */
    149149    public static void updateAllFormatsImporter() {
    150         for(int i=0; i < importers.size(); i++) {
    151             if(importers.get(i) instanceof AllFormatsImporter) {
     150        for (int i = 0; i < importers.size(); i++) {
     151            if (importers.get(i) instanceof AllFormatsImporter) {
    152152                importers.set(i, new AllFormatsImporter());
    153153            }
  • trunk/src/org/openstreetmap/josm/actions/FullscreenToggleAction.java

    r8308 r8510  
    3838              null, /* no icon */
    3939              tr("Toggle fullscreen view"),
    40               Shortcut.registerShortcut("menu:view:fullscreen", tr("Toggle fullscreen view"),KeyEvent.VK_F11, Shortcut.DIRECT),
     40              Shortcut.registerShortcut("menu:view:fullscreen", tr("Toggle fullscreen view"), KeyEvent.VK_F11, Shortcut.DIRECT),
    4141              false /* register */
    4242        );
  • trunk/src/org/openstreetmap/josm/actions/GpxExportAction.java

    r8509 r8510  
    4343     */
    4444    protected Layer getLayer() {
    45         if(!Main.isDisplayingMapView())
     45        if (!Main.isDisplayingMapView())
    4646            return null;
    4747        Layer layer = Main.map.mapView.getActiveLayer();
  • trunk/src/org/openstreetmap/josm/actions/HelpAction.java

    r7694 r8510  
    3838            String topic;
    3939            if (e.getSource() instanceof Component) {
    40                 Component c = SwingUtilities.getRoot((Component)e.getSource());
     40                Component c = SwingUtilities.getRoot((Component) e.getSource());
    4141                Point mouse = c.getMousePosition();
    4242                if (mouse != null) {
  • trunk/src/org/openstreetmap/josm/actions/ImageryAdjustAction.java

    r8443 r8510  
    119119            return;
    120120        }
    121         KeyEvent kev = (KeyEvent)event;
     121        KeyEvent kev = (KeyEvent) event;
    122122        int dx = 0, dy = 0;
    123123        switch (kev.getKeyCode()) {
     
    148148        if (layer.isVisible()) {
    149149            requestFocusInMapView();
    150             prevEastNorth=Main.map.mapView.getEastNorth(e.getX(),e.getY());
     150            prevEastNorth = Main.map.mapView.getEastNorth(e.getX(), e.getY());
    151151            Main.map.mapView.setNewCursor(Cursor.MOVE_CURSOR, this);
    152152        }
     
    157157        if (layer == null || prevEastNorth == null) return;
    158158        EastNorth eastNorth =
    159             Main.map.mapView.getEastNorth(e.getX(),e.getY());
     159            Main.map.mapView.getEastNorth(e.getX(), e.getY());
    160160        double dx = layer.getDx()+eastNorth.east()-prevEastNorth.east();
    161161        double dy = layer.getDy()+eastNorth.north()-prevEastNorth.north();
     
    194194            super(Main.parent,
    195195                    tr("Adjust imagery offset"),
    196                     new String[] {tr("OK"),tr("Cancel")},
     196                    new String[] {tr("OK"), tr("Cancel")},
    197197                    false);
    198198            setButtonIcons(new String[] {"ok", "cancel"});
     
    203203                    "If you want to save the offset as bookmark, enter the bookmark name below",
    204204                    Main.getProjection().toString())), GBC.eop());
    205             pnl.add(new JLabel(tr("Offset: ")),GBC.std());
    206             pnl.add(tOffset,GBC.eol().fill(GBC.HORIZONTAL).insets(0,0,0,5));
    207             pnl.add(new JLabel(tr("Bookmark name: ")),GBC.std());
    208             pnl.add(tBookmarkName,GBC.eol().fill(GBC.HORIZONTAL));
     205            pnl.add(new JLabel(tr("Offset: ")), GBC.std());
     206            pnl.add(tOffset, GBC.eol().fill(GBC.HORIZONTAL).insets(0, 0, 0, 5));
     207            pnl.add(new JLabel(tr("Bookmark name: ")), GBC.std());
     208            pnl.add(tBookmarkName, GBC.eol().fill(GBC.HORIZONTAL));
    209209            tOffset.setColumns(16);
    210210            updateOffsetIntl();
     
    269269                    tr("Overwrite"),
    270270                    new String[] {tr("Overwrite"), tr("Cancel")}
    271             ) {{
     271            ) { {
    272272                contentInsets = new Insets(10, 15, 10, 15);
    273             }};
     273            } };
    274274            dialog.setContent(tr("Offset bookmark already exists. Overwrite?"));
    275275            dialog.setButtonIcons(new String[] {"ok.png", "cancel.png"});
  • trunk/src/org/openstreetmap/josm/actions/InfoWebAction.java

    r8475 r8510  
    3535    protected String createInfoUrl(Object infoObject) {
    3636        if (infoObject instanceof OsmPrimitive) {
    37             OsmPrimitive primitive = (OsmPrimitive)infoObject;
     37            OsmPrimitive primitive = (OsmPrimitive) infoObject;
    3838            return Main.getBaseBrowseUrl() + "/" + OsmPrimitiveType.from(primitive).getAPIName() + "/" + primitive.getId();
    3939        } else if (infoObject instanceof Note) {
    40             Note note = (Note)infoObject;
     40            Note note = (Note) infoObject;
    4141            return Main.getBaseBrowseUrl() + "/note/" + note.getId();
    4242        } else {
  • trunk/src/org/openstreetmap/josm/actions/JoinAreasAction.java

    r8486 r8510  
    8484        public final Relation rel;
    8585        public final String role;
     86
    8687        public RelationRole(Relation rel, String role) {
    8788            this.rel = rel;
     
    101102        }
    102103    }
    103 
    104104
    105105    /**
     
    165165         */
    166166        public void reverse() {
    167             for(WayInPolygon way: ways)
     167            for (WayInPolygon way: ways)
    168168                way.insideToTheRight = !way.insideToTheRight;
    169169            Collections.reverse(ways);
     
    266266            double angle = Math.atan2(en3.getY() - en1.getY(), en3.getX() - en1.getX()) -
    267267                    Math.atan2(en2.getY() - en1.getY(), en2.getX() - en1.getX());
    268             while(angle >= 2*Math.PI)
     268            while (angle >= 2*Math.PI)
    269269                angle -= 2*Math.PI;
    270             while(angle < 0)
     270            while (angle < 0)
    271271                angle += 2*Math.PI;
    272272            return angle;
     
    301301                }
    302302
    303                 if(nextNode == prevNode) {
     303                if (nextNode == prevNode) {
    304304                    // go back
    305305                    lastWay = way;
     
    310310                double angle = Math.atan2(nextNode.getEastNorth().east() - headNode.getEastNorth().east(),
    311311                        nextNode.getEastNorth().north() - headNode.getEastNorth().north()) - headAngle;
    312                 if(angle > Math.PI)
     312                if (angle > Math.PI)
    313313                    angle -= 2*Math.PI;
    314                 if(angle <= -Math.PI)
     314                if (angle <= -Math.PI)
    315315                    angle += 2*Math.PI;
    316316
     
    345345                Node candidatePrevNode;
    346346
    347                 if(candidateWay.way.firstNode().equals(headNode)) {
     347                if (candidateWay.way.firstNode().equals(headNode)) {
    348348                    candidateComingToHead = !candidateWay.insideToTheRight;
    349349                    candidatePrevNode = candidateWay.way.getNode(1);
    350                 } else if(candidateWay.way.lastNode().equals(headNode)) {
     350                } else if (candidateWay.way.lastNode().equals(headNode)) {
    351351                     candidateComingToHead = candidateWay.insideToTheRight;
    352352                     candidatePrevNode = candidateWay.way.getNode(candidateWay.way.getNodesCount() - 2);
    353353                } else
    354354                    continue;
    355                 if(candidateWay.equals(lastWay) && candidateComingToHead)
     355                if (candidateWay.equals(lastWay) && candidateComingToHead)
    356356                    continue;
    357357
    358358                double candidateAngle = getAngle(headNode, candidatePrevNode, prevNode);
    359359
    360                 if(mostLeft == null || candidateAngle < angle || (Utils.equalsEpsilon(candidateAngle, angle) && !candidateComingToHead)) {
     360                if (mostLeft == null || candidateAngle < angle || (Utils.equalsEpsilon(candidateAngle, angle) && !candidateComingToHead)) {
    361361                    // Candidate is most left
    362362                    mostLeft = candidateWay;
     
    441441                tr("The selected area is incomplete. Continue?"),
    442442                allNodes, null);
    443         if(!ok) return;
     443        if (!ok) return;
    444444
    445445        //analyze multipolygon relations and collect all areas
     
    614614        commitCommands(marktr("Assemble new polygons"));
    615615
    616         for(Relation rel: relationsToDelete) {
     616        for (Relation rel: relationsToDelete) {
    617617            cmds.add(new DeleteCommand(rel));
    618618        }
     
    725725                }
    726726
    727                 Way newWay=new Way(way);
     727                Way newWay = new Way(way);
    728728                newWay.setNodes(newNodes);
    729729                cmds.add(new ChangeCommand(way, newWay));
     
    10801080
    10811081        WayInPolygon startWay;
    1082         while((startWay = traverser.startNewWay()) != null) {
     1082        while ((startWay = traverser.startNewWay()) != null) {
    10831083            List<WayInPolygon> path = new ArrayList<>();
    10841084            List<WayInPolygon> startWays = new ArrayList<>();
    10851085            path.add(startWay);
    1086             while(true) {
     1086            while (true) {
    10871087                WayInPolygon leftComing;
    1088                 while((leftComing = traverser.leftComingWay()) != null) {
    1089                     if(startWays.contains(leftComing))
     1088                while ((leftComing = traverser.leftComingWay()) != null) {
     1089                    if (startWays.contains(leftComing))
    10901090                        break;
    10911091                    // Need restart traverser walk
     
    10971097                }
    10981098                WayInPolygon nextWay = traverser.walk();
    1099                 if(nextWay == null)
     1099                if (nextWay == null)
    11001100                    throw new RuntimeException("Join areas internal error.");
    1101                 if(path.get(0) == nextWay) {
     1101                if (path.get(0) == nextWay) {
    11021102                    // path is closed -> stop here
    11031103                    AssembledPolygon ring = new AssembledPolygon(path);
    1104                     if(ring.getNodes().size() <= 2) {
     1104                    if (ring.getNodes().size() <= 2) {
    11051105                        // Invalid ring (2 nodes) -> remove
    11061106                        traverser.removeWays(path);
    1107                         for(WayInPolygon way: path)
     1107                        for (WayInPolygon way: path)
    11081108                            discardedResult.add(way.way);
    11091109                    } else {
     
    11141114                    break;
    11151115                }
    1116                 if(path.contains(nextWay)) {
     1116                if (path.contains(nextWay)) {
    11171117                    // Inner loop -> remove
    11181118                    int index = path.indexOf(nextWay);
    1119                     while(path.size() > index) {
     1119                    while (path.size() > index) {
    11201120                        WayInPolygon currentWay = path.get(index);
    11211121                        discardedResult.add(currentWay.way);
     
    11451145            WayInPolygon startWay;
    11461146
    1147             while((startWay = traverser.startNewWay()) != null) {
     1147            while ((startWay = traverser.startNewWay()) != null) {
    11481148                List<WayInPolygon> simpleRingWays = new ArrayList<>();
    11491149                simpleRingWays.add(startWay);
    11501150                WayInPolygon nextWay;
    1151                 while((nextWay = traverser.walk()) != startWay) {
    1152                     if(nextWay == null)
     1151                while ((nextWay = traverser.walk()) != startWay) {
     1152                    if (nextWay == null)
    11531153                        throw new RuntimeException("Join areas internal error.");
    11541154                    simpleRingWays.add(nextWay);
     
    12331233     * @return The newly created way
    12341234     */
    1235     private Way joinOrientedWays(List<WayInPolygon> ways) throws UserCancelException{
     1235    private Way joinOrientedWays(List<WayInPolygon> ways) throws UserCancelException {
    12361236        if (ways.size() < 2)
    12371237            return ways.get(0).way;
  • trunk/src/org/openstreetmap/josm/actions/JoinNodeWayAction.java

    r8447 r8510  
    183183        private EastNorth refPoint2;
    184184        private final boolean projectToSegment;
     185
    185186        NodeDistanceToRefNodeComparator(Node referenceNode, Node referenceNode2, boolean projectFirst) {
    186187            refPoint = referenceNode.getEastNorth();
     
    188189            projectToSegment = projectFirst;
    189190        }
     191
    190192        @Override
    191193        public int compare(Node first, Node second) {
  • trunk/src/org/openstreetmap/josm/actions/JosmAction.java

    r8509 r8510  
    7676            String toolbarId, boolean installAdapters) {
    7777        super(name);
    78         if(icon != null)
     78        if (icon != null)
    7979            icon.getResource().getImageIcon(this);
    8080        setHelpId();
     
    282282            });
    283283        }
     284
    284285        @Override
    285286        public void activeLayerChange(Layer oldLayer, Layer newLayer) {
  • trunk/src/org/openstreetmap/josm/actions/JumpToAction.java

    r8509 r8510  
    5050
    5151    class OsmURLListener implements DocumentListener {
    52         @Override public void changedUpdate(DocumentEvent e) { parseURL(); }
    53         @Override public void insertUpdate(DocumentEvent e) { parseURL(); }
    54         @Override public void removeUpdate(DocumentEvent e) { parseURL(); }
     52        @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(); }
    5560    }
    5661
    5762    class OsmLonLatListener implements DocumentListener {
    58         @Override public void changedUpdate(DocumentEvent e) { updateUrl(false); }
    59         @Override public void insertUpdate(DocumentEvent e) { updateUrl(false); }
    60         @Override public void removeUpdate(DocumentEvent e) { updateUrl(false); }
     63        @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); }
    6171    }
    6272
     
    112122        LatLon ll = null;
    113123        double zoomLvl = 100;
    114         while(ll == null) {
     124        while (ll == null) {
    115125            int option = JOptionPane.showOptionDialog(
    116126                            Main.parent,
     
    166176
    167177    private void updateUrl(boolean force) {
    168         if(!lat.hasFocus() && !lon.hasFocus() && !zm.hasFocus() && !force) return;
     178        if (!lat.hasFocus() && !lon.hasFocus() && !zm.hasFocus() && !force) return;
    169179        try {
    170180            double dlat = Double.parseDouble(lat.getText());
     
    172182            double m = Double.parseDouble(zm.getText());
    173183            // Inverse function to the one above. 18 is the current maximum zoom
    174             // available on standard renderers, so choose this is in case m
    175             // should be zero
     184            // available on standard renderers, so choose this is in case m should be zero
    176185            int zoomLvl = 18;
    177             if(m > 0)
    178                 zoomLvl = (int)Math.round((-1) * Math.log(m/10000000)/Math.log(2));
     186            if (m > 0)
     187                zoomLvl = (int) Math.round((-1) * Math.log(m/10000000)/Math.log(2));
    179188
    180189            url.setText(OsmUrlToBounds.getURL(dlat, dlon, zoomLvl));
  • trunk/src/org/openstreetmap/josm/actions/MapRectifierWMSmenuAction.java

    r8308 r8510  
    129129
    130130        JRadioButton firstBtn = null;
    131         for(RectifierService s : services) {
     131        for (RectifierService s : services) {
    132132            JRadioButton serviceBtn = new JRadioButton(s.name);
    133             if(firstBtn == null) {
     133            if (firstBtn == null) {
    134134                firstBtn = serviceBtn;
    135135            }
     
    137137            // If the contents match, they will be inserted into the text field and the corresponding
    138138            // service will be pre-selected.
    139             if(!clip.isEmpty() && tfWmsUrl.getText().isEmpty()
     139            if (!clip.isEmpty() && tfWmsUrl.getText().isEmpty()
    140140                    && (s.urlRegEx.matcher(clip).find() || s.idValidator.matcher(clip).matches())) {
    141141                serviceBtn.setSelected(true);
     
    144144            s.btn = serviceBtn;
    145145            group.add(serviceBtn);
    146             if(!s.url.isEmpty()) {
     146            if (!s.url.isEmpty()) {
    147147                panel.add(serviceBtn, GBC.std());
    148148                panel.add(new UrlLabel(s.url, tr("Visit Homepage")), GBC.eol().anchor(GridBagConstraints.EAST));
     
    153153
    154154        // Fallback in case no match was found
    155         if(tfWmsUrl.getText().isEmpty() && firstBtn != null) {
     155        if (tfWmsUrl.getText().isEmpty() && firstBtn != null) {
    156156            firstBtn.setSelected(true);
    157157        }
     
    169169        // This repeatedly shows the dialog in case there has been an error.
    170170        // The loop is break;-ed if the users cancels
    171         outer: while(true) {
     171        outer: while (true) {
    172172            diag.showDialog();
    173173            int answer = diag.getValue();
    174174            // Break loop when the user cancels
    175             if(answer != 1) {
     175            if (answer != 1) {
    176176                break;
    177177            }
     
    179179            String text = tfWmsUrl.getText().trim();
    180180            // Loop all services until we find the selected one
    181             for(RectifierService s : services) {
    182                 if(!s.isSelected()) {
     181            for (RectifierService s : services) {
     182                if (!s.isSelected()) {
    183183                    continue;
    184184                }
     
    186186                // We've reached the custom WMS URL service
    187187                // Just set the URL and hope everything works out
    188                 if(s.wmsUrl.isEmpty()) {
     188                if (s.wmsUrl.isEmpty()) {
    189189                    try {
    190190                        addWMSLayer(s.name + " (" + text + ")", text);
     
    197197                // First try to match if the entered string as an URL
    198198                Matcher m = s.urlRegEx.matcher(text);
    199                 if(m.find()) {
     199                if (m.find()) {
    200200                    String id = m.group(1);
    201201                    String newURL = s.wmsUrl.replaceAll("__s__", id);
     
    205205                }
    206206                // If not, look if it's a valid ID for the selected service
    207                 if(s.idValidator.matcher(text).matches()) {
     207                if (s.idValidator.matcher(text).matches()) {
    208208                    String newURL = s.wmsUrl.replaceAll("__s__", text);
    209209                    String title = s.name + " (" + text + ")";
     
    217217            }
    218218
    219             // and display an error message. The while(true) ensures that the dialog pops up again
     219            // and display an error message. The while (true) ensures that the dialog pops up again
    220220            JOptionPane.showMessageDialog(Main.parent,
    221221                    tr("Couldn''t match the entered link or id to the selected service. Please try again."),
  • trunk/src/org/openstreetmap/josm/actions/MergeLayerAction.java

    r7859 r8510  
    4848                    if (sourceLayer != null && !sourceLayer.equals(targetLayer)) {
    4949                        if (sourceLayer instanceof OsmDataLayer && targetLayer instanceof OsmDataLayer
    50                                 && ((OsmDataLayer)sourceLayer).isUploadDiscouraged() != ((OsmDataLayer)targetLayer).isUploadDiscouraged()) {
     50                                && ((OsmDataLayer) sourceLayer).isUploadDiscouraged() != ((OsmDataLayer) targetLayer).isUploadDiscouraged()) {
    5151                            if (warnMergingUploadDiscouragedLayers(sourceLayer, targetLayer)) {
    5252                                break;
  • trunk/src/org/openstreetmap/josm/actions/MergeNodesAction.java

    r8443 r8510  
    236236                    return null;
    237237                }
    238             } else if(newNodes.size() < 2 && w.getReferrers().isEmpty()) {
     238            } else if (newNodes.size() < 2 && w.getReferrers().isEmpty()) {
    239239                waysToDelete.add(w);
    240240            } else {
  • trunk/src/org/openstreetmap/josm/actions/MergeSelectionAction.java

    r7859 r8510  
    5050            return;
    5151        if (getEditLayer().isUploadDiscouraged() && targetLayer instanceof OsmDataLayer
    52                 && !((OsmDataLayer)targetLayer).isUploadDiscouraged()
     52                && !((OsmDataLayer) targetLayer).isUploadDiscouraged()
    5353                && getEditLayer().data.getAllSelected().size() > 1
    5454                && warnMergingUploadDiscouragedObjects(targetLayer)) {
     
    5656        }
    5757        MergeSourceBuildingVisitor builder = new MergeSourceBuildingVisitor(getEditLayer().data);
    58         ((OsmDataLayer)targetLayer).mergeFrom(builder.build());
     58        ((OsmDataLayer) targetLayer).mergeFrom(builder.build());
    5959    }
    6060
  • trunk/src/org/openstreetmap/josm/actions/MirrorAction.java

    r7859 r8510  
    5050        for (OsmPrimitive osm : sel) {
    5151            if (osm instanceof Node) {
    52                 nodes.add((Node)osm);
     52                nodes.add((Node) osm);
    5353            } else if (osm instanceof Way) {
    54                 nodes.addAll(((Way)osm).getNodes());
     54                nodes.addAll(((Way) osm).getNodes());
    5555            }
    5656        }
  • trunk/src/org/openstreetmap/josm/actions/MoveAction.java

    r8378 r8510  
    121121
    122122        getCurrentDataSet().beginUpdate();
    123         if (c instanceof MoveCommand && affectedNodes.equals(((MoveCommand)c).getParticipatingPrimitives())) {
    124             ((MoveCommand)c).moveAgain(distx, disty);
     123        if (c instanceof MoveCommand && affectedNodes.equals(((MoveCommand) c).getParticipatingPrimitives())) {
     124            ((MoveCommand) c).moveAgain(distx, disty);
    125125        } else {
    126126            c = new MoveCommand(selection, distx, disty);
  • trunk/src/org/openstreetmap/josm/actions/OpenLocationAction.java

    r8459 r8510  
    218218        final JList<DownloadTask> list = new JList<>(tasks.toArray(new DownloadTask[tasks.size()]));
    219219        list.addSelectionInterval(0, tasks.size() - 1);
    220         final ExtendedDialog dialog = new ExtendedDialog(Main.parent, tr("Which tasks to perform?"), new String[]{tr("Ok"), tr("Cancel")}, true) {{
     220        final ExtendedDialog dialog = new ExtendedDialog(Main.parent,
     221                tr("Which tasks to perform?"), new String[]{tr("Ok"), tr("Cancel")}, true) { {
    221222            setButtonIcons(new String[]{"ok", "cancel"});
    222223            final JPanel pane = new JPanel(new GridLayout(2, 1));
     
    224225            pane.add(list);
    225226            setContent(pane);
    226         }};
     227        } };
    227228        dialog.showDialog();
    228229        return dialog.getValue() == 1 ? list.getSelectedValuesList() : Collections.<DownloadTask>emptyList();
  • trunk/src/org/openstreetmap/josm/actions/OrthogonalizeAction.java

    r8459 r8510  
    9393                    true, "action/orthogonalize/undo", true);
    9494        }
     95
    9596        @Override
    9697        public void actionPerformed(ActionEvent e) {
     
    101102            try {
    102103                for (OsmPrimitive p : sel) {
    103                     if (!(p instanceof Node)) throw new InvalidUserInputException();
     104                    if (!(p instanceof Node)) throw new InvalidUserInputException("selected object is not a node");
    104105                    Node n = (Node) p;
    105106                    if (rememberMovements.containsKey(n)) {
     
    112113                    Main.main.undoRedo.add(new SequenceCommand(tr("Orthogonalize / Undo"), commands));
    113114                    Main.map.repaint();
    114                 } else throw new InvalidUserInputException();
     115                } else {
     116                    throw new InvalidUserInputException("Commands are empty");
     117                }
    115118            } catch (InvalidUserInputException ex) {
    116119                new Notification(
     
    308311        final Direction[] VERTICAL = {Direction.UP, Direction.DOWN};
    309312        final Direction[][] ORIENTATIONS = {HORIZONTAL, VERTICAL};
    310         for (Direction[] orientation : ORIENTATIONS){
     313        for (Direction[] orientation : ORIENTATIONS) {
    311314            final Set<Node> s = new HashSet<>(allNodes);
    312315            int s_size = s.size();
     
    324327                    somethingHappened = false;
    325328                    for (WayData w : wayDataList) {
    326                         for (int i=0; i < w.nSeg; ++i) {
     329                        for (int i = 0; i < w.nSeg; ++i) {
    327330                            Node n1 = w.way.getNodes().get(i);
    328331                            Node n2 = w.way.getNodes().get(i+1);
     
    407410        // segments turned by 90 degrees
    408411        public double heading;            // heading of segSum == approximate heading of the way
     412
    409413        public WayData(Way pWay) {
    410414            way = pWay;
     
    423427        public void calcDirections(Direction pInitialDirection) throws InvalidUserInputException {
    424428            final EastNorth[] en = new EastNorth[nNode]; // alias: way.getNodes().get(i).getEastNorth() ---> en[i]
    425             for (int i=0; i < nNode; i++) {
     429            for (int i = 0; i < nNode; i++) {
    426430                en[i] = new EastNorth(way.getNodes().get(i).getEastNorth().east(), way.getNodes().get(i).getEastNorth().north());
    427431            }
     
    429433            Direction direction = pInitialDirection;
    430434            segDirections[0] = direction;
    431             for (int i=0; i < nSeg - 1; i++) {
    432                 double h1 = EN.polar(en[i],en[i+1]);
    433                 double h2 = EN.polar(en[i+1],en[i+2]);
     435            for (int i = 0; i < nSeg - 1; i++) {
     436                double h1 = EN.polar(en[i], en[i+1]);
     437                double h2 = EN.polar(en[i+1], en[i+2]);
    434438                try {
    435439                    direction = direction.changeBy(angleToDirectionChange(h2 - h1, TOLERANCE1));
     
    441445
    442446            // sum up segments
    443             EastNorth h = new EastNorth(0.,0.);
    444             EastNorth v = new EastNorth(0.,0.);
     447            EastNorth h = new EastNorth(0., 0.);
     448            EastNorth v = new EastNorth(0., 0.);
    445449            for (int i = 0; i < nSeg; ++i) {
    446450                EastNorth segment = EN.diff(en[i+1], en[i]);
    447451                if      (segDirections[i] == Direction.RIGHT) {
    448                     h = EN.sum(h,segment);
     452                    h = EN.sum(h, segment);
    449453                } else if (segDirections[i] == Direction.UP) {
    450                     v = EN.sum(v,segment);
     454                    v = EN.sum(v, segment);
    451455                } else if (segDirections[i] == Direction.LEFT) {
    452                     h = EN.diff(h,segment);
     456                    h = EN.diff(h, segment);
    453457                } else if (segDirections[i] == Direction.DOWN) {
    454                     v = EN.diff(v,segment);
     458                    v = EN.diff(v, segment);
    455459                } else throw new IllegalStateException();
    456460                /**
     
    470474            segSum = EN.sum(h, new EastNorth(v.north(), -v.east()));
    471475            //            if (EN.abs(segSum) < lh) throw new AssertionError();
    472             this.heading = EN.polar(new EastNorth(0.,0.), segSum);
     476            this.heading = EN.polar(new EastNorth(0., 0.), segSum);
    473477        }
    474478    }
     
    518522            // Hide implicit public constructor for utility class
    519523        }
     524
    520525        /**
    521526         * Rotate counter-clock-wise.
     
    530535            return new EastNorth(nx, ny);
    531536        }
     537
    532538        public static EastNorth sum(EastNorth en1, EastNorth en2) {
    533539            return new EastNorth(en1.east() + en2.east(), en1.north() + en2.north());
    534540        }
     541
    535542        public static EastNorth diff(EastNorth en1, EastNorth en2) {
    536543            return new EastNorth(en1.east() - en2.east(), en1.north() - en2.north());
    537544        }
     545
    538546        public static double polar(EastNorth en1, EastNorth en2) {
    539547            return Math.atan2(en2.north() - en1.north(), en2.east() -  en1.east());
     
    575583            super(message);
    576584        }
     585
    577586        InvalidUserInputException(String message, Throwable cause) {
    578587            super(message, cause);
    579588        }
    580         InvalidUserInputException() {
    581             super();
    582         }
    583     }
     589    }
     590
    584591    /**
    585592     * Exception: angle cannot be recognized as 0, 90, 180 or 270 degrees
  • trunk/src/org/openstreetmap/josm/actions/ParameterizedAction.java

    r5926 r8510  
    99
    1010    List<ActionParameter<?>> getActionParameters();
     11
    1112    void actionPerformed(ActionEvent e, Map<String, Object> parameters);
    12 
    1313}
  • trunk/src/org/openstreetmap/josm/actions/ParameterizedActionDecorator.java

    r7005 r8510  
    2323        action.addPropertyChangeListener(listener);
    2424    }
     25
    2526    @Override
    2627    public Object getValue(String key) {
    2728        return action.getValue(key);
    2829    }
     30
    2931    @Override
    3032    public boolean isEnabled() {
    3133        return action.isEnabled();
    3234    }
     35
    3336    @Override
    3437    public void putValue(String key, Object value) {
    3538        action.putValue(key, value);
    3639    }
     40
    3741    @Override
    3842    public void removePropertyChangeListener(PropertyChangeListener listener) {
    3943        action.removePropertyChangeListener(listener);
    4044    }
     45
    4146    @Override
    4247    public void setEnabled(boolean b) {
    4348        action.setEnabled(b);
    4449    }
     50
    4551    @Override
    4652    public void actionPerformed(ActionEvent e) {
     
    5157        return parameters;
    5258    }
    53 
    5459}
  • trunk/src/org/openstreetmap/josm/actions/PasteAction.java

    r8061 r8510  
    6868        for (PrimitiveData data : pasteBuffer.getAll()) {
    6969            if (data instanceof NodeData) {
    70                 NodeData n = (NodeData)data;
     70                NodeData n = (NodeData) data;
    7171                if (n.getEastNorth() != null) {
    7272                    double east = n.getEastNorth().east();
     
    9797            final Point tl = Main.map.mapView.getLocationOnScreen();
    9898            final Point pos = new Point(mp.x-tl.x, mp.y-tl.y);
    99             if(Main.map.mapView.contains(pos)) {
     99            if (Main.map.mapView.contains(pos)) {
    100100                mPosition = Main.map.mapView.getEastNorth(pos.x, pos.y);
    101101            }
     
    133133        for (PrimitiveData data:bufferCopy) {
    134134            if (data instanceof NodeData) {
    135                 NodeData nodeData = (NodeData)data;
     135                NodeData nodeData = (NodeData) data;
    136136                if (Main.main.getEditLayer() == source) {
    137137                    nodeData.setEastNorth(nodeData.getEastNorth().add(offsetEast, offsetNorth));
     
    139139            } else if (data instanceof WayData) {
    140140                List<Long> newNodes = new ArrayList<>();
    141                 for (Long oldNodeId: ((WayData)data).getNodes()) {
     141                for (Long oldNodeId: ((WayData) data).getNodes()) {
    142142                    Long newNodeId = newNodeIds.get(oldNodeId);
    143143                    if (newNodeId != null) {
     
    145145                    }
    146146                }
    147                 ((WayData)data).setNodes(newNodes);
     147                ((WayData) data).setNodes(newNodes);
    148148            } else if (data instanceof RelationData) {
    149149                List<RelationMemberData> newMembers = new ArrayList<>();
    150                 for (RelationMemberData member: ((RelationData)data).getMembers()) {
     150                for (RelationMemberData member: ((RelationData) data).getMembers()) {
    151151                    OsmPrimitiveType memberType = member.getMemberType();
    152152                    Long newId = null;
     
    166166                    }
    167167                }
    168                 ((RelationData)data).setMembers(newMembers);
     168                ((RelationData) data).setMembers(newMembers);
    169169            }
    170170        }
    171171
    172172        /* Now execute the commands to add the duplicated contents of the paste buffer to the map */
    173 
    174173        Main.main.undoRedo.add(new AddPrimitivesCommand(bufferCopy, toSelect));
    175174        Main.map.mapView.repaint();
  • trunk/src/org/openstreetmap/josm/actions/PasteTagsAction.java

    r8470 r8510  
    263263    public static boolean pasteTagsFromText(Collection<OsmPrimitive> selection, String text) {
    264264        Map<String, String> tags = TextTagParser.readTagsFromText(text);
    265         if (tags==null || tags.isEmpty()) {
     265        if (tags == null || tags.isEmpty()) {
    266266            TextTagParser.showBadBufferMessage(help);
    267267            return false;
     
    272272        for (Entry<String, String> entry: tags.entrySet()) {
    273273            String v = entry.getValue();
    274             commands.add(new ChangePropertyCommand(selection, entry.getKey(), "".equals(v)?null:v));
     274            commands.add(new ChangePropertyCommand(selection, entry.getKey(), "".equals(v) ? null : v));
    275275        }
    276276        commitCommands(selection, commands);
     
    284284    public static boolean pasteTagsFromJOSMBuffer(Collection<OsmPrimitive> selection) {
    285285        List<PrimitiveData> directlyAdded = Main.pasteBuffer.getDirectlyAdded();
    286         if (directlyAdded==null || directlyAdded.isEmpty()) return false;
     286        if (directlyAdded == null || directlyAdded.isEmpty()) return false;
    287287
    288288        PasteTagsAction.TagPaster tagPaster = new PasteTagsAction.TagPaster(directlyAdded, selection);
     
    323323    @Override
    324324    protected void updateEnabledState(Collection<? extends OsmPrimitive> selection) {
    325         setEnabled(selection!= null && !selection.isEmpty());
     325        setEnabled(selection != null && !selection.isEmpty());
    326326    }
    327327}
  • trunk/src/org/openstreetmap/josm/actions/PurgeAction.java

    r8365 r8510  
    181181             */
    182182            List<Relation> relLst = new ArrayList<>(relSet);
    183             for (int i=0; i<relLst.size(); ++i) {
     183            for (int i = 0; i < relLst.size(); ++i) {
    184184                for (OsmPrimitive parent : relLst.get(i).getReferrers()) {
    185185                    if (!(toPurgeChecked.contains(parent))
     
    223223        JPanel pnl = new JPanel(new GridBagLayout());
    224224
    225         pnl.add(Box.createRigidArea(new Dimension(400,0)), GBC.eol().fill(GBC.HORIZONTAL));
     225        pnl.add(Box.createRigidArea(new Dimension(400, 0)), GBC.eol().fill(GBC.HORIZONTAL));
    226226
    227227        pnl.setBorder(BorderFactory.createEmptyBorder(5, 5, 5, 5));
     
    233233
    234234        if (!toPurgeAdditionally.isEmpty()) {
    235             pnl.add(new JSeparator(), GBC.eol().fill(GBC.HORIZONTAL).insets(0,5,0,5));
     235            pnl.add(new JSeparator(), GBC.eol().fill(GBC.HORIZONTAL).insets(0, 5, 0, 5));
    236236            pnl.add(new JLabel("<html>"+
    237237                    tr("The following dependent objects will be purged<br> " +
     
    268268                {
    269269                    putValue(SHORT_DESCRIPTION,   tr("Add to selection"));
    270                     putValue(SMALL_ICON, ImageProvider.get("dialogs","select"));
     270                    putValue(SMALL_ICON, ImageProvider.get("dialogs", "select"));
    271271                }
    272272
     
    276276                }
    277277            });
    278             addToSelection.setMargin(new Insets(0,0,0,0));
    279             pnl.add(addToSelection, GBC.eol().anchor(GBC.SOUTHWEST).weight(0.0, 1.0).insets(2,0,0,3));
     278            addToSelection.setMargin(new Insets(0, 0, 0, 0));
     279            pnl.add(addToSelection, GBC.eol().anchor(GBC.SOUTHWEST).weight(0.0, 1.0).insets(2, 0, 0, 3));
    280280        }
    281281
    282282        if (modified) {
    283             pnl.add(new JSeparator(), GBC.eol().fill(GBC.HORIZONTAL).insets(0,5,0,5));
     283            pnl.add(new JSeparator(), GBC.eol().fill(GBC.HORIZONTAL).insets(0, 5, 0, 5));
    284284            pnl.add(new JLabel("<html>"+tr("Some of the objects are modified.<br> " +
    285285                    "Proceed, if these changes should be discarded."+"</html>"),
     
    291291        cbClearUndoRedo.setSelected(Main.pref.getBoolean("purge.clear_undo_redo", false));
    292292
    293         pnl.add(new JSeparator(), GBC.eol().fill(GBC.HORIZONTAL).insets(0,5,0,5));
     293        pnl.add(new JSeparator(), GBC.eol().fill(GBC.HORIZONTAL).insets(0, 5, 0, 5));
    294294        pnl.add(cbClearUndoRedo, GBC.eol());
    295295        return pnl;
  • trunk/src/org/openstreetmap/josm/actions/RenameLayerAction.java

    r8308 r8510  
    5454        }
    5555
    56         final JOptionPane optionPane = new JOptionPane(panel, JOptionPane.QUESTION_MESSAGE, JOptionPane.OK_CANCEL_OPTION){
     56        final JOptionPane optionPane = new JOptionPane(panel, JOptionPane.QUESTION_MESSAGE, JOptionPane.OK_CANCEL_OPTION) {
    5757            @Override public void selectInitialValue() {
    5858                name.requestFocusInWindow();
     
    6666        Object answer = optionPane.getValue();
    6767        if (answer == null || answer == JOptionPane.UNINITIALIZED_VALUE ||
    68                 (answer instanceof Integer && (Integer)answer != JOptionPane.OK_OPTION))
     68                (answer instanceof Integer && (Integer) answer != JOptionPane.OK_OPTION))
    6969            return;
    7070
  • trunk/src/org/openstreetmap/josm/actions/SaveAction.java

    r8443 r8510  
    3838        File f = layer.getAssociatedFile();
    3939        if (f != null && !f.exists()) {
    40             f=null;
     40            f = null;
    4141        }
    4242
    4343        // Ask for overwrite in case of GpxLayer: GpxLayers usually are imports
    4444        // and modifying is an error most of the time.
    45         if(f != null && layer instanceof GpxLayer) {
     45        if (f != null && layer instanceof GpxLayer) {
    4646            ExtendedDialog dialog = new ExtendedDialog(
    4747                    Main.parent,
  • trunk/src/org/openstreetmap/josm/actions/SaveActionBase.java

    r8419 r8510  
    5252
    5353    public boolean doSave(Layer layer) {
    54         if(!layer.checkSaveConditions())
     54        if (!layer.checkSaveConditions())
    5555            return false;
    5656        file = getFile(layer);
     
    119119        boolean check = Main.isDisplayingMapView()
    120120        && Main.map.mapView.getActiveLayer() != null;
    121         if(!check) {
     121        if (!check) {
    122122            setEnabled(false);
    123123            return;
     
    172172            String fn = file.getPath();
    173173            if (ff instanceof ExtensionFileFilter) {
    174                 fn += "." + ((ExtensionFileFilter)ff).getDefaultExtension();
     174                fn += "." + ((ExtensionFileFilter) ff).getDefaultExtension();
    175175            } else if (extension != null) {
    176176                fn += "." + extension;
  • trunk/src/org/openstreetmap/josm/actions/SelectAllAction.java

    r6380 r8510  
    22package org.openstreetmap.josm.actions;
    33
     4import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
    45import static org.openstreetmap.josm.tools.I18n.tr;
    5 import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
    66
    77import java.awt.event.ActionEvent;
     
    1212public class SelectAllAction extends JosmAction {
    1313
     14    /**
     15     * Constructs a new {@code SelectAllAction}.
     16     */
    1417    public SelectAllAction() {
    15         super(tr("Select All"),"selectall", tr("Select all undeleted objects in the data layer. This selects incomplete objects too."),
     18        super(tr("Select All"), "selectall", tr("Select all undeleted objects in the data layer. This selects incomplete objects too."),
    1619                Shortcut.registerShortcut("system:selectall", tr("Edit: {0}", tr("Select All")), KeyEvent.VK_A, Shortcut.CTRL), true);
    1720        putValue("help", ht("/Action/SelectAll"));
  • trunk/src/org/openstreetmap/josm/actions/SessionSaveAsAction.java

    r8509 r8510  
    193193
    194194            int numNoExporter = 0;
    195             WHILE:while (numNoExporter != noExporter.size()) {
     195            WHILE: while (numNoExporter != noExporter.size()) {
    196196                numNoExporter = noExporter.size();
    197197                for (Layer layer : layers) {
     
    223223                }
    224224                wrapper.add(exportPanel, GBC.std().fill(GBC.HORIZONTAL));
    225                 ip.add(wrapper, GBC.eol().fill(GBC.HORIZONTAL).insets(2,2,4,2));
    226             }
    227             ip.add(GBC.glue(0,1), GBC.eol().fill(GBC.VERTICAL));
     225                ip.add(wrapper, GBC.eol().fill(GBC.HORIZONTAL).insets(2, 2, 4, 2));
     226            }
     227            ip.add(GBC.glue(0, 1), GBC.eol().fill(GBC.VERTICAL));
    228228            JScrollPane sp = new JScrollPane(ip);
    229229            sp.setBorder(BorderFactory.createEmptyBorder());
     
    244244            p.add(include, GBC.std());
    245245            p.add(lbl, GBC.std());
    246             p.add(GBC.glue(1,0), GBC.std().fill(GBC.HORIZONTAL));
     246            p.add(GBC.glue(1, 0), GBC.std().fill(GBC.HORIZONTAL));
    247247            return p;
    248248        }
  • trunk/src/org/openstreetmap/josm/actions/SimplifyWayAction.java

    r8509 r8510  
    174174        int i = 0;
    175175        List<Node> newNodes = new ArrayList<>(w.getNodesCount());
    176         while(i < w.getNodesCount()){
    177             if (isRequiredNode(w,w.getNode(i))) {
     176        while (i < w.getNodesCount()) {
     177            if (isRequiredNode(w, w.getNode(i))) {
    178178                // copy a required node to the list of new nodes. Simplify not possible
    179179                newNodes.add(w.getNode(i));
     
    184184            i++;
    185185            // find the longest sequence of not required nodes ...
    186             while(i<w.getNodesCount() && !isRequiredNode(w,w.getNode(i))) {
     186            while (i < w.getNodesCount() && !isRequiredNode(w, w.getNode(i))) {
    187187                i++;
    188188            }
    189189            // ... and simplify them
    190190            buildSimplifiedNodeList(w.getNodes(), lower, Math.min(w.getNodesCount()-1, i), threshold, newNodes);
    191             lower=i;
     191            lower = i;
    192192            i++;
    193193        }
     
    239239        if (imax != -1 && xtemax >= threshold) {
    240240            // Segment cannot be simplified - try shorter segments
    241             buildSimplifiedNodeList(wnew, from, imax,threshold,simplifiedNodes);
    242             buildSimplifiedNodeList(wnew, imax, to, threshold,simplifiedNodes);
     241            buildSimplifiedNodeList(wnew, from, imax, threshold, simplifiedNodes);
     242            buildSimplifiedNodeList(wnew, imax, to, threshold, simplifiedNodes);
    243243        } else {
    244244            // Simplify segment
  • trunk/src/org/openstreetmap/josm/actions/SplitWayAction.java

    r8465 r8510  
    183183            sel.addAll(selectedWays);
    184184            sel.addAll(selectedRelations);
    185             SplitWayResult result = splitWay(getEditLayer(),selectedWay, wayChunks, sel);
     185            SplitWayResult result = splitWay(getEditLayer(), selectedWay, wayChunks, sel);
    186186            Main.main.undoRedo.add(result.getCommand());
    187187            getCurrentDataSet().setSelected(result.getNewSelection());
     
    226226            new ArrayList<>(OsmPrimitive.getFilteredList(selectedNodes.get(0).getReferrers(),
    227227                                                         Way.class));
    228         for (int i=1; i<selectedNodes.size(); i++) {
     228        for (int i = 1; i < selectedNodes.size(); i++) {
    229229            List<OsmPrimitive> ref = selectedNodes.get(i).getReferrers();
    230230            for (Iterator<Way> it = result.iterator(); it.hasNext();) {
     
    268268     * @return the list of chunks
    269269     */
    270     public static List<List<Node>> buildSplitChunks(Way wayToSplit, List<Node> splitPoints){
     270    public static List<List<Node>> buildSplitChunks(Way wayToSplit, List<Node> splitPoints) {
    271271        CheckParameterUtil.ensureParameterNotNull(wayToSplit, "wayToSplit");
    272272        CheckParameterUtil.ensureParameterNotNull(splitPoints, "splitPoints");
     
    392392                        /* this code assumes the restriction is correct. No real error checking done */
    393393                        String role = rm.getRole();
    394                         if("from".equals(role) || "to".equals(role)) {
     394                        if ("from".equals(role) || "to".equals(role)) {
    395395                            OsmPrimitive via = null;
    396396                            for (RelationMember rmv : r.getMembers()) {
    397                                 if ("via".equals(rmv.getRole())){
     397                                if ("via".equals(rmv.getRole())) {
    398398                                    via = rmv.getMember();
    399399                                }
     
    402402                            if (via != null) {
    403403                                if (via instanceof Node) {
    404                                     nodes.add((Node)via);
     404                                    nodes.add((Node) via);
    405405                                } else if (via instanceof Way) {
    406                                     nodes.add(((Way)via).lastNode());
    407                                     nodes.add(((Way)via).firstNode());
     406                                    nodes.add(((Way) via).lastNode());
     407                                    nodes.add(((Way) via).firstNode());
    408408                                }
    409409                            }
    410410                            Way res = null;
    411411                            for (Node n : nodes) {
    412                                 if(changedWay.isFirstLastNode(n)) {
     412                                if (changedWay.isFirstLastNode(n)) {
    413413                                    res = way;
    414414                                }
     
    416416                            if (res == null) {
    417417                                for (Way wayToAdd : newWays) {
    418                                     for(Node n : nodes) {
    419                                         if(wayToAdd.isFirstLastNode(n)) {
     418                                    for (Node n : nodes) {
     419                                        if (wayToAdd.isFirstLastNode(n)) {
    420420                                            res = wayToAdd;
    421421                                        }
     
    433433                                insert = false;
    434434                            }
    435                         } else if(!"via".equals(role)) {
     435                        } else if (!"via".equals(role)) {
    436436                            warnme = true;
    437437                        }
     
    451451                        int k = 1;
    452452                        while (i_r - k >= 0 || i_r + k < relationMembers.size()) {
    453                             if ((i_r - k >= 0) && relationMembers.get(i_r - k).isWay()){
     453                            if ((i_r - k >= 0) && relationMembers.get(i_r - k).isWay()) {
    454454                                Way w = relationMembers.get(i_r - k).getWay();
    455455                                if ((w.lastNode() == way.firstNode()) || w.firstNode() == way.firstNode()) {
     
    460460                                break;
    461461                            }
    462                             if ((i_r + k < relationMembers.size()) && relationMembers.get(i_r + k).isWay()){
     462                            if ((i_r + k < relationMembers.size()) && relationMembers.get(i_r + k).isWay()) {
    463463                                Way w = relationMembers.get(i_r + k).getWay();
    464464                                if ((w.lastNode() == way.firstNode()) || w.firstNode() == way.firstNode()) {
     
    537537        List<List<Node>> chunks = buildSplitChunks(way, atNodes);
    538538        if (chunks == null) return null;
    539         return splitWay(layer,way, chunks, selection);
     539        return splitWay(layer, way, chunks, selection);
    540540    }
    541541
  • trunk/src/org/openstreetmap/josm/actions/UnGlueAction.java

    r8509 r8510  
    149149        }
    150150
    151         if(errMsg != null) {
     151        if (errMsg != null) {
    152152            new Notification(
    153153                    errMsg)
     
    177177
    178178        // If this wasn't called from menu, place it where the cursor is/was
    179         if(e.getSource() instanceof JPanel) {
     179        if (e.getSource() instanceof JPanel) {
    180180            MapView mv = Main.map.mapView;
    181181            n.setCoor(mv.getLatLon(mv.lastMEvent.getX(), mv.lastMEvent.getY()));
  • trunk/src/org/openstreetmap/josm/actions/UnJoinNodeWayAction.java

    r8465 r8510  
    155155        // List of ways shared by all nodes
    156156        List<Way> result = new ArrayList<>(OsmPrimitive.getFilteredList(selectedNodes.get(0).getReferrers(), Way.class));
    157         for (int i=1; i<selectedNodes.size(); i++) {
     157        for (int i = 1; i < selectedNodes.size(); i++) {
    158158            List<OsmPrimitive> ref = selectedNodes.get(i).getReferrers();
    159159            for (Iterator<Way> it = result.iterator(); it.hasNext();) {
  • trunk/src/org/openstreetmap/josm/actions/UpdateDataAction.java

    r8449 r8510  
    5151
    5252        List<Area> areas = new ArrayList<>();
    53         for(DataSource ds : getEditLayer().data.dataSources) {
     53        for (DataSource ds : getEditLayer().data.dataSources) {
    5454            areas.add(new Area(ds.bounds.asRect()));
    5555        }
     
    5959        // and also skips smaller bounding boxes that are contained within larger ones
    6060        // entirely.
    61         for(int i = 0; i < areas.size(); i++) {
    62             for(int j = i+1; j < areas.size(); j++) {
     61        for (int i = 0; i < areas.size(); i++) {
     62            for (int j = i+1; j < areas.size(); j++) {
    6363                areas.get(i).subtract(areas.get(j));
    6464            }
    6565        }
    6666
    67         for(int i = areas.size()-1; i > 0; i--) {
    68             for(int j = i-1; j > 0; j--) {
     67        for (int i = areas.size()-1; i > 0; i--) {
     68            for (int j = i-1; j > 0; j--) {
    6969                areas.get(i).subtract(areas.get(j));
    7070            }
     
    7272
    7373        List<Area> areasToDownload = new ArrayList<>();
    74         for(Area a : areas) {
    75             if(a.isEmpty()) {
     74        for (Area a : areas) {
     75            if (a.isEmpty()) {
    7676                continue;
    7777            }
     
    7979        }
    8080
    81         if(areasToDownload.isEmpty()) {
     81        if (areasToDownload.isEmpty()) {
    8282            // no bounds defined in the dataset? we update all primitives in the data set
    8383            // using a series of multi fetch requests
     
    9595                            try {
    9696                                future.get();
    97                             } catch(Exception e) {
     97                            } catch (Exception e) {
    9898                                Main.error(e);
    9999                                return;
  • trunk/src/org/openstreetmap/josm/actions/UpdateSelectionAction.java

    r8509 r8510  
    4444            DataSet ds = reader.parseOsm(NullProgressMonitor.INSTANCE);
    4545            Main.main.getEditLayer().mergeFrom(ds);
    46         } catch(Exception e) {
     46        } catch (Exception e) {
    4747            ExceptionDialogUtil.explainException(e);
    4848        }
     
    5757     */
    5858    public static void updatePrimitives(final Collection<OsmPrimitive> selection) {
    59         UpdatePrimitivesTask task = new UpdatePrimitivesTask(Main.main.getEditLayer(),selection);
     59        UpdatePrimitivesTask task = new UpdatePrimitivesTask(Main.main.getEditLayer(), selection);
    6060        Main.worker.submit(task);
    6161    }
  • trunk/src/org/openstreetmap/josm/actions/UploadAction.java

    r8338 r8510  
    5757    private static final List<UploadHook> uploadHooks = new LinkedList<>();
    5858    private static final List<UploadHook> lateUploadHooks = new LinkedList<>();
     59
    5960    static {
    6061        /**
     
    102103     */
    103104    public static void registerUploadHook(UploadHook hook, boolean late) {
    104         if(hook == null) return;
     105        if (hook == null) return;
    105106        if (late) {
    106107            if (!lateUploadHooks.contains(hook)) {
     
    120121     */
    121122    public static void unregisterUploadHook(UploadHook hook) {
    122         if(hook == null) return;
     123        if (hook == null) return;
    123124        if (uploadHooks.contains(hook)) {
    124125            uploadHooks.remove(hook);
     
    146147    public static boolean checkPreUploadConditions(AbstractModifiableLayer layer) {
    147148        return checkPreUploadConditions(layer,
    148                 layer instanceof OsmDataLayer ? new APIDataSet(((OsmDataLayer)layer).data) : null);
     149                layer instanceof OsmDataLayer ? new APIDataSet(((OsmDataLayer) layer).data) : null);
    149150    }
    150151
  • trunk/src/org/openstreetmap/josm/actions/UploadNotesAction.java

    r8474 r8510  
    2424    /** Create a new action to upload notes */
    2525    public UploadNotesAction() {
    26         putValue(SHORT_DESCRIPTION,tr("Upload note changes to server"));
     26        putValue(SHORT_DESCRIPTION, tr("Upload note changes to server"));
    2727        putValue(NAME, tr("Upload notes"));
    2828        putValue(SMALL_ICON, ImageProvider.get("upload"));
     
    4747        NoteData noteData = layer.getNoteData();
    4848
    49         if(noteData == null || !noteData.isModified()) {
     49        if (noteData == null || !noteData.isModified()) {
    5050            if (Main.isDebugEnabled()) {
    5151                Main.debug("No changed notes to upload");
  • trunk/src/org/openstreetmap/josm/actions/UploadSelectionAction.java

    r8338 r8510  
    190190        private Set<OsmPrimitive> hull;
    191191
    192         public UploadHullBuilder(){
     192        public UploadHullBuilder() {
    193193            hull = new HashSet<>();
    194194        }
     
    321321                toCheck.addAll(getPrimitivesToCheckForParents());
    322322                Set<OsmPrimitive> checked = new HashSet<>();
    323                 while(!toCheck.isEmpty()) {
     323                while (!toCheck.isEmpty()) {
    324324                    if (canceled) return;
    325325                    OsmPrimitive current = toCheck.pop();
    326                     synchronized(this) {
     326                    synchronized (this) {
    327327                        reader = new OsmServerBackreferenceReader(current);
    328328                    }
    329329                    getProgressMonitor().subTask(tr("Reading parents of ''{0}''", current.getDisplayName(DefaultNameFormatter.getInstance())));
    330330                    DataSet ds = reader.parseOsm(getProgressMonitor().createSubTaskMonitor(1, false));
    331                     synchronized(this) {
     331                    synchronized (this) {
    332332                        reader = null;
    333333                    }
     
    350350                    }
    351351                }
    352             } catch(Exception e) {
     352            } catch (Exception e) {
    353353                if (canceled)
    354354                    // ignore exception
  • trunk/src/org/openstreetmap/josm/actions/ValidateAction.java

    r8509 r8510  
    165165                    return;
    166166                testCounter++;
    167                 getProgressMonitor().setCustomText(tr("Test {0}/{1}: Starting {2}", testCounter, tests.size(),test.getName()));
     167                getProgressMonitor().setCustomText(tr("Test {0}/{1}: Starting {2}", testCounter, tests.size(), test.getName()));
    168168                test.setPartialSelection(formerValidatedPrimitives != null);
    169169                test.startTest(getProgressMonitor().createSubTaskMonitor(validatedPrimitives.size(), false));
  • trunk/src/org/openstreetmap/josm/actions/WireframeToggleAction.java

    r6336 r8510  
    2626              null, /* no icon */
    2727              tr("Enable/disable rendering the map as wireframe only"),
    28               Shortcut.registerShortcut("menu:view:wireframe", tr("Toggle Wireframe view"),KeyEvent.VK_W, Shortcut.CTRL),
     28              Shortcut.registerShortcut("menu:view:wireframe", tr("Toggle Wireframe view"), KeyEvent.VK_W, Shortcut.CTRL),
    2929              false /* register toolbar */
    3030        );
  • trunk/src/org/openstreetmap/josm/actions/ZoomInAction.java

    r6710 r8510  
    2626                tr("Zoom In"),
    2727                // Although it might be possible on few custom keyboards, the vast majority of layouts do not have a direct '+' key, see below
    28                 Shortcut.registerShortcut("view:zoomin", tr("View: {0}", tr("Zoom In")),KeyEvent.VK_PLUS, Shortcut.DIRECT),
     28                Shortcut.registerShortcut("view:zoomin", tr("View: {0}", tr("Zoom In")), KeyEvent.VK_PLUS, Shortcut.DIRECT),
    2929                true
    3030        );
  • trunk/src/org/openstreetmap/josm/actions/audio/AudioPlayPauseAction.java

    r8384 r8510  
    3838            if (AudioPlayer.paused() && url != null) {
    3939                AudioPlayer.play(url);
    40             } else if (AudioPlayer.playing()){
     40            } else if (AudioPlayer.playing()) {
    4141                if (!Utils.equalsEpsilon(AudioPlayer.speed(), 1.0))
    4242                    AudioPlayer.play(url, AudioPlayer.position());
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/AbstractDownloadTask.java

    r8379 r8510  
    5555            .append(":</td><td>");
    5656        String[] patterns = getPatterns();
    57         if (patterns.length>0) {
     57        if (patterns.length > 0) {
    5858            buff.append("<ul>");
    5959            for (String pattern: patterns) {
     
    7070    // Can be overridden for more complex checking logic
    7171    public boolean acceptsUrl(String url) {
    72         if (url==null) return false;
     72        if (url == null) return false;
    7373        for (String p: getPatterns()) {
    7474            if (url.matches(p)) {
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadGpsTask.java

    r8444 r8510  
    8282        } else if (url.matches(PATTERN_TRACKPOINTS_BBOX)) {
    8383            String[] table = url.split("\\?|=|&");
    84             for (int i = 0; i<table.length; i++) {
    85                 if ("bbox".equals(table[i]) && i<table.length-1)
     84            for (int i = 0; i < table.length; i++) {
     85                if ("bbox".equals(table[i]) && i < table.length-1)
    8686                    return download(newLayer, new Bounds(table[i+1], ",", ParseMethod.LEFT_BOTTOM_RIGHT_TOP), progressMonitor);
    8787            }
     
    115115                ProgressMonitor subMonitor = progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false);
    116116                rawData = reader.parseRawGps(subMonitor);
    117             } catch(Exception e) {
     117            } catch (Exception e) {
    118118                if (isCanceled())
    119119                    return;
     
    160160            boolean merge = Main.pref.getBoolean("download.gps.mergeWithLocal", false);
    161161            Layer active = Main.map.mapView.getActiveLayer();
    162             if (active instanceof GpxLayer && (merge || ((GpxLayer)active).data.fromServer))
     162            if (active instanceof GpxLayer && (merge || ((GpxLayer) active).data.fromServer))
    163163                return (GpxLayer) active;
    164164            for (GpxLayer l : Main.map.mapView.getLayersOfType(GpxLayer.class)) {
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadNotesUrlBoundsTask.java

    r8209 r8510  
    22package org.openstreetmap.josm.actions.downloadtasks;
    33
    4 import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    5 import org.openstreetmap.josm.tools.OsmUrlToBounds;
     4import static org.openstreetmap.josm.tools.I18n.tr;
    65
    76import java.util.concurrent.Future;
    87
    9 import static org.openstreetmap.josm.tools.I18n.tr;
     8import org.openstreetmap.josm.gui.progress.ProgressMonitor;
     9import org.openstreetmap.josm.tools.OsmUrlToBounds;
    1010
    1111public class DownloadNotesUrlBoundsTask extends DownloadNotesTask {
     
    2121                "https?://www\\.(osm|openstreetmap)\\.org/(.*)?#map=\\p{Digit}+/.*/.*&layers=[A-MO-Z]*N[A-MO-Z]*"};
    2222    }
     23
    2324    @Override
    2425    public String getTitle() {
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmChangeTask.java

    r8419 r8510  
    154154                        case NODE:
    155155                            data = new NodeData();
    156                             ((NodeData)data).setCoor(((HistoryNode)hp).getCoords());
     156                            ((NodeData) data).setCoor(((HistoryNode) hp).getCoords());
    157157                            break;
    158158                        case WAY:
    159159                            data = new WayData();
    160                             List<Long> nodeIds = ((HistoryWay)hp).getNodes();
    161                             ((WayData)data).setNodes(nodeIds);
     160                            List<Long> nodeIds = ((HistoryWay) hp).getNodes();
     161                            ((WayData) data).setNodes(nodeIds);
    162162                            // Find incomplete nodes to load at next run
    163163                            for (Long nodeId : nodeIds) {
     
    171171                        case RELATION:
    172172                            data = new RelationData();
    173                             List<RelationMemberData> members = ((HistoryRelation)hp).getMembers();
    174                             ((RelationData)data).setMembers(members);
     173                            List<RelationMemberData> members = ((HistoryRelation) hp).getMembers();
     174                            ((RelationData) data).setMembers(members);
    175175                            break;
    176176                        default: throw new AssertionError("Unknown primitive type");
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java

    r8130 r8510  
    313313                    return;
    314314                dataSet = parseDataSet();
    315             } catch(Exception e) {
     315            } catch (Exception e) {
    316316                if (isCanceled()) {
    317317                    Main.info(tr("Ignoring exception because download has been canceled. Exception was: {0}", e.toString()));
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadReferrersTask.java

    r8444 r8510  
    157157    protected void cancel() {
    158158        canceled = true;
    159         synchronized(this) {
     159        synchronized (this) {
    160160            if (reader != null) {
    161161                reader.cancel();
     
    180180                    public void run() {
    181181                        targetLayer.onPostDownloadFromServer();
    182                         if(Main.map != null)
     182                        if (Main.map != null)
    183183                            Main.map.mapView.repaint();
    184184                    }
     
    202202    }
    203203
    204     protected void downloadParents(long id, OsmPrimitiveType type, ProgressMonitor progressMonitor) throws OsmTransferException{
     204    protected void downloadParents(long id, OsmPrimitiveType type, ProgressMonitor progressMonitor) throws OsmTransferException {
    205205        reader = new OsmServerBackreferenceReader(id, type);
    206206        DataSet ds = reader.parseOsm(progressMonitor.createSubTaskMonitor(1, false));
    207         synchronized(this) { // avoid race condition in cancel()
     207        synchronized (this) { // avoid race condition in cancel()
    208208            reader = null;
    209209        }
     
    221221            if (!nodes.isEmpty()) {
    222222                reader = new MultiFetchServerObjectReader();
    223                 ((MultiFetchServerObjectReader)reader).append(nodes);
     223                ((MultiFetchServerObjectReader) reader).append(nodes);
    224224                DataSet wayNodes = reader.parseOsm(progressMonitor.createSubTaskMonitor(1, false));
    225                 synchronized(this) { // avoid race condition in cancel()
     225                synchronized (this) { // avoid race condition in cancel()
    226226                    reader = null;
    227227                }
     
    238238        try {
    239239            progressMonitor.setTicksCount(children.size());
    240             int i=1;
     240            int i = 1;
    241241            for (Entry<Long, OsmPrimitiveType> entry: children.entrySet()) {
    242242                if (canceled)
     
    244244                String msg = "";
    245245                switch(entry.getValue()) {
    246                 case NODE: msg = tr("({0}/{1}) Loading parents of node {2}", i+1,children.size(), entry.getKey()); break;
    247                 case WAY: msg = tr("({0}/{1}) Loading parents of way {2}", i+1,children.size(), entry.getKey()); break;
    248                 case RELATION: msg = tr("({0}/{1}) Loading parents of relation {2}", i+1,children.size(), entry.getKey()); break;
     246                case NODE: msg = tr("({0}/{1}) Loading parents of node {2}", i+1, children.size(), entry.getKey()); break;
     247                case WAY: msg = tr("({0}/{1}) Loading parents of way {2}", i+1, children.size(), entry.getKey()); break;
     248                case RELATION: msg = tr("({0}/{1}) Loading parents of relation {2}", i+1, children.size(), entry.getKey()); break;
    249249                }
    250250                progressMonitor.subTask(msg);
     
    252252                i++;
    253253            }
    254         } catch(Exception e) {
     254        } catch (Exception e) {
    255255            if (canceled)
    256256                return;
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/PostDownloadHandler.java

    r8472 r8510  
    7070            try {
    7171                future.get();
    72             } catch(Exception e) {
     72            } catch (Exception e) {
    7373                Main.error(e);
    7474                return;
     
    9191                public void run() {
    9292                    if (error instanceof Exception) {
    93                         ExceptionDialogUtil.explainException((Exception)error);
     93                        ExceptionDialogUtil.explainException((Exception) error);
    9494                    } else if (tr("No data found in this area.").equals(error)) {
    9595                        new Notification(error.toString()).setIcon(JOptionPane.WARNING_MESSAGE).show();
     
    114114                    items.add((String) error);
    115115                } else if (error instanceof Exception) {
    116                     items.add(ExceptionUtil.explainException((Exception)error));
     116                    items.add(ExceptionUtil.explainException((Exception) error));
    117117                }
    118118            }
  • trunk/src/org/openstreetmap/josm/actions/mapmode/DeleteAction.java

    r8470 r8510  
    9393                "delete",
    9494                tr("Delete nodes or ways."),
    95                 Shortcut.registerShortcut("mapmode:delete", tr("Mode: {0}",tr("Delete")),
     95                Shortcut.registerShortcut("mapmode:delete", tr("Mode: {0}", tr("Delete")),
    9696                KeyEvent.VK_DELETE, Shortcut.CTRL),
    9797                mapFrame,
     
    135135            return;
    136136        boolean ctrl = (e.getModifiers() & ActionEvent.CTRL_MASK) != 0;
    137         boolean alt = (e.getModifiers() & (ActionEvent.ALT_MASK|InputEvent.ALT_GRAPH_MASK)) != 0;
     137        boolean alt = (e.getModifiers() & (ActionEvent.ALT_MASK | InputEvent.ALT_GRAPH_MASK)) != 0;
    138138
    139139        Command c;
    140140        if (ctrl) {
    141             c = DeleteCommand.deleteWithReferences(getEditLayer(),getCurrentDataSet().getSelected());
     141            c = DeleteCommand.deleteWithReferences(getEditLayer(), getCurrentDataSet().getSelected());
    142142        } else {
    143             c = DeleteCommand.delete(getEditLayer(),getCurrentDataSet().getSelected(), !alt /* also delete nodes in way */);
     143            c = DeleteCommand.delete(getEditLayer(), getCurrentDataSet().getSelected(), !alt /* also delete nodes in way */);
    144144        }
    145145        // if c is null, an error occurred or the user aborted. Don't do anything in that case.
     
    172172        highlightHelper.clear();
    173173        DataSet ds = getCurrentDataSet();
    174         if(ds != null) {
     174        if (ds != null) {
    175175            ds.clearHighlightedWaySegments();
    176176        }
     
    184184     */
    185185    private void addHighlighting(MouseEvent e, int modifiers) {
    186         if(!drawTargetHighlight)
     186        if (!drawTargetHighlight)
    187187            return;
    188188
     
    190190        DeleteParameters parameters = getDeleteParameters(e, modifiers);
    191191
    192         if(parameters.mode == DeleteMode.segment) {
     192        if (parameters.mode == DeleteMode.segment) {
    193193            // deleting segments is the only action not working on OsmPrimitives
    194194            // so we have to handle them separately.
     
    198198            // silent operation and SplitWayAction will show dialogs. A lot.
    199199            Command delCmd = buildDeleteCommands(e, modifiers, true);
    200             if(delCmd != null) {
     200            if (delCmd != null) {
    201201                // all other cases delete OsmPrimitives directly, so we can safely do the following
    202                 for(OsmPrimitive osm : delCmd.getParticipatingPrimitives()) {
     202                for (OsmPrimitive osm : delCmd.getParticipatingPrimitives()) {
    203203                    newHighlights.add(osm);
    204204                }
     
    212212        DataSet ds = getCurrentDataSet();
    213213
    214         if(newHighlightedWaySegment == null && oldHighlightedWaySegment != null) {
    215             if(ds != null) {
     214        if (newHighlightedWaySegment == null && oldHighlightedWaySegment != null) {
     215            if (ds != null) {
    216216                ds.clearHighlightedWaySegments();
    217217                needsRepaint = true;
    218218            }
    219219            oldHighlightedWaySegment = null;
    220         } else if(newHighlightedWaySegment != null && !newHighlightedWaySegment.equals(oldHighlightedWaySegment)) {
    221             if(ds != null) {
     220        } else if (newHighlightedWaySegment != null && !newHighlightedWaySegment.equals(oldHighlightedWaySegment)) {
     221            if (ds != null) {
    222222                ds.setHighlightedWaySegments(Collections.singleton(newHighlightedWaySegment));
    223223                needsRepaint = true;
     
    226226        }
    227227        needsRepaint |= highlightHelper.highlightOnly(newHighlights);
    228         if(needsRepaint) {
     228        if (needsRepaint) {
    229229            Main.map.mapView.repaint();
    230230        }
     
    240240        if (!Main.isDisplayingMapView())
    241241            return;
    242         if(!Main.map.mapView.isActiveLayerVisible() || e == null)
     242        if (!Main.map.mapView.isActiveLayerVisible() || e == null)
    243243            return;
    244244
     
    278278        if (e.getButton() != MouseEvent.BUTTON1)
    279279            return;
    280         if(!Main.map.mapView.isActiveLayerVisible())
     280        if (!Main.map.mapView.isActiveLayerVisible())
    281281            return;
    282282
     
    346346                    result.mode = DeleteMode.way_with_references;
    347347                } else {
    348                     result.mode = alt?DeleteMode.way:DeleteMode.way_with_nodes;
     348                    result.mode = alt ? DeleteMode.way : DeleteMode.way_with_nodes;
    349349                }
    350350            } else {
     
    372372        switch (parameters.mode) {
    373373        case node:
    374             return DeleteCommand.delete(getEditLayer(),Collections.singleton(parameters.nearestNode), false, silent);
     374            return DeleteCommand.delete(getEditLayer(), Collections.singleton(parameters.nearestNode), false, silent);
    375375        case node_with_references:
    376             return DeleteCommand.deleteWithReferences(getEditLayer(),Collections.singleton(parameters.nearestNode), silent);
     376            return DeleteCommand.deleteWithReferences(getEditLayer(), Collections.singleton(parameters.nearestNode), silent);
    377377        case segment:
    378378            return DeleteCommand.deleteWaySegment(getEditLayer(), parameters.nearestSegment);
  • trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

    r8509 r8510  
    7474public class DrawAction extends MapMode implements MapViewPaintable, SelectionChangedListener, KeyPressReleaseListener, ModifierListener {
    7575
    76     private static final Color ORANGE_TRANSPARENT = new Color(Color.ORANGE.getRed(),Color.ORANGE.getGreen(),Color.ORANGE.getBlue(),128);
     76    private static final Color ORANGE_TRANSPARENT = new Color(Color.ORANGE.getRed(), Color.ORANGE.getGreen(), Color.ORANGE.getBlue(), 128);
    7777    private static final double PHI = Math.toRadians(90);
    7878
     
    144144
    145145    private JCheckBoxMenuItem addMenuItem() {
    146         int n=Main.main.menu.editMenu.getItemCount();
    147         for (int i=n-1;i>0;i--) {
     146        int n = Main.main.menu.editMenu.getItemCount();
     147        for (int i = n-1; i > 0; i--) {
    148148            JMenuItem item = Main.main.menu.editMenu.getItem(i);
    149             if (item!=null && item.getAction() !=null && item.getAction() instanceof SnapChangeAction) {
     149            if (item != null && item.getAction() != null && item.getAction() instanceof SnapChangeAction) {
    150150                Main.main.menu.editMenu.remove(i);
    151151            }
     
    161161        // repaint required if the helper line is active.
    162162        boolean needsRepaint = drawHelperLine && !wayIsFinished;
    163         if(drawTargetHighlight) {
     163        if (drawTargetHighlight) {
    164164            // move newHighlights to oldHighlights; only update changed primitives
    165             for(OsmPrimitive x : newHighlights) {
    166                 if(oldHighlights.contains(x)) {
     165            for (OsmPrimitive x : newHighlights) {
     166                if (oldHighlights.contains(x)) {
    167167                    continue;
    168168                }
     
    171171            }
    172172            oldHighlights.removeAll(newHighlights);
    173             for(OsmPrimitive x : oldHighlights) {
     173            for (OsmPrimitive x : oldHighlights) {
    174174                x.setHighlighted(false);
    175175                needsRepaint = true;
     
    195195        }
    196196
    197         if(needsRepaint) {
     197        if (needsRepaint) {
    198198            Main.map.mapView.repaint();
    199199        }
     
    244244        if (rubberLineColor == null) rubberLineColor = PaintColors.SELECTED.get();
    245245
    246         rubberLineStroke = GuiHelper.getCustomizedStroke(Main.pref.get("draw.stroke.helper-line","3"));
     246        rubberLineStroke = GuiHelper.getCustomizedStroke(Main.pref.get("draw.stroke.helper-line", "3"));
    247247        drawHelperLine = Main.pref.getBoolean("draw.helper-line", true);
    248248        drawTargetHighlight = Main.pref.getBoolean("draw.target-highlight", true);
    249         snapToIntersectionThreshold = Main.pref.getInteger("edit.snap-intersection-threshold",10);
     249        snapToIntersectionThreshold = Main.pref.getInteger("edit.snap-intersection-threshold", 10);
    250250    }
    251251
     
    272272        //
    273273        DataSet ds = getCurrentDataSet();
    274         if(ds != null) {
     274        if (ds != null) {
    275275            ds.fireSelectionChanged();
    276276        }
     
    316316    @Override
    317317    public void selectionChanged(Collection<? extends OsmPrimitive> newSelection) {
    318         if(!Main.map.mapView.isActiveLayerDrawable())
     318        if (!Main.map.mapView.isActiveLayerDrawable())
    319319            return;
    320320        computeHelperLine();
     
    372372        if (e.getButton() != MouseEvent.BUTTON1)
    373373            return;
    374         if(!Main.map.mapView.isActiveLayerDrawable())
     374        if (!Main.map.mapView.isActiveLayerDrawable())
    375375            return;
    376376        // request focus in order to enable the expected keyboard shortcuts
     
    378378        Main.map.mapView.requestFocus();
    379379
    380         if(e.getClickCount() > 1 && mousePos != null && mousePos.equals(oldMousePos)) {
     380        if (e.getClickCount() > 1 && mousePos != null && mousePos.equals(oldMousePos)) {
    381381            // A double click equals "user clicked last node again, finish way"
    382382            // Change draw tool only if mouse position is nearly the same, as
     
    410410                if (!it.next().containsNode(n)) n = null;
    411411            } else {
    412                 n=null; // ctrl-click + no selected way = new node
     412                n = null; // ctrl-click + no selected way = new node
    413413            }
    414414        }
     
    434434        } else {
    435435            EastNorth newEN;
    436             if (n!=null) {
     436            if (n != null) {
    437437                EastNorth foundPoint = n.getEastNorth();
    438438                // project found node to snapping line
     
    470470                        Main.map.mapView.getPoint(n), OsmPrimitive.isSelectablePredicate);
    471471                if (snapHelper.isActive()) {
    472                     tryToMoveNodeOnIntersection(wss,n);
     472                    tryToMoveNodeOnIntersection(wss, n);
    473473                }
    474474                insertNodeIntoAllNearbySegments(wss, n, newSelection, cmds, replacedWays, reuseWays);
     
    477477        // now "n" is newly created or reused node that shoud be added to some way
    478478
    479         // This part decides whether or not a "segment" (i.e. a connection) is made to an
    480         // existing node.
     479        // This part decides whether or not a "segment" (i.e. a connection) is made to an existing node.
    481480
    482481        // For a connection to be made, the user must either have a node selected (connection
     
    517516            Node n0 = findNodeToContinueFrom(selectedNode, selectedWay);
    518517            // We have a selection but it isn't suitable. Try again.
    519             if(n0 == null) {
     518            if (n0 == null) {
    520519                tryAgain(e);
    521520                return;
    522521            }
    523             if(!wayIsFinishedTemp){
    524                 if(isSelfContainedWay(selectedWay, n0, n))
     522            if (!wayIsFinishedTemp) {
     523                if (isSelfContainedWay(selectedWay, n0, n))
    525524                    return;
    526525
    527526                // User clicked last node again, finish way
    528                 if(n0 == n) {
     527                if (n0 == n) {
    529528                    finishDrawing();
    530529                    return;
     
    538537
    539538                // Don't allow creation of self-overlapping ways
    540                 if(way != null) {
    541                     int nodeCount=0;
     539                if (way != null) {
     540                    int nodeCount = 0;
    542541                    for (Node p : way.getNodes())
    543                         if(p.equals(n0)) {
     542                        if (p.equals(n0)) {
    544543                            nodeCount++;
    545544                        }
    546                     if(nodeCount > 1) {
     545                    if (nodeCount > 1) {
    547546                        way = null;
    548547                    }
     
    568567
    569568                // Connected to a node that's already in the way
    570                 if(way.containsNode(n)) {
     569                if (way.containsNode(n)) {
    571570                    wayIsFinished = true;
    572571                    selection.clear();
     
    611610
    612611        Main.main.undoRedo.add(c);
    613         if(!wayIsFinished) {
     612        if (!wayIsFinished) {
    614613            lastUsedNode = n;
    615614        }
     
    641640        }
    642641
    643         Set<Pair<Node,Node>> segSet = new HashSet<>();
     642        Set<Pair<Node, Node>> segSet = new HashSet<>();
    644643
    645644        for (Map.Entry<Way, List<Integer>> insertPoint : insertPoints.entrySet()) {
     
    662661            // but pressing ALT prevents this. Therefore we must de-select the way manually
    663662            // here so /only/ the new way will be selected after this method finishes.
    664             if(alt) {
     663            if (alt) {
    665664                newSelection.add(insertPoint.getKey());
    666665            }
     
    690689     */
    691690    private boolean isSelfContainedWay(Way selectedWay, Node currentNode, Node targetNode) {
    692         if(selectedWay != null) {
     691        if (selectedWay != null) {
    693692            int posn0 = selectedWay.getNodes().indexOf(currentNode);
    694693            if (posn0 != -1 && // n0 is part of way
     
    713712        // No nodes or ways have been selected, this occurs when a relation
    714713        // has been selected or the selection is empty
    715         if(selectedNode == null && selectedWay == null)
     714        if (selectedNode == null && selectedWay == null)
    716715            return null;
    717716
     
    741740    @Override
    742741    public void mouseMoved(MouseEvent e) {
    743         if(!Main.map.mapView.isActiveLayerDrawable())
     742        if (!Main.map.mapView.isActiveLayerDrawable())
    744743            return;
    745744
     
    761760    private void tryToSetBaseSegmentForAngleSnap() {
    762761        WaySegment seg = Main.map.mapView.getNearestWaySegment(mousePos, OsmPrimitive.isSelectablePredicate);
    763         if (seg!=null) {
     762        if (seg != null) {
    764763            snapHelper.setBaseSegment(seg);
    765764        }
     
    794793        // We need this for highlighting and we'll only do so if we actually want to re-use
    795794        // *and* there is no node nearby (because nodes beat ways when re-using)
    796         if(!ctrl && currentMouseNode == null) {
     795        if (!ctrl && currentMouseNode == null) {
    797796            List<WaySegment> wss = mv.getNearestWaySegments(mousePos, OsmPrimitive.isSelectablePredicate);
    798             for(WaySegment ws : wss) {
     797            for (WaySegment ws : wss) {
    799798                mouseOnExistingWays.add(ws.way);
    800799            }
     
    822821        double curHdg = Math.toDegrees(getCurrentBaseNode().getEastNorth()
    823822                .heading(currentMouseEastNorth));
    824         double baseHdg=-1;
     823        double baseHdg = -1;
    825824        if (previousNode != null) {
    826825            baseHdg =  Math.toDegrees(previousNode.getEastNorth()
     
    828827        }
    829828
    830         snapHelper.checkAngleSnapping(currentMouseEastNorth,baseHdg, curHdg);
     829        snapHelper.checkAngleSnapping(currentMouseEastNorth, baseHdg, curHdg);
    831830
    832831        // status bar was filled by snapHelper
     
    871870            for (OsmPrimitive p: selectedNode.getReferrers()) {
    872871                if (p.isUsable() && p instanceof Way && ((Way) p).isFirstLastNode(selectedNode)) {
    873                     if (selectedWay!=null) { // two uncontinued ways, nothing to take as reference
    874                         selectedWay=null;
     872                    if (selectedWay != null) { // two uncontinued ways, nothing to take as reference
     873                        selectedWay = null;
    875874                        break;
    876875                    } else {
     
    899898    private void continueWayFromNode(Way way, Node node) {
    900899        int n = way.getNodesCount();
    901         if (node == way.firstNode()){
     900        if (node == way.firstNode()) {
    902901            currentBaseNode = node;
    903             if (n>1) previousNode = way.getNode(1);
     902            if (n > 1) previousNode = way.getNode(1);
    904903        } else if (node == way.lastNode()) {
    905904            currentBaseNode = node;
    906             if (n>1) previousNode = way.getNode(n-2);
     905            if (n > 1) previousNode = way.getNode(n-2);
    907906        }
    908907    }
     
    913912    @Override
    914913    public void mouseExited(MouseEvent e) {
    915         if(!Main.map.mapView.isActiveLayerDrawable())
     914        if (!Main.map.mapView.isActiveLayerDrawable())
    916915            return;
    917916        mousePos = e.getPoint();
     
    920919        // force repaint in case snapHelper needs one. If removeHighlighting
    921920        // caused one already, don’t do it again.
    922         if(!repaintIssued) {
     921        if (!repaintIssued) {
    923922            Main.map.mapView.repaint();
    924923        }
     
    985984     * @param n the node to adjust
    986985     */
    987     private static void adjustNode(Collection<Pair<Node,Node>> segs, Node n) {
     986    private static void adjustNode(Collection<Pair<Node, Node>> segs, Node n) {
    988987
    989988        switch (segs.size()) {
     
    991990            return;
    992991        case 2:
    993             // This computes the intersection between
    994             // the two segments and adjusts the node position.
    995             Iterator<Pair<Node,Node>> i = segs.iterator();
    996             Pair<Node,Node> seg = i.next();
     992            // This computes the intersection between the two segments and adjusts the node position.
     993            Iterator<Pair<Node, Node>> i = segs.iterator();
     994            Pair<Node, Node> seg = i.next();
    997995            EastNorth A = seg.a.getEastNorth();
    998996            EastNorth B = seg.b.getEastNorth();
     
    1001999            EastNorth D = seg.b.getEastNorth();
    10021000
    1003             double u=det(B.east() - A.east(), B.north() - A.north(), C.east() - D.east(), C.north() - D.north());
     1001            double u = det(B.east() - A.east(), B.north() - A.north(), C.east() - D.east(), C.north() - D.north());
    10041002
    10051003            // Check for parallel segments and do nothing if they are
     
    10481046            return;
    10491047        WaySegment ws = wss.get(0);
    1050         EastNorth p1=ws.getFirstNode().getEastNorth();
    1051         EastNorth p2=ws.getSecondNode().getEastNorth();
     1048        EastNorth p1 = ws.getFirstNode().getEastNorth();
     1049        EastNorth p2 = ws.getSecondNode().getEastNorth();
    10521050        if (snapHelper.dir2 != null && getCurrentBaseNode() != null) {
    10531051            EastNorth xPoint = Geometry.getSegmentSegmentIntersection(p1, p2, snapHelper.dir2,
    10541052                    getCurrentBaseNode().getEastNorth());
    1055             if (xPoint!=null) {
     1053            if (xPoint != null) {
    10561054                n.setEastNorth(xPoint);
    10571055            }
     
    11241122
    11251123        Graphics2D g2 = g;
    1126         snapHelper.drawIfNeeded(g2,mv);
     1124        snapHelper.drawIfNeeded(g2, mv);
    11271125        if (!drawHelperLine || wayIsFinished || shift)
    11281126            return;
     
    11341132            return;
    11351133        GeneralPath b = new GeneralPath();
    1136         Point p1=mv.getPoint(getCurrentBaseNode());
    1137         Point p2=mv.getPoint(currentMouseEastNorth);
     1134        Point p1 = mv.getPoint(getCurrentBaseNode());
     1135        Point p2 = mv.getPoint(currentMouseEastNorth);
    11381136
    11391137        double t = Math.atan2(p2.y-p1.y, p2.x-p1.x) + Math.PI;
    11401138
    1141         b.moveTo(p1.x,p1.y); b.lineTo(p2.x, p2.y);
     1139        b.moveTo(p1.x, p1.y);
     1140        b.lineTo(p2.x, p2.y);
    11421141
    11431142        // if alt key is held ("start new way"), draw a little perpendicular line
    11441143        if (alt) {
    1145             b.moveTo((int)(p1.x + 8*Math.cos(t+PHI)), (int)(p1.y + 8*Math.sin(t+PHI)));
    1146             b.lineTo((int)(p1.x + 8*Math.cos(t-PHI)), (int)(p1.y + 8*Math.sin(t-PHI)));
     1144            b.moveTo((int) (p1.x + 8*Math.cos(t+PHI)), (int) (p1.y + 8*Math.sin(t+PHI)));
     1145            b.lineTo((int) (p1.x + 8*Math.cos(t-PHI)), (int) (p1.y + 8*Math.sin(t-PHI)));
    11471146        }
    11481147
     
    12701269        public void actionPerformed(ActionEvent e) {
    12711270            Main.main.undoRedo.undo();
    1272             Node n=null;
    1273             Command lastCmd=Main.main.undoRedo.commands.peekLast();
    1274             if (lastCmd==null) return;
     1271            Node n = null;
     1272            Command lastCmd = Main.main.undoRedo.commands.peekLast();
     1273            if (lastCmd == null) return;
    12751274            for (OsmPrimitive p: lastCmd.getParticipatingPrimitives()) {
    12761275                if (p instanceof Node) {
    1277                     if (n==null) {
    1278                         n=(Node) p; // found one node
    1279                         wayIsFinished=false;
     1276                    if (n == null) {
     1277                        n = (Node) p; // found one node
     1278                        wayIsFinished = false;
    12801279                    }  else {
    12811280                        // if more than 1 node were affected by previous command,
    12821281                        // we have no way to continue, so we forget about found node
    1283                         n=null;
     1282                        n = null;
    12841283                        break;
    12851284                    }
     
    12871286            }
    12881287            // select last added node - maybe we will continue drawing from it
    1289             if (n!=null) {
     1288            if (n != null) {
    12901289                getCurrentDataSet().addSelected(n);
    12911290            }
     
    13001299                @Override
    13011300                public void actionPerformed(ActionEvent e) {
    1302                     boolean sel=((JCheckBoxMenuItem) e.getSource()).getState();
     1301                    boolean sel = ((JCheckBoxMenuItem) e.getSource()).getState();
    13031302                    Main.pref.put("draw.anglesnap.toggleOnRepeatedA", sel);
    13041303                    init();
     
    13101309                @Override
    13111310                public void actionPerformed(ActionEvent e) {
    1312                     boolean sel=((JCheckBoxMenuItem) e.getSource()).getState();
     1311                    boolean sel = ((JCheckBoxMenuItem) e.getSource()).getState();
    13131312                    Main.pref.put("draw.anglesnap.drawConstructionGeometry", sel);
    13141313                    Main.pref.put("draw.anglesnap.drawProjectedPoint", sel);
     
    13231322                @Override
    13241323                public void actionPerformed(ActionEvent e) {
    1325                     boolean sel=((JCheckBoxMenuItem) e.getSource()).getState();
     1324                    boolean sel = ((JCheckBoxMenuItem) e.getSource()).getState();
    13261325                    Main.pref.put("draw.anglesnap.projectionsnap", sel);
    13271326                    init();
     
    13311330
    13321331            private AnglePopupMenu() {
    1333                 helperCb.setState(Main.pref.getBoolean("draw.anglesnap.drawConstructionGeometry",true));
    1334                 projectionCb.setState(Main.pref.getBoolean("draw.anglesnap.projectionsnapgvff",true));
    1335                 repeatedCb.setState(Main.pref.getBoolean("draw.anglesnap.toggleOnRepeatedA",true));
     1332                helperCb.setState(Main.pref.getBoolean("draw.anglesnap.drawConstructionGeometry", true));
     1333                projectionCb.setState(Main.pref.getBoolean("draw.anglesnap.projectionsnapgvff", true));
     1334                repeatedCb.setState(Main.pref.getBoolean("draw.anglesnap.toggleOnRepeatedA", true));
    13361335                add(repeatedCb);
    13371336                add(helperCb);
     
    13461345                add(new AbstractAction(tr("0,90,...")) {
    13471346                    @Override public void actionPerformed(ActionEvent e) {
    1348                         saveAngles("0","90","180");
     1347                        saveAngles("0", "90", "180");
    13491348                        init();
    13501349                        enableSnapping();
     
    13531352                add(new AbstractAction(tr("0,45,90,...")) {
    13541353                    @Override public void actionPerformed(ActionEvent e) {
    1355                         saveAngles("0","45","90","135","180");
     1354                        saveAngles("0", "45", "90", "135", "180");
    13561355                        init();
    13571356                        enableSnapping();
     
    13601359                add(new AbstractAction(tr("0,30,45,60,90,...")) {
    13611360                    @Override public void actionPerformed(ActionEvent e) {
    1362                         saveAngles("0","30","45","60","90","120","135","150","180");
     1361                        saveAngles("0", "30", "45", "60", "90", "120", "135", "150", "180");
    13631362                        init();
    13641363                        enableSnapping();
     
    13851384        private double lastAngle;
    13861385
    1387         private double customBaseHeading=-1; // angle of base line, if not last segment)
     1386        private double customBaseHeading = -1; // angle of base line, if not last segment)
    13881387        private EastNorth segmentPoint1; // remembered first point of base segment
    13891388        private EastNorth segmentPoint2; // remembered second point of base segment
     
    13931392        private double snapAngleTolerance;
    13941393
    1395         private double pe,pn; // (pe,pn) - direction of snapping line
    1396         private double e0,n0; // (e0,n0) - origin of snapping line
    1397 
    1398         private final String fixFmt="%d "+tr("FIX");
     1394        private double pe, pn; // (pe, pn) - direction of snapping line
     1395        private double e0, n0; // (e0, n0) - origin of snapping line
     1396
     1397        private final String fixFmt = "%d "+tr("FIX");
    13991398        private Color snapHelperColor;
    14001399        private Color highlightColor;
     
    14181417
    14191418        public void init() {
    1420             snapOn=false;
     1419            snapOn = false;
    14211420            checkBox.setState(snapOn);
    1422             fixed=false; absoluteFix=false;
     1421            fixed = false;
     1422            absoluteFix = false;
    14231423
    14241424            Collection<String> angles = Main.pref.getCollection("draw.anglesnap.angles",
    1425                     Arrays.asList("0","30","45","60","90","120","135","150","180"));
     1425                    Arrays.asList("0", "30", "45", "60", "90", "120", "135", "150", "180"));
    14261426
    14271427            snapAngles = new double[2*angles.size()];
    1428             int i=0;
     1428            int i = 0;
    14291429            for (String s: angles) {
    14301430                try {
     
    14331433                } catch (NumberFormatException e) {
    14341434                    Main.warn("Incorrect number in draw.anglesnap.angles preferences: "+s);
    1435                     snapAngles[i]=0;i++;
    1436                     snapAngles[i]=0;i++;
     1435                    snapAngles[i] = 0; i++;
     1436                    snapAngles[i] = 0; i++;
    14371437                }
    14381438            }
     
    14491449
    14501450            highlightColor = Main.pref.getColor(marktr("draw angle snap highlight"), ORANGE_TRANSPARENT);
    1451             highlightStroke = GuiHelper.getCustomizedStroke(Main.pref.get("draw.anglesnap.stroke.highlight","10"));
    1452             helperStroke = GuiHelper.getCustomizedStroke(Main.pref.get("draw.anglesnap.stroke.helper","1 4"));
     1451            highlightStroke = GuiHelper.getCustomizedStroke(Main.pref.get("draw.anglesnap.stroke.highlight", "10"));
     1452            helperStroke = GuiHelper.getCustomizedStroke(Main.pref.get("draw.anglesnap.stroke.helper", "1 4"));
    14531453        }
    14541454
     
    14641464            if (!snapOn || !active)
    14651465                return;
    1466             Point p1=mv.getPoint(getCurrentBaseNode());
    1467             Point p2=mv.getPoint(dir2);
    1468             Point p3=mv.getPoint(projected);
     1466            Point p1 = mv.getPoint(getCurrentBaseNode());
     1467            Point p2 = mv.getPoint(dir2);
     1468            Point p3 = mv.getPoint(projected);
    14691469            GeneralPath b;
    14701470            if (drawConstructionGeometry) {
     
    14741474                b = new GeneralPath();
    14751475                if (absoluteFix) {
    1476                     b.moveTo(p2.x,p2.y);
    1477                     b.lineTo(2*p1.x-p2.x,2*p1.y-p2.y); // bi-directional line
     1476                    b.moveTo(p2.x, p2.y);
     1477                    b.lineTo(2*p1.x-p2.x, 2*p1.y-p2.y); // bi-directional line
    14781478                } else {
    1479                     b.moveTo(p2.x,p2.y);
    1480                     b.lineTo(p3.x,p3.y);
     1479                    b.moveTo(p2.x, p2.y);
     1480                    b.lineTo(p3.x, p3.y);
    14811481                }
    14821482                g2.draw(b);
     
    14861486                g2.setStroke(helperStroke);
    14871487                b = new GeneralPath();
    1488                 b.moveTo(p3.x,p3.y);
    1489                 Point pp=mv.getPoint(projectionSource);
    1490                 b.lineTo(pp.x,pp.y);
     1488                b.moveTo(p3.x, p3.y);
     1489                Point pp = mv.getPoint(projectionSource);
     1490                b.lineTo(pp.x, pp.y);
    14911491                g2.draw(b);
    14921492            }
     
    14961496                g2.setStroke(highlightStroke);
    14971497                b = new GeneralPath();
    1498                 Point pp1=mv.getPoint(segmentPoint1);
    1499                 Point pp2=mv.getPoint(segmentPoint2);
    1500                 b.moveTo(pp1.x,pp1.y);
    1501                 b.lineTo(pp2.x,pp2.y);
     1498                Point pp1 = mv.getPoint(segmentPoint1);
     1499                Point pp2 = mv.getPoint(segmentPoint2);
     1500                b.moveTo(pp1.x, pp1.y);
     1501                b.lineTo(pp2.x, pp2.y);
    15021502                g2.draw(b);
    15031503            }
     
    15061506            g2.setStroke(normalStroke);
    15071507            b = new GeneralPath();
    1508             b.moveTo(p1.x,p1.y);
    1509             b.lineTo(p3.x,p3.y);
     1508            b.moveTo(p1.x, p1.y);
     1509            b.lineTo(p3.x, p3.y);
    15101510            g2.draw(b);
    15111511
     
    15271527            double angle = -1;
    15281528
    1529             double activeBaseHeading = (customBaseHeading>=0)? customBaseHeading : baseHeading;
    1530 
    1531             if (snapOn && (activeBaseHeading>=0)) {
     1529            double activeBaseHeading = (customBaseHeading >= 0) ? customBaseHeading : baseHeading;
     1530
     1531            if (snapOn && (activeBaseHeading >= 0)) {
    15321532                angle = curHeading - activeBaseHeading;
    15331533                if (angle < 0) {
    1534                     angle+=360;
     1534                    angle += 360;
    15351535                }
    15361536                if (angle > 360) {
    1537                     angle=0;
     1537                    angle = 0;
    15381538                }
    15391539
     
    15451545                    nearestAngle = getNearestAngle(angle);
    15461546                    if (getAngleDelta(nearestAngle, angle) < snapAngleTolerance) {
    1547                         active = (customBaseHeading>=0)? true : Math.abs(nearestAngle - 180) > 1e-3;
     1547                        active = (customBaseHeading >= 0) ? true : Math.abs(nearestAngle - 180) > 1e-3;
    15481548                        // if angle is to previous segment, exclude 180 degrees
    15491549                        lastAngle = nearestAngle;
    15501550                    } else {
    1551                         active=false;
     1551                        active = false;
    15521552                    }
    15531553                }
     
    15571557                    e0 = p0.east();
    15581558                    n0 = p0.north();
    1559                     buildLabelText((nearestAngle<=180) ? nearestAngle : nearestAngle-360);
     1559                    buildLabelText((nearestAngle <= 180) ? nearestAngle : nearestAngle-360);
    15601560
    15611561                    phi = (nearestAngle + activeBaseHeading) * Math.PI / 180;
     
    15801580                angle = hdg - baseHeading;
    15811581                if (angle < 0) {
    1582                     angle+=360;
     1582                    angle += 360;
    15831583                }
    15841584                if (angle > 360) {
    1585                     angle=0;
     1585                    angle = 0;
    15861586                }
    15871587            }
     
    16161616            if (!active)
    16171617                return p;
    1618             double de=p.east()-e0;
    1619             double dn=p.north()-n0;
     1618            double de = p.east()-e0;
     1619            double dn = p.north()-n0;
    16201620            double l = de*pe+dn*pn;
    16211621            double delta = Main.map.mapView.getDist100Pixel()/20;
    1622             if (!absoluteFix && l<delta) {
    1623                 active=false;
     1622            if (!absoluteFix && l < delta) {
     1623                active = false;
    16241624                return p;
    16251625            } //  do not go backward!
    16261626
    1627             projectionSource=null;
     1627            projectionSource = null;
    16281628            if (snapToProjections) {
    16291629                DataSet ds = getCurrentDataSet();
    16301630                Collection<Way> selectedWays = ds.getSelectedWays();
    1631                 if (selectedWays.size()==1) {
     1631                if (selectedWays.size() == 1) {
    16321632                    Way w = selectedWays.iterator().next();
    16331633                    Collection<EastNorth> pointsToProject = new ArrayList<>();
    1634                     if (w.getNodesCount()<1000) {
     1634                    if (w.getNodesCount() < 1000) {
    16351635                        for (Node n: w.getNodes()) {
    16361636                            pointsToProject.add(n.getEastNorth());
     
    16411641                        pointsToProject.add(segmentPoint2);
    16421642                    }
    1643                     EastNorth enOpt=null;
    1644                     double dOpt=1e5;
     1643                    EastNorth enOpt = null;
     1644                    double dOpt = 1e5;
    16451645                    for (EastNorth en: pointsToProject) { // searching for besht projection
    16461646                        double l1 = (en.east()-e0)*pe+(en.north()-n0)*pn;
    16471647                        double d1 = Math.abs(l1-l);
    16481648                        if (d1 < delta && d1 < dOpt) {
    1649                             l=l1;
     1649                            l = l1;
    16501650                            enOpt = en;
    16511651                            dOpt = d1;
    16521652                        }
    16531653                    }
    1654                     if (enOpt!=null) {
     1654                    if (enOpt != null) {
    16551655                        projectionSource =  enOpt;
    16561656                    }
     
    16611661
    16621662        public void noSnapNow() {
    1663             active=false;
    1664             dir2=null; projected=null;
    1665             labelText=null;
     1663            active = false;
     1664            dir2 = null;
     1665            projected = null;
     1666            labelText = null;
    16661667        }
    16671668
    16681669        public void setBaseSegment(WaySegment seg) {
    1669             if (seg==null) return;
    1670             segmentPoint1=seg.getFirstNode().getEastNorth();
    1671             segmentPoint2=seg.getSecondNode().getEastNorth();
     1670            if (seg == null) return;
     1671            segmentPoint1 = seg.getFirstNode().getEastNorth();
     1672            segmentPoint2 = seg.getSecondNode().getEastNorth();
    16721673
    16731674            double hdg = segmentPoint1.heading(segmentPoint2);
    1674             hdg=Math.toDegrees(hdg);
    1675             if (hdg<0) {
    1676                 hdg+=360;
    1677             }
    1678             if (hdg>360) {
    1679                 hdg-=360;
    1680             }
    1681             customBaseHeading=hdg;
     1675            hdg = Math.toDegrees(hdg);
     1676            if (hdg < 0) {
     1677                hdg += 360;
     1678            }
     1679            if (hdg > 360) {
     1680                hdg -= 360;
     1681            }
     1682            customBaseHeading = hdg;
    16821683        }
    16831684
     
    16921693                }
    16931694            } else {
    1694                 snapOn=true;
     1695                snapOn = true;
    16951696                unsetFixedMode();
    16961697            }
    16971698            checkBox.setState(snapOn);
    1698             customBaseHeading=-1;
     1699            customBaseHeading = -1;
    16991700        }
    17001701
     
    17021703            snapOn = true;
    17031704            checkBox.setState(snapOn);
    1704             customBaseHeading=-1;
     1705            customBaseHeading = -1;
    17051706            unsetFixedMode();
    17061707        }
     
    17091710            snapOn = !snapOn;
    17101711            checkBox.setState(snapOn);
    1711             customBaseHeading=-1;
     1712            customBaseHeading = -1;
    17121713            unsetFixedMode();
    17131714        }
     
    17151716        public void setFixedMode() {
    17161717            if (active) {
    1717                 fixed=true;
     1718                fixed = true;
    17181719            }
    17191720        }
    17201721
    17211722        public  void unsetFixedMode() {
    1722             fixed=false;
    1723             absoluteFix=false;
    1724             lastAngle=0;
    1725             active=false;
     1723            fixed = false;
     1724            absoluteFix = false;
     1725            lastAngle = 0;
     1726            active = false;
    17261727        }
    17271728
     
    17351736
    17361737        private double getNearestAngle(double angle) {
    1737             double delta,minDelta=1e5, bestAngle=0.0;
     1738            double delta, minDelta = 1e5, bestAngle = 0.0;
    17381739            for (double snapAngle : snapAngles) {
    17391740                delta = getAngleDelta(angle, snapAngle);
     
    17441745            }
    17451746            if (Math.abs(bestAngle-360) < 1e-3) {
    1746                 bestAngle=0;
     1747                bestAngle = 0;
    17471748            }
    17481749            return bestAngle;
     
    17511752        private double getAngleDelta(double a, double b) {
    17521753            double delta = Math.abs(a-b);
    1753             if (delta>180)
     1754            if (delta > 180)
    17541755                return 360-delta;
    17551756            else
     
    17781779        @Override
    17791780        public void actionPerformed(ActionEvent e) {
    1780             if (snapHelper!=null) {
     1781            if (snapHelper != null) {
    17811782                snapHelper.toggleSnapping();
    17821783            }
  • trunk/src/org/openstreetmap/josm/actions/mapmode/ExtrudeAction.java

    r8509 r8510  
    233233    private JCheckBoxMenuItem addDualAlignMenuItem() {
    234234        int n = Main.main.menu.editMenu.getItemCount();
    235         for (int i = n-1; i>0; i--) {
     235        for (int i = n-1; i > 0; i--) {
    236236            JMenuItem item = Main.main.menu.editMenu.getItem(i);
    237237            if (item != null && item.getAction() != null && item.getAction() instanceof DualAlignChangeAction) {
     
    297297
    298298    private void readPreferences() {
    299         initialMoveDelay = Main.pref.getInteger("edit.initial-move-delay",200);
     299        initialMoveDelay = Main.pref.getInteger("edit.initial-move-delay", 200);
    300300        initialMoveThreshold = Main.pref.getInteger("extrude.initial-move-threshold", 1);
    301301        mainColor = Main.pref.getColor(marktr("Extrude: main line"), null);
     
    374374    @Override
    375375    public void mousePressed(MouseEvent e) {
    376         if(!Main.map.mapView.isActiveLayerVisible())
     376        if (!Main.map.mapView.isActiveLayerVisible())
    377377            return;
    378         if (!(Boolean)this.getValue("active"))
     378        if (!(Boolean) this.getValue("active"))
    379379            return;
    380380        if (e.getButton() != MouseEvent.BUTTON1)
     
    453453    @Override
    454454    public void mouseDragged(MouseEvent e) {
    455         if(!Main.map.mapView.isActiveLayerVisible())
     455        if (!Main.map.mapView.isActiveLayerVisible())
    456456            return;
    457457
     
    516516    public void mouseReleased(MouseEvent e) {
    517517
    518         if(!Main.map.mapView.isActiveLayerVisible())
     518        if (!Main.map.mapView.isActiveLayerVisible())
    519519            return;
    520520
     
    703703        Node locNode = MergeNodesAction.selectTargetLocationNode(changedNodes);
    704704        Command mergeCmd = MergeNodesAction.mergeNodes(Main.main.getEditLayer(), changedNodes, targetNode, locNode);
    705         if (mergeCmd!=null) {
     705        if (mergeCmd != null) {
    706706            Main.main.undoRedo.add(mergeCmd);
    707707        } else {
     
    772772            EastNorth targetPos) {
    773773        EastNorth intersectionPoint;
    774         if (segmentP1.distanceSq(segmentP2)>1e-7) {
     774        if (segmentP1.distanceSq(segmentP2) > 1e-7) {
    775775            intersectionPoint = Geometry.getLineLineIntersection(segmentP1, segmentP2, targetPos, targetPos.add(moveDirection));
    776776        } else {
     
    858858        EastNorth n1en = selectedSegment.getFirstNode().getEastNorth();
    859859        EastNorth n2en = selectedSegment.getSecondNode().getEastNorth();
    860         if (n1en.distance(prevNode.getEastNorth())<1e-4 ||
    861             n2en.distance(nextNode.getEastNorth())<1e-4) {
     860        if (n1en.distance(prevNode.getEastNorth()) < 1e-4 ||
     861            n2en.distance(nextNode.getEastNorth()) < 1e-4) {
    862862            return false;
    863863        }
     
    10531053                    if (p1.distance(p2) < 3) {
    10541054                        g2.setStroke(mainStroke);
    1055                         g2.drawOval((int)(p1.x-symbolSize/2), (int)(p1.y-symbolSize/2),
    1056                                 (int)(symbolSize), (int)(symbolSize));
     1055                        g2.drawOval((int) (p1.x-symbolSize/2), (int) (p1.y-symbolSize/2),
     1056                                (int) (symbolSize), (int) (symbolSize));
    10571057                    } else {
    10581058                        Line2D oldline = new Line2D.Double(p1, p2);
     
    11341134
    11351135        GeneralPath ra = new GeneralPath();
    1136         ra.moveTo((float)ra1.getX(), (float)ra1.getY());
    1137         ra.lineTo((float)ra2.getX(), (float)ra2.getY());
    1138         ra.lineTo((float)ra3.getX(), (float)ra3.getY());
     1136        ra.moveTo((float) ra1.getX(), (float) ra1.getY());
     1137        ra.lineTo((float) ra2.getX(), (float) ra2.getY());
     1138        ra.lineTo((float) ra3.getX(), (float) ra3.getY());
    11391139        g2.setStroke(helperStrokeRA);
    11401140        g2.draw(ra);
     
    11691169        try {
    11701170            AffineTransform invtrans = g.getTransform().createInverse();
    1171             Point2D widthpoint = invtrans.deltaTransform(new Point2D.Double(bounds.width,0), null);
    1172             Point2D heightpoint = invtrans.deltaTransform(new Point2D.Double(0,bounds.height), null);
     1171            Point2D widthpoint = invtrans.deltaTransform(new Point2D.Double(bounds.width, 0), null);
     1172            Point2D heightpoint = invtrans.deltaTransform(new Point2D.Double(0, bounds.height), null);
    11731173
    11741174            // Here we should end up with a gross overestimate of the maximum viewport diagonal in what
  • trunk/src/org/openstreetmap/josm/actions/mapmode/ImproveWayAccuracyAction.java

    r8378 r8510  
    149149        addNodeStroke = GuiHelper.getCustomizedStroke(Main.pref.get("improvewayaccuracy.stroke.add-node", "1"));
    150150        deleteNodeStroke = GuiHelper.getCustomizedStroke(Main.pref.get("improvewayaccuracy.stroke.delete-node", "1"));
    151         dotSize = Main.pref.getInteger("improvewayaccuracy.dot-size",6);
     151        dotSize = Main.pref.getInteger("improvewayaccuracy.dot-size", 6);
    152152    }
    153153
  • trunk/src/org/openstreetmap/josm/actions/mapmode/MapMode.java

    r8378 r8510  
    100100    protected void updateKeyModifiers(int modifiers) {
    101101        ctrl = (modifiers & ActionEvent.CTRL_MASK) != 0;
    102         alt = (modifiers & (ActionEvent.ALT_MASK|InputEvent.ALT_GRAPH_MASK)) != 0;
     102        alt = (modifiers & (ActionEvent.ALT_MASK | InputEvent.ALT_GRAPH_MASK)) != 0;
    103103        shift = (modifiers & ActionEvent.SHIFT_MASK) != 0;
    104104    }
     
    112112
    113113    @Override
    114     public void mouseReleased(MouseEvent e) {requestFocusInMapView();}
     114    public void mouseReleased(MouseEvent e) {
     115        requestFocusInMapView();
     116    }
     117
    115118    @Override
    116     public void mouseExited(MouseEvent e) {}
     119    public void mouseExited(MouseEvent e) {
     120        // Do nothing
     121    }
     122
    117123    @Override
    118     public void mousePressed(MouseEvent e) {requestFocusInMapView();}
     124    public void mousePressed(MouseEvent e) {
     125        requestFocusInMapView();
     126    }
     127
    119128    @Override
    120     public void mouseClicked(MouseEvent e) {}
     129    public void mouseClicked(MouseEvent e) {
     130        // Do nothing
     131    }
     132
    121133    @Override
    122     public void mouseEntered(MouseEvent e) {}
     134    public void mouseEntered(MouseEvent e) {
     135        // Do nothing
     136    }
     137
    123138    @Override
    124     public void mouseMoved(MouseEvent e) {}
     139    public void mouseMoved(MouseEvent e) {
     140        // Do nothing
     141    }
     142
    125143    @Override
    126     public void mouseDragged(MouseEvent e) {}
     144    public void mouseDragged(MouseEvent e) {
     145        // Do nothing
     146    }
    127147}
  • trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java

    r8470 r8510  
    106106            c = ImageProvider.getCursor(main, sub);
    107107        }
     108
    108109        private SelectActionCursor(int systemCursor) {
    109110            c = Cursor.getPredefinedCursor(systemCursor);
    110111        }
     112
    111113        public Cursor cursor() {
    112114            return c;
     
    218220    @Override
    219221    public void modifiersChanged(int modifiers) {
    220         if (!Main.isDisplayingMapView() || oldEvent==null) return;
    221         if(giveUserFeedback(oldEvent, modifiers)) {
     222        if (!Main.isDisplayingMapView() || oldEvent == null) return;
     223        if (giveUserFeedback(oldEvent, modifiers)) {
    222224            mv.repaint();
    223225        }
     
    251253
    252254        virtualManager.clear();
    253         if(mode == Mode.MOVE) {
     255        if (mode == Mode.MOVE) {
    254256            if (!dragInProgress() && virtualManager.activateVirtualNodeNearPoint(e.getPoint())) {
    255257                DataSet ds = getCurrentDataSet();
     
    266268
    267269        // return early if there can't be any highlights
    268         if(!drawTargetHighlight || mode != Mode.MOVE || c.isEmpty())
     270        if (!drawTargetHighlight || mode != Mode.MOVE || c.isEmpty())
    269271            return repaintIfRequired(newHighlights);
    270272
    271273        // CTRL toggles selection, but if while dragging CTRL means merge
    272274        final boolean isToggleMode = ctrl && !dragInProgress();
    273         for(OsmPrimitive x : c) {
     275        for (OsmPrimitive x : c) {
    274276            // only highlight primitives that will change the selection
    275277            // when clicked. I.e. don't highlight selected elements unless
    276278            // we are in toggle mode.
    277             if(isToggleMode || !x.isSelected()) {
     279            if (isToggleMode || !x.isSelected()) {
    278280                newHighlights.add(x);
    279281            }
     
    293295        switch(mode) {
    294296        case MOVE:
    295             if(virtualManager.hasVirtualNode()) {
     297            if (virtualManager.hasVirtualNode()) {
    296298                c = "virtual_node";
    297299                break;
     
    300302            final OsmPrimitive osm = it.hasNext() ? it.next() : null;
    301303
    302             if(dragInProgress()) {
     304            if (dragInProgress()) {
    303305                // only consider merge if ctrl is pressed and there are nodes in
    304306                // the selection that could be merged
    305                 if(!ctrl || getCurrentDataSet().getSelectedNodes().isEmpty()) {
     307                if (!ctrl || getCurrentDataSet().getSelectedNodes().isEmpty()) {
    306308                    c = "move";
    307309                    break;
     
    316318            c = (osm instanceof Node) ? "node" : c;
    317319            c = (osm instanceof Way) ? "way" : c;
    318             if(shift) {
     320            if (shift) {
    319321                c += "_add";
    320             } else if(ctrl) {
     322            } else if (ctrl) {
    321323                c += osm == null || osm.isSelected() ? "_rm" : "_add";
    322324            }
     
    346348        boolean needsRepaint = false;
    347349        DataSet ds = getCurrentDataSet();
    348         if(ds != null && !ds.getHighlightedVirtualNodes().isEmpty()) {
     350        if (ds != null && !ds.getHighlightedVirtualNodes().isEmpty()) {
    349351            needsRepaint = true;
    350352            ds.clearHighlightedVirtualNodes();
    351353        }
    352         if(oldHighlights.isEmpty())
     354        if (oldHighlights.isEmpty())
    353355            return needsRepaint;
    354356
    355         for(OsmPrimitive prim : oldHighlights) {
     357        for (OsmPrimitive prim : oldHighlights) {
    356358            prim.setHighlighted(false);
    357359        }
     
    361363
    362364    private boolean repaintIfRequired(Set<OsmPrimitive> newHighlights) {
    363         if(!drawTargetHighlight)
     365        if (!drawTargetHighlight)
    364366            return false;
    365367
    366368        boolean needsRepaint = false;
    367         for(OsmPrimitive x : newHighlights) {
    368             if(oldHighlights.contains(x)) {
     369        for (OsmPrimitive x : newHighlights) {
     370            if (oldHighlights.contains(x)) {
    369371                continue;
    370372            }
     
    373375        }
    374376        oldHighlights.removeAll(newHighlights);
    375         for(OsmPrimitive x : oldHighlights) {
     377        for (OsmPrimitive x : oldHighlights) {
    376378            x.setHighlighted(false);
    377379            needsRepaint = true;
     
    412414        mouseDownTime = System.currentTimeMillis();
    413415        lastMousePos = e.getPoint();
    414         startEN = mv.getEastNorth(lastMousePos.x,lastMousePos.y);
     416        startEN = mv.getEastNorth(lastMousePos.x, lastMousePos.y);
    415417
    416418        // primitives under cursor are stored in c collection
     
    418420        OsmPrimitive nearestPrimitive = mv.getNearestNodeOrWay(e.getPoint(), mv.isSelectablePredicate, true);
    419421
    420         determineMapMode(nearestPrimitive!=null);
     422        determineMapMode(nearestPrimitive != null);
    421423
    422424        switch(mode) {
     
    473475        }
    474476        oldEvent = e;
    475         if(giveUserFeedback(e)) {
     477        if (giveUserFeedback(e)) {
    476478            mv.repaint();
    477479        }
     
    520522            final OsmPrimitive p = canMerge ? findNodeToMergeTo(e.getPoint()) : null;
    521523            boolean needsRepaint = removeHighlighting();
    522             if(p != null) {
     524            if (p != null) {
    523525                p.setHighlighted(true);
    524526                oldHighlights.add(p);
     
    529531            // when dragging a node onto another one and then press CTRL to merge
    530532            oldEvent = e;
    531             if(needsRepaint) {
     533            if (needsRepaint) {
    532534                mv.repaint();
    533535            }
     
    570572    @Override
    571573    public void mouseExited(MouseEvent e) {
    572         if(removeHighlighting()) {
     574        if (removeHighlighting()) {
    573575            mv.repaint();
    574576        }
     
    629631        // selected beforehand this would put us into move mode as well, which breaks
    630632        // the cycling through primitives on top of each other (see #6739).
    631         if(e.getButton() == MouseEvent.BUTTON2) {
     633        if (e.getButton() == MouseEvent.BUTTON2) {
    632634            removeHighlighting();
    633635        } else {
     
    954956                return tr("Release the mouse button to select the objects in the rectangle.");
    955957            else if (mode == Mode.MOVE && (System.currentTimeMillis() - mouseDownTime >= initialMoveDelay)) {
    956                 final boolean canMerge = getCurrentDataSet()!=null && !getCurrentDataSet().getSelectedNodes().isEmpty();
     958                final boolean canMerge = getCurrentDataSet() != null && !getCurrentDataSet().getSelectedNodes().isEmpty();
    957959                final String mergeHelp = canMerge ? " " + tr("Ctrl to merge with nearest node.") : "";
    958960                return tr("Release the mouse button to stop moving.") + mergeHelp;
     
    962964                return tr("Release the mouse button to stop scaling.");
    963965        }
    964         return tr("Move objects by dragging; Shift to add to selection (Ctrl to toggle); Shift-Ctrl to rotate selected; Alt-Ctrl to scale selected; or change selection");
     966        return tr("Move objects by dragging; Shift to add to selection (Ctrl to toggle); Shift-Ctrl to rotate selected; " +
     967                  "Alt-Ctrl to scale selected; or change selection");
    965968    }
    966969
  • trunk/src/org/openstreetmap/josm/actions/relation/DeleteRelationsAction.java

    r8444 r8510  
    2121     */
    2222    public DeleteRelationsAction() {
    23         putValue(SHORT_DESCRIPTION,tr("Delete the selected relation"));
     23        putValue(SHORT_DESCRIPTION, tr("Delete the selected relation"));
    2424        putValue(NAME, tr("Delete"));
    2525        putValue(SMALL_ICON, ImageProvider.get("dialogs", "delete"));
     
    3232                .deleteRelation(Main.main.getEditLayer(), toDelete);
    3333        // clear selection after deletion
    34         if (Main.map.relationListDialog!=null)
     34        if (Main.map.relationListDialog != null)
    3535                Main.map.relationListDialog.selectRelations(null);
    3636    }
  • trunk/src/org/openstreetmap/josm/actions/relation/DownloadMembersAction.java

    r7937 r8510  
    4242    public void setPrimitives(Collection<? extends OsmPrimitive> primitives) {
    4343        // selected non-new relations
    44         this.relations = Utils.filter(getRelations(primitives), new Predicate<Relation>(){
     44        this.relations = Utils.filter(getRelations(primitives), new Predicate<Relation>() {
    4545            @Override public boolean evaluate(Relation r) {
    4646                return !r.isNew();
  • trunk/src/org/openstreetmap/josm/actions/relation/DownloadSelectedIncompleteMembersAction.java

    r8308 r8510  
    6565    public void setPrimitives(Collection<? extends OsmPrimitive> primitives) {
    6666        // selected relations with incomplete members
    67         this.relations = Utils.filter(getRelations(primitives), new Predicate<Relation>(){
     67        this.relations = Utils.filter(getRelations(primitives), new Predicate<Relation>() {
    6868            @Override
    6969            public boolean evaluate(Relation r) {
     
    7676    @Override
    7777    protected void updateEnabledState() {
    78         setEnabled(!relations.isEmpty() &&!incompleteMembers.isEmpty() && !Main.isOffline(OnlineResource.OSM_API));
     78        setEnabled(!relations.isEmpty() && !incompleteMembers.isEmpty() && !Main.isOffline(OnlineResource.OSM_API));
    7979    }
    8080}
  • trunk/src/org/openstreetmap/josm/actions/relation/DuplicateRelationAction.java

    r8444 r8510  
    5252    protected void updateEnabledState() {
    5353        // only one selected relation can be edited
    54         setEnabled(relations.size()==1);
     54        setEnabled(relations.size() == 1);
    5555    }
    5656}
  • trunk/src/org/openstreetmap/josm/actions/relation/SelectInRelationListAction.java

    r7937 r8510  
    2626    @Override
    2727    public void actionPerformed(ActionEvent e) {
    28         if (!isEnabled() || relations.isEmpty() || Main.map==null || Main.map.relationListDialog==null) return;
     28        if (!isEnabled() || relations.isEmpty() || Main.map == null || Main.map.relationListDialog == null) return;
    2929        Main.map.relationListDialog.unfurlDialog();
    3030        Main.map.relationListDialog.selectRelations(relations);
  • trunk/src/org/openstreetmap/josm/actions/relation/SelectMembersAction.java

    r8509 r8510  
    2727     */
    2828    public SelectMembersAction(boolean add) {
    29         putValue(SHORT_DESCRIPTION,add ? tr("Add the members of all selected relations to current selection")
     29        putValue(SHORT_DESCRIPTION, add ? tr("Add the members of all selected relations to current selection")
    3030                : tr("Select the members of all selected relations"));
    3131        putValue(SMALL_ICON, ImageProvider.get("selectall"));
  • trunk/src/org/openstreetmap/josm/actions/relation/SelectRelationAction.java

    r8509 r8510  
    3535        if (!isEnabled() || relations.isEmpty()) return;
    3636        OsmDataLayer editLayer = Main.main.getEditLayer();
    37         if (editLayer==null || editLayer.data==null) return;
     37        if (editLayer == null || editLayer.data == null) return;
    3838        if (add) {
    3939            editLayer.data.addSelected(relations);
  • trunk/src/org/openstreetmap/josm/actions/search/PushbackTokenizer.java

    r8390 r8510  
    7070    }
    7171
    72 
    7372    private void getChar() {
    7473        try {
     
    8685        StringBuilder s = new StringBuilder();
    8786        boolean escape = false;
    88         while (c != -1 && (escape || (!sChars.contains((char)c) && (quoted || !Character.isWhitespace(c))))) {
     87        while (c != -1 && (escape || (!sChars.contains((char) c) && (quoted || !Character.isWhitespace(c))))) {
    8988            if (c == '\\' && !escape) {
    9089                escape = true;
    9190            } else {
    92                 s.append((char)c);
     91                s.append((char) c);
    9392                escape = false;
    9493            }
  • trunk/src/org/openstreetmap/josm/actions/search/SearchAction.java

    r8509 r8510  
    9999
    100100    public static void saveToHistory(SearchSetting s) {
    101         if(searchHistory.isEmpty() || !s.equals(searchHistory.getFirst())) {
     101        if (searchHistory.isEmpty() || !s.equals(searchHistory.getFirst())) {
    102102            searchHistory.addFirst(new SearchSetting(s));
    103103        } else if (searchHistory.contains(s)) {
     
    261261        left.add(in_selection, GBC.eop());
    262262        left.add(caseSensitive, GBC.eol());
    263         if(Main.pref.getBoolean("expert", false)) {
     263        if (Main.pref.getBoolean("expert", false)) {
    264264            left.add(allElements, GBC.eol());
    265265            left.add(regexSearch, GBC.eol());
     
    306306        int result = dialog.getValue();
    307307
    308         if(result != 1) return null;
     308        if (result != 1) return null;
    309309
    310310        // User pressed OK - let's perform the search
     
    342342        right.add(new SearchKeywordRow(hcbSearchString)
    343343                .addTitle(tr("basics"))
    344                 .addKeyword("<i>key</i>:<i>valuefragment</i>", null, tr("''valuefragment'' anywhere in ''key''"), "name:str matches name=Bakerstreet")
     344                .addKeyword("<i>key</i>:<i>valuefragment</i>", null,
     345                        tr("''valuefragment'' anywhere in ''key''"), "name:str matches name=Bakerstreet")
    345346                .addKeyword("-<i>key</i>:<i>valuefragment</i>", null, tr("''valuefragment'' nowhere in ''key''"))
    346347                .addKeyword("<i>key</i>=<i>value</i>", null, tr("''key'' with exactly ''value''"))
     
    350351                .addKeyword("<i>key</i>><i>value</i>", null, tr("matches if ''key'' is greater than ''value'' (analogously, less than)"))
    351352                .addKeyword("\"key\"=\"value\"", "\"\"=\"\"",
    352                         tr("to quote operators.<br>Within quoted strings the <b>\"</b> and <b>\\</b> characters need to be escaped by a preceding <b>\\</b> (e.g. <b>\\\"</b> and <b>\\\\</b>)."),
     353                        tr("to quote operators.<br>Within quoted strings the <b>\"</b> and <b>\\</b> characters need to be escaped " +
     354                           "by a preceding <b>\\</b> (e.g. <b>\\\"</b> and <b>\\\\</b>)."),
    353355                        "\"addr:street\"")
    354356                , GBC.eol());
     
    420422    public static void search() {
    421423        SearchSetting se = showSearchDialog(lastSearch);
    422         if(se != null) {
     424        if (se != null) {
    423425            searchWithHistory(se);
    424426        }
     
    460462
    461463            Collection<OsmPrimitive> all;
    462             if(s.allElements) {
     464            if (s.allElements) {
    463465                all = Main.main.getCurrentDataSet().allPrimitives();
    464466            } else {
     
    505507        try {
    506508            String searchText = s.text;
    507             if (s instanceof Filter && ((Filter)s).inverted) {
     509            if (s instanceof Filter && ((Filter) s).inverted) {
    508510                searchText = String.format("-(%s)", searchText);
    509511            }
     
    543545        final DataSet ds = Main.main.getCurrentDataSet();
    544546        Collection<OsmPrimitive> sel = new HashSet<>(ds.getAllSelected());
    545         int foundMatches = getSelection(s, sel, new Predicate<OsmPrimitive>(){
     547        int foundMatches = getSelection(s, sel, new Predicate<OsmPrimitive>() {
    546548            @Override
    547             public boolean evaluate(OsmPrimitive o){
     549            public boolean evaluate(OsmPrimitive o) {
    548550                return ds.isSelected(o);
    549551            }
     
    617619        @Override
    618620        public boolean equals(Object other) {
    619             if(!(other instanceof SearchSetting))
     621            if (!(other instanceof SearchSetting))
    620622                return false;
    621623            SearchSetting o = (SearchSetting) other;
  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r8495 r8510  
    8989            // TODO: check for keyword collisions
    9090            if (factory instanceof SimpleMatchFactory) {
    91                 simpleMatchFactoryMap.put(keyword, (SimpleMatchFactory)factory);
     91                simpleMatchFactoryMap.put(keyword, (SimpleMatchFactory) factory);
    9292            } else if (factory instanceof UnaryMatchFactory) {
    93                 unaryMatchFactoryMap.put(keyword, (UnaryMatchFactory)factory);
     93                unaryMatchFactoryMap.put(keyword, (UnaryMatchFactory) factory);
    9494            } else if (factory instanceof BinaryMatchFactory) {
    95                 binaryMatchFactoryMap.put(keyword, (BinaryMatchFactory)factory);
     95                binaryMatchFactoryMap.put(keyword, (BinaryMatchFactory) factory);
    9696            } else
    9797                throw new AssertionError("Unknown match factory");
     
    326326            super(match);
    327327        }
     328
    328329        @Override
    329330        public boolean match(OsmPrimitive osm) {
    330331            return !match.match(osm);
    331332        }
     333
    332334        @Override
    333335        public String toString() {
    334336            return "!" + match;
    335337        }
     338
    336339        public Match getMatch() {
    337340            return match;
     
    350353            this.defaultValue = defaultValue;
    351354        }
     355
    352356        @Override
    353357        public boolean match(OsmPrimitive osm) {
     
    367371            super(lhs, rhs);
    368372        }
     373
    369374        @Override
    370375        public boolean match(OsmPrimitive osm) {
    371376            return lhs.match(osm) && rhs.match(osm);
    372377        }
     378
    373379        @Override
    374380        public String toString() {
     
    384390            super(lhs, rhs);
    385391        }
     392
    386393        @Override
    387394        public boolean match(OsmPrimitive osm) {
    388395            return lhs.match(osm) || rhs.match(osm);
    389396        }
     397
    390398        @Override
    391399        public String toString() {
     
    401409            super(lhs, rhs);
    402410        }
     411
    403412        @Override
    404413        public boolean match(OsmPrimitive osm) {
    405414            return lhs.match(osm) ^ rhs.match(osm);
    406415        }
     416
    407417        @Override
    408418        public String toString() {
     
    418428            super(range);
    419429        }
     430
    420431        public Id(PushbackTokenizer tokenizer) throws ParseError {
    421432            this(tokenizer.readRange(tr("Range of primitive ids expected")));
    422433        }
     434
    423435        @Override
    424436        protected Long getNumber(OsmPrimitive osm) {
    425437            return osm.isNew() ? 0 : osm.getUniqueId();
    426438        }
     439
    427440        @Override
    428441        protected String getString() {
     
    438451            super(range);
    439452        }
     453
    440454        public ChangesetId(PushbackTokenizer tokenizer) throws ParseError {
    441455            this(tokenizer.readRange(tr("Range of changeset ids expected")));
    442456        }
     457
    443458        @Override
    444459        protected Long getNumber(OsmPrimitive osm) {
    445460            return (long) osm.getChangesetId();
    446461        }
     462
    447463        @Override
    448464        protected String getString() {
     
    458474            super(range);
    459475        }
     476
    460477        public Version(PushbackTokenizer tokenizer) throws ParseError {
    461478            this(tokenizer.readRange(tr("Range of versions expected")));
    462479        }
     480
    463481        @Override
    464482        protected Long getNumber(OsmPrimitive osm) {
    465483            return (long) osm.getVersion();
    466484        }
     485
    467486        @Override
    468487        protected String getString() {
     
    551570                } else {
    552571                    mv = osm.get(key);
    553                     if(!caseSensitive && mv == null) {
     572                    if (!caseSensitive && mv == null) {
    554573                        for (String k: osm.keySet()) {
    555                             if(key.equalsIgnoreCase(k)) {
     574                            if (key.equalsIgnoreCase(k)) {
    556575                                mv = osm.get(k);
    557576                                break;
     
    574593            return false;
    575594        }
     595
    576596        @Override
    577597        public String toString() {
     
    628648                throw new ParseError(tr("Key cannot be empty when tag operator is used. Sample use: key=value"));
    629649            this.key = key;
    630             this.value = value == null?"":value;
     650            this.value = value == null ? "" : value;
    631651            if ("".equals(this.value) && "*".equals(key)) {
    632652                mode = Mode.NONE;
     
    800820            return false;
    801821        }
     822
    802823        @Override
    803824        public String toString() {
     
    808829    private static class ExactType extends Match {
    809830        private final OsmPrimitiveType type;
     831
    810832        public ExactType(String type) throws ParseError {
    811833            this.type = OsmPrimitiveType.from(type);
     
    814836                        type));
    815837        }
     838
    816839        @Override
    817840        public boolean match(OsmPrimitive osm) {
    818841            return type.equals(osm.getType());
    819842        }
     843
    820844        @Override
    821845        public String toString() {
     
    829853    private static class UserMatch extends Match {
    830854        private String user;
     855
    831856        public UserMatch(String user) {
    832857            if ("anonymous".equals(user)) {
     
    856881    private static class RoleMatch extends Match {
    857882        private String role;
     883
    858884        public RoleMatch(String role) {
    859885            if (role == null) {
     
    871897                        if (m.getMember() == osm) {
    872898                            String testRole = m.getRole();
    873                             if(role.equals(testRole == null ? "" : testRole))
     899                            if (role.equals(testRole == null ? "" : testRole))
    874900                                return true;
    875901                        }
     
    10681094            return "timestamp";
    10691095        }
    1070 
    10711096    }
    10721097
     
    10791104            return osm.isNew();
    10801105        }
     1106
    10811107        @Override
    10821108        public String toString() {
     
    10931119            return osm.isModified() || osm.isNewOrUndeleted();
    10941120        }
     1121
    10951122        @Override
    10961123        public String toString() {
     
    11071134            return Main.main.getCurrentDataSet().isSelected(osm);
    11081135        }
     1136
    11091137        @Override
    11101138        public String toString() {
     
    11231151            return osm.isIncomplete();
    11241152        }
     1153
    11251154        @Override
    11261155        public String toString() {
     
    11391168            return !osm.isTagged() && !osm.isIncomplete();
    11401169        }
     1170
    11411171        @Override
    11421172        public String toString() {
     
    11531183            return osm instanceof Way && ((Way) osm).isClosed();
    11541184        }
     1185
    11551186        @Override
    11561187        public String toString() {
     
    11661197            super(m);
    11671198        }
     1199
    11681200        @Override
    11691201        public boolean match(OsmPrimitive osm) {
     
    11711203
    11721204            if (osm instanceof Way) {
    1173                 for (Node n : ((Way)osm).getNodes()) {
     1205                for (Node n : ((Way) osm).getNodes()) {
    11741206                    isParent |= match.match(n);
    11751207                }
    11761208            } else if (osm instanceof Relation) {
    1177                 for (RelationMember member : ((Relation)osm).getMembers()) {
     1209                for (RelationMember member : ((Relation) osm).getMembers()) {
    11781210                    isParent |= match.match(member.getMember());
    11791211                }
     
    11811213            return isParent;
    11821214        }
     1215
    11831216        @Override
    11841217        public String toString() {
     
    12041237            return isChild;
    12051238        }
     1239
    12061240        @Override
    12071241        public String toString() {
     
    12711305    private abstract static class InArea extends Match {
    12721306
    1273         protected abstract Bounds getBounds();
    12741307        protected final boolean all;
    12751308
     
    12801313            this.all = all;
    12811314        }
     1315
     1316        protected abstract Bounds getBounds();
    12821317
    12831318        @Override
     
    13371372            super(msg);
    13381373        }
     1374
    13391375        public ParseError(String msg, Throwable cause) {
    13401376            super(msg, cause);
    13411377        }
     1378
    13421379        public ParseError(Token expected, Token found) {
    13431380            this(tr("Unexpected token. Expected {0}, found {1}", expected, found));
  • trunk/src/org/openstreetmap/josm/actions/upload/ApiPreconditionCheckerHook.java

    r8444 r8510  
    7979
    8080            if (osmPrimitive instanceof Way &&
    81                     ((Way)osmPrimitive).getNodesCount() > maxNodes) {
     81                    ((Way) osmPrimitive).getNodesCount() > maxNodes) {
    8282                JOptionPane.showMessageDialog(
    8383                        Main.parent,
    8484                        tr("{0} nodes in way {1} exceed the max. allowed number of nodes {2}",
    85                                 ((Way)osmPrimitive).getNodesCount(),
     85                                ((Way) osmPrimitive).getNodesCount(),
    8686                                Long.toString(osmPrimitive.getId()),
    8787                                maxNodes
  • trunk/src/org/openstreetmap/josm/actions/upload/CyclicUploadDependencyException.java

    r8444 r8510  
    3434        sb.append(tr("Cyclic dependency between relations:"))
    3535          .append('[');
    36         for (int i=0; i< cycle.size(); i++) {
     36        for (int i = 0; i < cycle.size(); i++) {
    3737            if (i > 0) {
    3838                sb.append(',');
  • trunk/src/org/openstreetmap/josm/actions/upload/FixDataHook.java

    r8419 r8510  
    4747            @Override
    4848            public boolean fixKeys(Map<String, String> keys, OsmPrimitive osm) {
    49                 if(osm instanceof Relation && "multipolygon".equals(keys.get("type")) && "administrative".equals(keys.get("boundary"))) {
     49                if (osm instanceof Relation && "multipolygon".equals(keys.get("type")) && "administrative".equals(keys.get("boundary"))) {
    5050                    keys.put("type", "boundary");
    5151                    return true;
     
    8181                String k = Tag.removeWhiteSpaces(e.getKey());
    8282                boolean drop = k.isEmpty() || v.isEmpty();
    83                 if(!e.getKey().equals(k)) {
    84                     if(drop || !keys.containsKey(k)) {
     83                if (!e.getKey().equals(k)) {
     84                    if (drop || !keys.containsKey(k)) {
    8585                        newKeys.put(e.getKey(), null);
    86                         if(!drop)
     86                        if (!drop)
    8787                            newKeys.put(k, v);
    8888                    }
    89                 } else if(!e.getValue().equals(v)) {
     89                } else if (!e.getValue().equals(v)) {
    9090                    newKeys.put(k, v.isEmpty() ? null : v);
    9191                } else if (drop) {
     
    124124        @Override
    125125        public boolean fixKeys(Map<String, String> keys, OsmPrimitive osm) {
    126             if(keys.containsKey(oldKey) && !keys.containsKey(newKey)) {
     126            if (keys.containsKey(oldKey) && !keys.containsKey(newKey)) {
    127127                keys.put(newKey, keys.get(oldKey));
    128128                keys.put(oldKey, null);
    129129                return true;
    130             } else if(keys.containsKey(oldKey) && keys.containsKey(newKey) && keys.get(oldKey).equals(keys.get(newKey))) {
     130            } else if (keys.containsKey(oldKey) && keys.containsKey(newKey) && keys.get(oldKey).equals(keys.get(newKey))) {
    131131                keys.put(oldKey, null);
    132132                return true;
     
    166166        @Override
    167167        public boolean fixKeys(Map<String, String> keys, OsmPrimitive osm) {
    168             if(oldValue.equals(keys.get(oldKey)) && (newKey.equals(oldKey)
     168            if (oldValue.equals(keys.get(oldKey)) && (newKey.equals(oldKey)
    169169            || !keys.containsKey(newKey) || keys.get(newKey).equals(newValue))) {
    170170                keys.put(newKey, newValue);
    171                 if(!newKey.equals(oldKey))
     171                if (!newKey.equals(oldKey))
    172172                    keys.put(oldKey, null);
    173173                return true;
     
    183183    @Override
    184184    public boolean checkUpload(APIDataSet apiDataSet) {
    185         if(!Main.pref.getBoolean("fix.data.on.upload", true))
     185        if (!Main.pref.getBoolean("fix.data.on.upload", true))
    186186            return true;
    187187
     
    191191        for (OsmPrimitive osm : objectsToUpload) {
    192192            Map<String, String> keys = new HashMap<>(osm.getKeys());
    193             if(!keys.isEmpty()) {
     193            if (!keys.isEmpty()) {
    194194                boolean modified = false;
    195195                for (FixData fix : deprecated) {
    196                     if(fix.fixKeys(keys, osm))
     196                    if (fix.fixKeys(keys, osm))
    197197                        modified = true;
    198198                }
    199                 if(modified)
     199                if (modified)
    200200                    cmds.add(new ChangePropertyCommand(Collections.singleton(osm), keys));
    201201            }
    202202        }
    203203
    204         if(!cmds.isEmpty())
     204        if (!cmds.isEmpty())
    205205            Main.main.undoRedo.add(new SequenceCommand(tr("Fix deprecated tags"), cmds));
    206206        return true;
  • trunk/src/org/openstreetmap/josm/actions/upload/RelationUploadOrderHook.java

    r6362 r8510  
    4848        model.addColumn(tr("Relation ..."));
    4949        model.addColumn(tr("... refers to relation"));
    50         for (int i=0; i<dep.size()-1;i++) {
     50        for (int i = 0; i < dep.size()-1; i++) {
    5151            Relation r1 = dep.get(i);
    5252            Relation r2 = dep.get(i+1);
    53             model.addRow(new Relation[] {r1,r2});
     53            model.addRow(new Relation[] {r1, r2});
    5454        }
    5555        JTable tbl = new JTable(model);
     
    7070        Relation last = dep.get(dep.size() -1);
    7171        Iterator<Relation> it = dep.iterator();
    72         while(it.hasNext()) {
     72        while (it.hasNext()) {
    7373            if (it.next() != last) {
    7474                it.remove();
     
    9797            apiDataSet.adjustRelationUploadOrder();
    9898            return true;
    99         } catch(CyclicUploadDependencyException e) {
     99        } catch (CyclicUploadDependencyException e) {
    100100            warnCyclicUploadDependency(e);
    101101            return false;
  • trunk/src/org/openstreetmap/josm/actions/upload/UploadNotesTask.java

    r8473 r8510  
    6565            OsmApi api = OsmApi.getOsmApi();
    6666            for (Note note : noteData.getNotes()) {
    67                 if(isCanceled) {
     67                if (isCanceled) {
    6868                    Main.info("Note upload interrupted by user");
    6969                    break;
Note: See TracChangeset for help on using the changeset viewer.