Changeset 2762 in josm for trunk


Ignore:
Timestamp:
2010-01-07T21:25:13+01:00 (10 years ago)
Author:
jttt
Message:

Remove error checks form MapPaintVisitor that are already done in validator

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/MapPaintVisitor.java

    r2706 r2762  
    356356        for (RelationMember m : r.getMembers())
    357357        {
    358             if (m.getMember().isDeleted()) {
    359                 putError(r, tr("Deleted member ''{0}'' in relation.",
    360                         m.getMember().getDisplayName(DefaultNameFormatter.getInstance())), true);
    361             } else if(m.getMember().isIncomplete())
     358            if(m.getMember().isIncomplete())
    362359                return;
    363360            else
     
    366363                {
    367364                    Way w = m.getWay();
    368                     if(w.getNodesCount() < 2)
    369                     {
    370                         putError(r, tr("Way ''{0}'' with less than two points.",
    371                                 w.getDisplayName(DefaultNameFormatter.getInstance())), true);
    372                     }
    373                     else if("from".equals(m.getRole())) {
     365                    if(w.getNodesCount() < 2) {
     366                        continue;
     367                    }
     368
     369                    if("from".equals(m.getRole())) {
    374370                        if(fromWay != null) {
    375371                            putError(r, tr("More than one \"from\" way found."), true);
     
    671667        // Fill inner and outer list with valid ways
    672668        for (RelationMember m : r.getMembers()) {
    673             if (m.getMember().isDeleted()) {
    674                 putError(r, tr("Deleted member ''{0}'' in relation.",
    675                         m.getMember().getDisplayName(DefaultNameFormatter.getInstance())), true);
    676             } else if (m.getMember().isIncomplete()) {
     669            if (m.getMember().isIncomplete()) {
    677670                incomplete = true;
    678671            } else if(m.getMember().isDrawable()) {
     
    680673                    Way w = m.getWay();
    681674                    if(w.getNodesCount() < 2) {
    682                         putError(r, tr("Way ''{0}'' with less than two points.",
    683                                 w.getDisplayName(DefaultNameFormatter.getInstance())), true);
    684                     }
    685                     else if("inner".equals(m.getRole())) {
     675                        continue;
     676                    }
     677
     678                    if("inner".equals(m.getRole())) {
    686679                        inner.add(w);
    687680                    } else if("outer".equals(m.getRole())) {
Note: See TracChangeset for help on using the changeset viewer.