Changeset 10662 in josm for trunk


Ignore:
Timestamp:
2016-07-28T01:00:46+02:00 (8 years ago)
Author:
Don-vip
Message:

see #12472, fix #13230, fix #13225, fix #13228 - disable ReferenceEquality warning + partial revert of r10656 + r10659, causes too much problems

Location:
trunk
Files:
21 edited

Legend:

Unmodified
Added
Removed

  • trunk/build.xml

    r10659 r10662  
    265265            <!-- Undocumented argument to ignore "Sun internal proprietary API" warning, see http://stackoverflow.com/a/13862308/2257172 -->
    266266            <compilerarg value="-XDignore.symbol.file"/>
     267            <compilerarg value="-Xep:ReferenceEquality:OFF"/>
    267268            <compilerarg line="-Xmaxwarns 1000"/>
    268269        </javac>
    … …  
    287288            <!-- Undocumented argument to ignore "Sun internal proprietary API" warning, see http://stackoverflow.com/a/13862308/2257172 -->
    288289            <compilerarg value="-XDignore.symbol.file"/>
     290            <compilerarg value="-Xep:ReferenceEquality:OFF"/>
    289291            <compilerarg line="-Xmaxwarns 1000"/>
    290292        </javac>

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

    r10659 r10662  
    1313import java.util.LinkedList;
    1414import java.util.List;
    15 import java.util.Objects;
    1615import java.util.Set;
    1716
    … …  
    314313        Node lastNode = null;
    315314        Way lastWay = null;
    316         while (!Objects.equals(firstNode, lastNode)) {
     315        while (firstNode != lastNode) {
    317316            if (lastNode == null) lastNode = firstNode;
    318317            for (Way way: ways) {
    319                 if (Objects.equals(way, lastWay)) continue;
    320                 if (Objects.equals(way.firstNode(), lastNode)) {
     318                if (way == lastWay) continue;
     319                if (way.firstNode() == lastNode) {
    321320                    List<Node> wayNodes = way.getNodes();
    322321                    for (int i = 0; i < wayNodes.size() - 1; i++) {
    … …  
    327326                    break;
    328327                }
    329                 if (Objects.equals(way.lastNode(), lastNode)) {
     328                if (way.lastNode() == lastNode) {
    330329                    List<Node> wayNodes = way.getNodes();
    331330                    for (int i = wayNodes.size() - 1; i > 0; i--) {
    … …  
    395394                if (way.isFirstLastNode(node)) continue;
    396395                for (Way wayOther: ways) {
    397                     if (Objects.equals(way, wayOther)) continue;
     396                    if (way == wayOther) continue;
    398397                    if (node.getReferrers().contains(wayOther)) return false;
    399398                }
    … …  
    408407            for (Way w: ways) {
    409408                if (w.isClosed()) return ways.size() == 1;
    410                 if (Objects.equals(w, currentWay)) continue;
     409                if (w == currentWay) continue;
    411410                if (currentWay == null) {
    412411                    nextWay = w;
    … …  
    415414                    break;
    416415                }
    417                 if (Objects.equals(w.firstNode(), endNode)) {
     416                if (w.firstNode() == endNode) {
    418417                    nextWay = w;
    419418                    endNode = w.lastNode();
    420419                    break;
    421420                }
    422                 if (Objects.equals(w.lastNode(), endNode)) {
     421                if (w.lastNode() == endNode) {
    423422                    nextWay = w;
    424423                    endNode = w.firstNode();
    … …  
    429428            used += 1;
    430429            currentWay = nextWay;
    431             if (Objects.equals(endNode, startNode)) return used == ways.size();
     430            if (endNode == startNode) return used == ways.size();
    432431        }
    433432    }

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

    r10659 r10662  
    295295
    296296        public boolean isSuccessorOf(NodePair other) {
    297             return Objects.equals(other.getB(), a);
     297            return other.getB() == a;
    298298        }
    299299
    300300        public boolean isPredecessorOf(NodePair other) {
    301             return Objects.equals(b, other.getA());
     301            return b == other.getA();
    302302        }
    303303
    … …  
    323323         */
    324324        public boolean contains(Node n) {
    325             return Objects.equals(a, n) || Objects.equals(b, n);
     325            return a == n || b == n;
    326326        }
    327327

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

    r10659 r10662  
    1414import java.util.LinkedList;
    1515import java.util.List;
    16 import java.util.Objects;
    1716import java.util.Set;
    1817
    … …  
    244243        } else {
    245244            // Simplify segment
    246             if (simplifiedNodes.isEmpty() || !Objects.equals(simplifiedNodes.get(simplifiedNodes.size()-1), fromN)) {
     245            if (simplifiedNodes.isEmpty() || simplifiedNodes.get(simplifiedNodes.size()-1) != fromN) {
    247246                simplifiedNodes.add(fromN);
    248247            }
    249             if (!Objects.equals(fromN, toN)) {
     248            if (fromN != toN) {
    250249                simplifiedNodes.add(toN);
    251250            }

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

    r10659 r10662  
    1818import java.util.LinkedList;
    1919import java.util.List;
    20 import java.util.Objects;
    2120import java.util.Set;
    2221import java.util.concurrent.atomic.AtomicInteger;
    … …  
    445444
    446445        if (wayChunks.size() < 2) {
    447             if (Objects.equals(wayChunks.get(0).get(0), wayChunks.get(0).get(wayChunks.get(0).size() - 1))) {
     446            if (wayChunks.get(0).get(0) == wayChunks.get(0).get(wayChunks.get(0).size() - 1)) {
    448447                new Notification(
    449448                        tr("You must select two or more nodes to split a circular way."))
    … …  
    570569            List<RelationMember> relationMembers = r.getMembers();
    571570            for (RelationMember rm: relationMembers) {
    572                 if (rm.isWay() && Objects.equals(rm.getMember(), way)) {
     571                if (rm.isWay() && rm.getMember() == way) {
    573572                    boolean insert = true;
    574573                    if ("restriction".equals(type) || "destination_sign".equals(type)) {
    … …  
    631630                            if ((ir - k >= 0) && relationMembers.get(ir - k).isWay()) {
    632631                                Way w = relationMembers.get(ir - k).getWay();
    633                                 if (Objects.equals(w.lastNode(), way.firstNode()) || Objects.equals(w.firstNode(), way.firstNode())) {
     632                                if ((w.lastNode() == way.firstNode()) || w.firstNode() == way.firstNode()) {
    634633                                    backwards = Boolean.FALSE;
    635                                 } else if (Objects.equals(w.firstNode(), way.lastNode()) || Objects.equals(w.lastNode(), way.lastNode())) {
     634                                } else if ((w.firstNode() == way.lastNode()) || w.lastNode() == way.lastNode()) {
    636635                                    backwards = Boolean.TRUE;
    637636                                }
    … …  
    640639                            if ((ir + k < relationMembers.size()) && relationMembers.get(ir + k).isWay()) {
    641640                                Way w = relationMembers.get(ir + k).getWay();
    642                                 if (Objects.equals(w.lastNode(), way.firstNode()) || Objects.equals(w.firstNode(), way.firstNode())) {
     641                                if ((w.lastNode() == way.firstNode()) || w.firstNode() == way.firstNode()) {
    643642                                    backwards = Boolean.TRUE;
    644                                 } else if (Objects.equals(w.firstNode(), way.lastNode()) || Objects.equals(w.lastNode(), way.lastNode())) {
     643                                } else if ((w.firstNode() == way.lastNode()) || w.lastNode() == way.lastNode()) {
    645644                                    backwards = Boolean.FALSE;
    646645                                }

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

    r10659 r10662  
    2828import java.util.List;
    2929import java.util.Map;
    30 import java.util.Objects;
    3130import java.util.Set;
    3231
    … …  
    527526
    528527                // User clicked last node again, finish way
    529                 if (Objects.equals(n0, n)) {
     528                if (n0 == n) {
    530529                    finishDrawing();
    531530                    return;
    … …  
    576575
    577576                // Add new node to way
    578                 if (Objects.equals(way.getNode(way.getNodesCount() - 1), n0)) {
     577                if (way.getNode(way.getNodesCount() - 1) == n0) {
    579578                    way.addNode(n);
    580579                } else {
    … …  
    818817        }
    819818
    820         if (getCurrentBaseNode() == null || getCurrentBaseNode().equals(currentMouseNode))
     819        if (getCurrentBaseNode() == null || getCurrentBaseNode() == currentMouseNode)
    821820            return; // Don't create zero length way segments.
     821
    822822
    823823        double curHdg = Math.toDegrees(getCurrentBaseNode().getEastNorth()
    … …  
    904904    private void continueWayFromNode(Way way, Node node) {
    905905        int n = way.getNodesCount();
    906         if (Objects.equals(node, way.firstNode())) {
     906        if (node == way.firstNode()) {
    907907            currentBaseNode = node;
    908908            if (n > 1) previousNode = way.getNode(1);
    909         } else if (Objects.equals(node, way.lastNode())) {
     909        } else if (node == way.lastNode()) {
    910910            currentBaseNode = node;
    911911            if (n > 1) previousNode = way.getNode(n-2);
    … …  
    944944            Node firstNode = w.getNode(0);
    945945            Node lastNode = w.getNode(w.getNodesCount() - 1);
    946             if ((Objects.equals(firstNode, n) || Objects.equals(lastNode, n)) && !Objects.equals(firstNode, lastNode)) {
     946            if ((firstNode == n || lastNode == n) && (firstNode != lastNode)) {
    947947                if (way != null)
    948948                    return null;

  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r10657 r10662  
    10331033                if (ref instanceof Relation && !ref.isIncomplete() && !ref.isDeleted()) {
    10341034                    for (RelationMember m : ((Relation) ref).getMembers()) {
    1035                         if (osm.equals(m.getMember())) {
     1035                        if (m.getMember() == osm) {
    10361036                            String testRole = m.getRole();
    10371037                            if (role.equals(testRole == null ? "" : testRole))

  • trunk/src/org/openstreetmap/josm/data/conflict/Conflict.java

    r10659 r10662  
    4848
    4949    public boolean isMatchingMy(OsmPrimitive my) {
    50         return Objects.equals(this.my, my);
     50        return this.my == my;
    5151    }
    5252
    5353    public boolean isMatchingTheir(OsmPrimitive their) {
    54         return Objects.equals(this.their, their);
     54        return this.their == their;
    5555    }
    5656

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

    r10659 r10662  
    334334     */
    335335    public String getUniqueId(ImageryInfo info) {
    336         if (info.getId() != null && info.equals(layerIds.get(info.getId()))) {
     336        if (info.getId() != null && layerIds.get(info.getId()) == info) {
    337337            return info.getId();
    338338        }

  • trunk/src/org/openstreetmap/josm/data/osm/Relation.java

    r10657 r10662  
    99import java.util.List;
    1010import java.util.Map;
    11 import java.util.Objects;
    1211import java.util.Set;
    1312
    … …  
    117116            RelationMember originalMember = members[index];
    118117            members[index] = member;
    119             if (!Objects.equals(originalMember.getMember(), member.getMember())) {
     118            if (originalMember.getMember() != member.getMember()) {
    120119                member.getMember().addReferrer(this);
    121120                member.getMember().clearCachedStyle();

  • trunk/src/org/openstreetmap/josm/data/osm/RelationMember.java

    r10656 r10662  
    161161     */
    162162    public boolean refersTo(OsmPrimitive primitive) {
    163         return Objects.equals(member, primitive);
     163        return member == primitive;
    164164    }
    165165

  • trunk/src/org/openstreetmap/josm/data/osm/Way.java

    r10656 r10662  
    99import java.util.List;
    1010import java.util.Map;
    11 import java.util.Objects;
    1211import java.util.Set;
    1312
    … …  
    8988        for (int i = 0; i < count && count > 2;) {
    9089            Node n = nodes.get(i);
    91             if (Objects.equals(last, n)) {
     90            if (last == n) {
    9291                nodes.remove(i);
    9392                --count;
    … …  
    362361        boolean locked = writeLock();
    363362        try {
    364             boolean closed = Objects.equals(lastNode(), n) && Objects.equals(firstNode(), n);
     363            boolean closed = lastNode() == n && firstNode() == n;
    365364            int i;
    366365            List<Node> copy = getNodes();
    … …  
    371370            if (closed && i > 2) {
    372371                copy.add(copy.get(0));
    373             } else if (i >= 2 && i <= 3 && Objects.equals(copy.get(0), copy.get(i-1))) {
     372            } else if (i >= 2 && i <= 3 && copy.get(0) == copy.get(i-1)) {
    374373                copy.remove(i-1);
    375374            }
    … …  
    402401            if (closed && i > 2) {
    403402                copy.add(copy.get(0));
    404             } else if (i >= 2 && i <= 3 && Objects.equals(copy.get(0), copy.get(i-1))) {
     403            } else if (i >= 2 && i <= 3 && copy.get(0) == copy.get(i-1)) {
    405404                copy.remove(i-1);
    406405            }
    … …  
    495494
    496495        Node[] nodes = this.nodes;
    497         return nodes.length >= 3 && Objects.equals(nodes[nodes.length-1], nodes[0]);
     496        return nodes.length >= 3 && nodes[nodes.length-1] == nodes[0];
    498497    }
    499498
    … …  
    508507            Node distinctNode = null;
    509508            for (int i = 1; i < nodes.length-1; i++) {
    510                 if (distinctNode == null && !Objects.equals(nodes[i], nodes[0])) {
     509                if (distinctNode == null && nodes[i] != nodes[0]) {
    511510                    distinctNode = nodes[i];
    512                 } else if (distinctNode != null && !Objects.equals(nodes[i], nodes[0]) && !Objects.equals(nodes[i], distinctNode)) {
     511                } else if (distinctNode != null && nodes[i] != nodes[0] && nodes[i] != distinctNode) {
    513512                    return true;
    514513                }
    … …  
    551550        Node[] nodes = this.nodes;
    552551        if (isIncomplete() || nodes.length == 0) return false;
    553         return Objects.equals(n, nodes[0]) || Objects.equals(n, nodes[nodes.length -1]);
     552        return n == nodes[0] || n == nodes[nodes.length -1];
    554553    }
    555554
    … …  
    564563        if (isIncomplete() || nodes.length <= 2) return false;
    565564        /* circular ways have only inner nodes, so return true for them! */
    566         if (Objects.equals(n, nodes[0]) && Objects.equals(n, nodes[nodes.length-1])) return true;
     565        if (n == nodes[0] && n == nodes[nodes.length-1]) return true;
    567566        for (int i = 1; i < nodes.length - 1; ++i) {
    568             if (Objects.equals(nodes[i], n)) return true;
     567            if (nodes[i] == n) return true;
    569568        }
    570569        return false;

  • trunk/src/org/openstreetmap/josm/data/osm/history/HistoryWay.java

    r10656 r10662  
    88import java.util.Date;
    99import java.util.List;
    10 import java.util.Objects;
    1110
    1211import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    … …  
    138137     */
    139138    public boolean isClosed() {
    140         return getNumNodes() >= 3 && Objects.equals(nodeIds.get(0), nodeIds.get(nodeIds.size()-1));
     139        return getNumNodes() >= 3 && nodeIds.get(0) == nodeIds.get(nodeIds.size()-1);
    141140    }
    142141

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

    r10656 r10662  
    33
    44import java.awt.Color;
    5 import java.util.Objects;
    65
    76import org.openstreetmap.josm.Main;
    … …  
    8483        taggedColor = PaintColors.TAGGED.get();
    8584        connectionColor = PaintColors.CONNECTION.get();
    86         if (!Objects.equals(taggedColor, nodeColor)) {
     85        if (taggedColor != nodeColor) {
    8786            taggedConnectionColor = taggedColor;
    8887        } else {

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

    r10656 r10662  
    3434import java.util.Map;
    3535import java.util.NoSuchElementException;
    36 import java.util.Objects;
    3736import java.util.concurrent.ForkJoinPool;
    3837import java.util.concurrent.ForkJoinTask;
    … …  
    11471146        /* find the "direct" nodes before the via node */
    11481147        Node fromNode;
    1149         if (Objects.equals(fromWay.firstNode(), via)) {
     1148        if (fromWay.firstNode() == via) {
    11501149            fromNode = fromWay.getNode(1);
    11511150        } else {
    … …  
    14421441            GeneralPath highlightSegs = null;
    14431442            for (WaySegment ws : highlightWaySegments) {
    1444                 if (!Objects.equals(ws.way, way) || ws.lowerIndex < offset) {
     1443                if (ws.way != way || ws.lowerIndex < offset) {
    14451444                    continue;
    14461445                }

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

    r10656 r10662  
    1111import java.util.Iterator;
    1212import java.util.List;
    13 import java.util.Objects;
    1413import java.util.Set;
    1514
    … …  
    299298                }
    300299            }
    301             if (nodes.size() >= 3 && Objects.equals(nodes.get(0), nodes.get(nodes.size() - 1))) {
     300            if (nodes.size() >= 3 && nodes.get(0) == nodes.get(nodes.size() - 1)) {
    302301                poly.closePath();
    303302            }
    … …  
    549548                            if (nodes == null) {
    550549                                nl = w.getNodesCount()-1;
    551                                 if (Objects.equals(w.getNode(nl), c.getNode(0))) {
     550                                if (w.getNode(nl) == c.getNode(0)) {
    552551                                    mode = 21;
    553                                 } else if (Objects.equals(w.getNode(nl), c.getNode(cl))) {
     552                                } else if (w.getNode(nl) == c.getNode(cl)) {
    554553                                    mode = 22;
    555                                 } else if (Objects.equals(w.getNode(0), c.getNode(0))) {
     554                                } else if (w.getNode(0) == c.getNode(0)) {
    556555                                    mode = 11;
    557                                 } else if (Objects.equals(w.getNode(0), c.getNode(cl))) {
     556                                } else if (w.getNode(0) == c.getNode(cl)) {
    558557                                    mode = 12;
    559558                                }
    560559                            } else {
    561560                                nl = nodes.size()-1;
    562                                 if (Objects.equals(nodes.get(nl), c.getNode(0))) {
     561                                if (nodes.get(nl) == c.getNode(0)) {
    563562                                    mode = 21;
    564                                 } else if (Objects.equals(nodes.get(0), c.getNode(cl))) {
     563                                } else if (nodes.get(0) == c.getNode(cl)) {
    565564                                    mode = 12;
    566                                 } else if (Objects.equals(nodes.get(0), c.getNode(0))) {
     565                                } else if (nodes.get(0) == c.getNode(0)) {
    567566                                    mode = 11;
    568                                 } else if (Objects.equals(nodes.get(nl), c.getNode(cl))) {
     567                                } else if (nodes.get(nl) == c.getNode(cl)) {
    569568                                    mode = 22;
    570569                                }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesDialog.java

    r10659 r10662  
    692692                    int i = 1;
    693693                    for (RelationMember m : r.getMembers()) {
    694                         if (primitive.equals(m.getMember())) {
     694                        if (m.getMember() == primitive) {
    695695                            mi.add(m, i);
    696696                        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/TagEditHelper.java

    r10659 r10662  
    678678                for (int i = 0; i < tagData.getRowCount(); ++i) {
    679679                    if (item.getValue().equals(tagData.getValueAt(i, 0) /* sic! do not use getDataKey*/)) {
    680                         if (item.equals(itemToSelect)) {
     680                        if (itemToSelect == item) {
    681681                            itemToSelect = null;
    682682                        }

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

    r10659 r10662  
    2727import java.util.List;
    2828import java.util.Map;
    29 import java.util.Objects;
    3029import java.util.concurrent.ConcurrentHashMap;
    3130
    … …  
    660659            @Override
    661660            public boolean isDataFlavorSupported(DataFlavor flavor) {
    662                 return Objects.equals(flavors[0], flavor);
     661                return flavors[0] == flavor;
    663662            }
    664663        }

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

    r10659 r10662  
    77import java.io.IOException;
    88import java.net.URL;
    9 import java.util.Objects;
    109
    1110import javax.sound.sampled.AudioFormat;
    … …  
    298297                            double offset = command.offset();
    299298                            speed = command.speed();
    300                             if (!Objects.equals(playingUrl, command.url()) ||
     299                            if (playingUrl != command.url() ||
    301300                                    stateChange != State.PAUSED ||
    302301                                    offset != 0) {

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

    r10658 r10662  
    1515import java.util.LinkedHashSet;
    1616import java.util.List;
    17 import java.util.Objects;
    1817import java.util.Set;
    1918
    … …  
    116115                        int commonCount = 0;
    117116                        //test if we have common nodes to add.
    118                         if (Objects.equals(seg1Node1, seg2Node1) || Objects.equals(seg1Node1, seg2Node2)) {
     117                        if (seg1Node1 == seg2Node1 || seg1Node1 == seg2Node2) {
    119118                            commonCount++;
    120119
    … …  
    128127                        }
    129128
    130                         if (Objects.equals(seg1Node2, seg2Node1) || Objects.equals(seg1Node2, seg2Node2)) {
     129                        if (seg1Node2 == seg2Node1 || seg1Node2 == seg2Node2) {
    131130                            commonCount++;
    132131
    … …  
    190189                                intersectionNodes.add(intNode);
    191190
    192                                 if (Objects.equals(intNode, newNode)) {
     191                                if (intNode == newNode) {
    193192                                    cmds.add(new AddCommand(intNode));
    194193                                }
    … …  
    703702    public static boolean isClockwise(List<Node> nodes) {
    704703        int nodesCount = nodes.size();
    705         if (nodesCount < 3 || !Objects.equals(nodes.get(0), nodes.get(nodesCount - 1))) {
     704        if (nodesCount < 3 || nodes.get(0) != nodes.get(nodesCount - 1)) {
    706705            throw new IllegalArgumentException("Way must be closed to check orientation.");
    707706        }
    … …  
    992991        double perimeter = 0;
    993992        if (!nodes.isEmpty()) {
    994             boolean closed = Objects.equals(nodes.get(0), nodes.get(nodes.size() - 1));
     993            boolean closed = nodes.get(0) == nodes.get(nodes.size() - 1);
    995994            int numSegments = closed ? nodes.size() - 1 : nodes.size();
    996995            EastNorth p1 = projection == null ? nodes.get(0).getEastNorth() : projection.latlon2eastNorth(nodes.get(0).getCoor());
Note: See TracChangeset for help on using the changeset viewer.