Ignore:
Timestamp:
2016-07-28T01:00:46+02:00 (3 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/src/org/openstreetmap/josm/actions
Files:
6 edited

Legend:

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