Changeset 7885 in josm


Ignore:
Timestamp:
2014-12-25T16:38:00+01:00 (5 years ago)
Author:
Don-vip
Message:

see #9844 - code cleanup

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java

    r7884 r7885  
    1515import org.openstreetmap.josm.command.DeleteCommand;
    1616import org.openstreetmap.josm.data.osm.OsmPrimitive;
    17 import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    1817import org.openstreetmap.josm.data.osm.Relation;
    1918import org.openstreetmap.josm.data.osm.RelationMember;
     
    168167    private boolean checkMemberType(Role r, RelationMember member) {
    169168        if (r.types != null) {
    170             if (member.getDisplayType().equals(OsmPrimitiveType.NODE)) {
     169            switch (member.getDisplayType()) {
     170            case NODE:
    171171                return r.types.contains(TaggingPresetType.NODE);
    172             }
    173             if (member.getDisplayType().equals(OsmPrimitiveType.CLOSEDWAY)) {
     172            case CLOSEDWAY:
    174173                return r.types.contains(TaggingPresetType.CLOSEDWAY);
    175             }
    176             if (member.getDisplayType().equals(OsmPrimitiveType.WAY)) {
     174            case WAY:
    177175                return r.types.contains(TaggingPresetType.WAY);
    178             }
    179             if (member.getDisplayType().equals(OsmPrimitiveType.RELATION)) {
     176            case RELATION:
    180177                return r.types.contains(TaggingPresetType.RELATION);
    181             }
    182             // not matching type
    183             return false;
     178            default: // not matching type
     179                return false;
     180            }
    184181        } else {
    185182            // if no types specified, then test is passed
Note: See TracChangeset for help on using the changeset viewer.