Ignore:
Timestamp:
2016-01-09T23:20:37+01:00 (4 years ago)
Author:
simon04
Message:

Java 7: use Objects.equals and Objects.hash

Location:
trunk/src/org/openstreetmap/josm/tools
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/tools/MultiMap.java

    r9074 r9371  
    99import java.util.Map;
    1010import java.util.Map.Entry;
     11import java.util.Objects;
    1112import java.util.Set;
    1213
     
    214215    @Override
    215216    public int hashCode() {
    216         return map.hashCode();
     217        return Objects.hash(map);
    217218    }
    218219
    219220    @Override
    220221    public boolean equals(Object obj) {
    221         if (this == obj)
    222             return true;
    223         if (obj == null)
    224             return false;
    225         if (!(obj instanceof MultiMap))
    226             return false;
    227         return map.equals(((MultiMap<?, ?>) obj).map);
     222        if (this == obj) return true;
     223        if (obj == null || getClass() != obj.getClass()) return false;
     224        MultiMap<?, ?> multiMap = (MultiMap<?, ?>) obj;
     225        return Objects.equals(map, multiMap.map);
    228226    }
    229227
    230228    @Override
     229
    231230    public String toString() {
    232231        List<String> entries = new ArrayList<>(map.size());
  • trunk/src/org/openstreetmap/josm/tools/Pair.java

    r9246 r9371  
    33import java.util.ArrayList;
    44import java.util.List;
     5import java.util.Objects;
    56
    67/**
     
    3435    @Override
    3536    public int hashCode() {
    36         return a.hashCode() + b.hashCode();
     37        return Objects.hash(a, b);
    3738    }
    3839
    3940    @Override
    4041    public boolean equals(Object other) {
    41         if (other instanceof Pair<?, ?>) {
    42             Pair<?, ?> o = (Pair<?, ?>) other;
    43             return a.equals(o.a) && b.equals(o.b);
    44         } else
    45             return false;
     42        if (this == other) return true;
     43        if (other == null || getClass() != other.getClass()) return false;
     44        Pair<?, ?> pair = (Pair<?, ?>) other;
     45        return Objects.equals(a, pair.a) &&
     46                Objects.equals(b, pair.b);
    4647    }
    4748
Note: See TracChangeset for help on using the changeset viewer.