Changeset 10656 in josm for trunk


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

see #12472 - fix warning "ReferenceEquality"

Location:
trunk/src/org/openstreetmap/josm
Files:
10 edited

Legend:

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

    r10626 r10656  
    10331033                if (ref instanceof Relation && !ref.isIncomplete() && !ref.isDeleted()) {
    10341034                    for (RelationMember m : ((Relation) ref).getMembers()) {
    1035                         if (m.getMember() == osm) {
     1035                        if (osm.equals(m.getMember())) {
    10361036                            String testRole = m.getRole();
    10371037                            if (role.equals(testRole == null ? "" : testRole))
  • trunk/src/org/openstreetmap/josm/data/osm/OsmPrimitive.java

    r10611 r10656  
    1717import java.util.Locale;
    1818import java.util.Map;
     19import java.util.Objects;
    1920import java.util.Set;
    2021
     
    10041005    /**
    10051006     * Add new referrer. If referrer is already included then no action is taken
    1006      * @param referrer The referrer to add
     1007     * @param referrer The referrer to add. Must not be null
     1008     * @throws NullPointerException if {@code referrer} is null
    10071009     */
    10081010    protected void addReferrer(OsmPrimitive referrer) {
     
    10101012            referrers = referrer;
    10111013        } else if (referrers instanceof OsmPrimitive) {
    1012             if (referrers != referrer) {
     1014            if (!referrer.equals(referrers)) {
    10131015                referrers = new OsmPrimitive[] {(OsmPrimitive) referrers, referrer};
    10141016            }
    10151017        } else {
    10161018            for (OsmPrimitive primitive:(OsmPrimitive[]) referrers) {
    1017                 if (primitive == referrer)
     1019                if (referrer.equals(primitive))
    10181020                    return;
    10191021            }
     
    10241026    /**
    10251027     * Remove referrer. No action is taken if referrer is not registered
    1026      * @param referrer The referrer to remove
     1028     * @param referrer The referrer to remove. Must not be null
     1029     * @throws NullPointerException if {@code referrer} is null
    10271030     */
    10281031    protected void removeReferrer(OsmPrimitive referrer) {
    10291032        if (referrers instanceof OsmPrimitive) {
    1030             if (referrers == referrer) {
     1033            if (referrer.equals(referrers)) {
    10311034                referrers = null;
    10321035            }
     
    10351038            int idx = -1;
    10361039            for (int i = 0; i < orig.length; i++) {
    1037                 if (orig[i] == referrer) {
     1040                if (referrer.equals(orig[i])) {
    10381041                    idx = i;
    10391042                    break;
     
    12631266                && version == other.version
    12641267                && isVisible() == other.isVisible()
    1265                 && (user == null ? other.user == null : user == other.user)
     1268                && Objects.equals(user, other.user)
    12661269                && changesetId == other.changesetId;
    12671270    }
  • trunk/src/org/openstreetmap/josm/data/osm/Relation.java

    r10632 r10656  
    99import java.util.List;
    1010import java.util.Map;
     11import java.util.Objects;
    1112import java.util.Set;
    1213
     
    115116            RelationMember originalMember = members[index];
    116117            members[index] = member;
    117             if (originalMember.getMember() != member.getMember()) {
     118            if (!Objects.equals(originalMember.getMember(), member.getMember())) {
    118119                member.getMember().addReferrer(this);
    119120                member.getMember().clearCachedStyle();
  • trunk/src/org/openstreetmap/josm/data/osm/RelationMember.java

    r9371 r10656  
    161161     */
    162162    public boolean refersTo(OsmPrimitive primitive) {
    163         return member == primitive;
     163        return Objects.equals(member, primitive);
    164164    }
    165165
     
    175175        RelationMember that = (RelationMember) obj;
    176176        return Objects.equals(role, that.role) &&
    177                 Objects.equals(member, that.member);
     177               Objects.equals(member, that.member);
    178178    }
    179179
  • trunk/src/org/openstreetmap/josm/data/osm/Way.java

    r10632 r10656  
    99import java.util.List;
    1010import java.util.Map;
     11import java.util.Objects;
    1112import java.util.Set;
    1213
     
    8889        for (int i = 0; i < count && count > 2;) {
    8990            Node n = nodes.get(i);
    90             if (last == n) {
     91            if (Objects.equals(last, n)) {
    9192                nodes.remove(i);
    9293                --count;
     
    361362        boolean locked = writeLock();
    362363        try {
    363             boolean closed = lastNode() == n && firstNode() == n;
     364            boolean closed = Objects.equals(lastNode(), n) && Objects.equals(firstNode(), n);
    364365            int i;
    365366            List<Node> copy = getNodes();
     
    370371            if (closed && i > 2) {
    371372                copy.add(copy.get(0));
    372             } else if (i >= 2 && i <= 3 && copy.get(0) == copy.get(i-1)) {
     373            } else if (i >= 2 && i <= 3 && Objects.equals(copy.get(0), copy.get(i-1))) {
    373374                copy.remove(i-1);
    374375            }
     
    401402            if (closed && i > 2) {
    402403                copy.add(copy.get(0));
    403             } else if (i >= 2 && i <= 3 && copy.get(0) == copy.get(i-1)) {
     404            } else if (i >= 2 && i <= 3 && Objects.equals(copy.get(0), copy.get(i-1))) {
    404405                copy.remove(i-1);
    405406            }
     
    494495
    495496        Node[] nodes = this.nodes;
    496         return nodes.length >= 3 && nodes[nodes.length-1] == nodes[0];
     497        return nodes.length >= 3 && Objects.equals(nodes[nodes.length-1], nodes[0]);
    497498    }
    498499
     
    507508            Node distinctNode = null;
    508509            for (int i = 1; i < nodes.length-1; i++) {
    509                 if (distinctNode == null && nodes[i] != nodes[0]) {
     510                if (distinctNode == null && !Objects.equals(nodes[i], nodes[0])) {
    510511                    distinctNode = nodes[i];
    511                 } else if (distinctNode != null && nodes[i] != nodes[0] && nodes[i] != distinctNode) {
     512                } else if (distinctNode != null && !Objects.equals(nodes[i], nodes[0]) && !Objects.equals(nodes[i], distinctNode)) {
    512513                    return true;
    513514                }
     
    550551        Node[] nodes = this.nodes;
    551552        if (isIncomplete() || nodes.length == 0) return false;
    552         return n == nodes[0] || n == nodes[nodes.length -1];
     553        return Objects.equals(n, nodes[0]) || Objects.equals(n, nodes[nodes.length -1]);
    553554    }
    554555
     
    563564        if (isIncomplete() || nodes.length <= 2) return false;
    564565        /* circular ways have only inner nodes, so return true for them! */
    565         if (n == nodes[0] && n == nodes[nodes.length-1]) return true;
     566        if (Objects.equals(n, nodes[0]) && Objects.equals(n, nodes[nodes.length-1])) return true;
    566567        for (int i = 1; i < nodes.length - 1; ++i) {
    567             if (nodes[i] == n) return true;
     568            if (Objects.equals(nodes[i], n)) return true;
    568569        }
    569570        return false;
  • trunk/src/org/openstreetmap/josm/data/osm/history/HistoryWay.java

    r10632 r10656  
    88import java.util.Date;
    99import java.util.List;
     10import java.util.Objects;
    1011
    1112import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
     
    137138     */
    138139    public boolean isClosed() {
    139         return getNumNodes() >= 3 && nodeIds.get(0) == nodeIds.get(nodeIds.size()-1);
     140        return getNumNodes() >= 3 && Objects.equals(nodeIds.get(0), nodeIds.get(nodeIds.size()-1));
    140141    }
    141142
  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/MapPaintSettings.java

    r10655 r10656  
    33
    44import java.awt.Color;
     5import java.util.Objects;
    56
    67import org.openstreetmap.josm.Main;
     
    8384        taggedColor = PaintColors.TAGGED.get();
    8485        connectionColor = PaintColors.CONNECTION.get();
    85         if (taggedColor != nodeColor) {
     86        if (!Objects.equals(taggedColor, nodeColor)) {
    8687            taggedConnectionColor = taggedColor;
    8788        } else {
  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java

    r10626 r10656  
    3434import java.util.Map;
    3535import java.util.NoSuchElementException;
     36import java.util.Objects;
    3637import java.util.concurrent.ForkJoinPool;
    3738import java.util.concurrent.ForkJoinTask;
     
    226227
    227228            // simple node on top of icons and shapes
    228             if (this.style == NodeElement.SIMPLE_NODE_ELEMSTYLE && other.style != NodeElement.SIMPLE_NODE_ELEMSTYLE)
     229            if (NodeElement.SIMPLE_NODE_ELEMSTYLE.equals(this.style) && !NodeElement.SIMPLE_NODE_ELEMSTYLE.equals(other.style))
    229230                return 1;
    230             if (this.style != NodeElement.SIMPLE_NODE_ELEMSTYLE && other.style == NodeElement.SIMPLE_NODE_ELEMSTYLE)
     231            if (!NodeElement.SIMPLE_NODE_ELEMSTYLE.equals(this.style) && NodeElement.SIMPLE_NODE_ELEMSTYLE.equals(other.style))
    231232                return -1;
    232233
     
    11461147        /* find the "direct" nodes before the via node */
    11471148        Node fromNode;
    1148         if (fromWay.firstNode() == via) {
     1149        if (Objects.equals(fromWay.firstNode(), via)) {
    11491150            fromNode = fromWay.getNode(1);
    11501151        } else {
     
    14411442            GeneralPath highlightSegs = null;
    14421443            for (WaySegment ws : highlightWaySegments) {
    1443                 if (ws.way != way || ws.lowerIndex < offset) {
     1444                if (!Objects.equals(ws.way, way) || ws.lowerIndex < offset) {
    14441445                    continue;
    14451446                }
  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/relations/Multipolygon.java

    r10378 r10656  
    1111import java.util.Iterator;
    1212import java.util.List;
     13import java.util.Objects;
    1314import java.util.Set;
    1415
     
    298299                }
    299300            }
    300             if (nodes.size() >= 3 && nodes.get(0) == nodes.get(nodes.size() - 1)) {
     301            if (nodes.size() >= 3 && Objects.equals(nodes.get(0), nodes.get(nodes.size() - 1))) {
    301302                poly.closePath();
    302303            }
     
    548549                            if (nodes == null) {
    549550                                nl = w.getNodesCount()-1;
    550                                 if (w.getNode(nl) == c.getNode(0)) {
     551                                if (Objects.equals(w.getNode(nl), c.getNode(0))) {
    551552                                    mode = 21;
    552                                 } else if (w.getNode(nl) == c.getNode(cl)) {
     553                                } else if (Objects.equals(w.getNode(nl), c.getNode(cl))) {
    553554                                    mode = 22;
    554                                 } else if (w.getNode(0) == c.getNode(0)) {
     555                                } else if (Objects.equals(w.getNode(0), c.getNode(0))) {
    555556                                    mode = 11;
    556                                 } else if (w.getNode(0) == c.getNode(cl)) {
     557                                } else if (Objects.equals(w.getNode(0), c.getNode(cl))) {
    557558                                    mode = 12;
    558559                                }
    559560                            } else {
    560561                                nl = nodes.size()-1;
    561                                 if (nodes.get(nl) == c.getNode(0)) {
     562                                if (Objects.equals(nodes.get(nl), c.getNode(0))) {
    562563                                    mode = 21;
    563                                 } else if (nodes.get(0) == c.getNode(cl)) {
     564                                } else if (Objects.equals(nodes.get(0), c.getNode(cl))) {
    564565                                    mode = 12;
    565                                 } else if (nodes.get(0) == c.getNode(0)) {
     566                                } else if (Objects.equals(nodes.get(0), c.getNode(0))) {
    566567                                    mode = 11;
    567                                 } else if (nodes.get(nl) == c.getNode(cl)) {
     568                                } else if (Objects.equals(nodes.get(nl), c.getNode(cl))) {
    568569                                    mode = 22;
    569570                                }
  • trunk/src/org/openstreetmap/josm/tools/Geometry.java

    r10627 r10656  
    1515import java.util.LinkedHashSet;
    1616import java.util.List;
     17import java.util.Objects;
    1718import java.util.Set;
    1819
     
    115116                        int commonCount = 0;
    116117                        //test if we have common nodes to add.
    117                         if (seg1Node1 == seg2Node1 || seg1Node1 == seg2Node2) {
     118                        if (Objects.equals(seg1Node1, seg2Node1) || Objects.equals(seg1Node1, seg2Node2)) {
    118119                            commonCount++;
    119120
     
    127128                        }
    128129
    129                         if (seg1Node2 == seg2Node1 || seg1Node2 == seg2Node2) {
     130                        if (Objects.equals(seg1Node2, seg2Node1) || Objects.equals(seg1Node2, seg2Node2)) {
    130131                            commonCount++;
    131132
     
    189190                                intersectionNodes.add(intNode);
    190191
    191                                 if (intNode == newNode) {
     192                                if (Objects.equals(intNode, newNode)) {
    192193                                    cmds.add(new AddCommand(intNode));
    193194                                }
     
    702703    public static boolean isClockwise(List<Node> nodes) {
    703704        int nodesCount = nodes.size();
    704         if (nodesCount < 3 || nodes.get(0) != nodes.get(nodesCount - 1)) {
     705        if (nodesCount < 3 || !Objects.equals(nodes.get(0), nodes.get(nodesCount - 1))) {
    705706            throw new IllegalArgumentException("Way must be closed to check orientation.");
    706707        }
     
    990991        double perimeter = 0;
    991992        if (!nodes.isEmpty()) {
    992             boolean closed = nodes.get(0) == nodes.get(nodes.size() - 1);
     993            boolean closed = Objects.equals(nodes.get(0), nodes.get(nodes.size() - 1));
    993994            int numSegments = closed ? nodes.size() - 1 : nodes.size();
    994995            EastNorth p1 = projection == null ? nodes.get(0).getEastNorth() : projection.latlon2eastNorth(nodes.get(0).getCoor());
Note: See TracChangeset for help on using the changeset viewer.