Changeset 8614 in josm for trunk/src/org


Ignore:
Timestamp:
2015-07-23T12:14:43+02:00 (5 years ago)
Author:
bastiK
Message:

applied #11708 - Removed redundant code from StyledMapRenderer (patch by michael2402)

File:
1 edited

Legend:

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

    r8510 r8614  
    14681468    }
    14691469
     1470    /**
     1471     * Computes the flags for a given OSM primitive.
     1472     * @param primitive The primititve to compute the flags for.
     1473     * @param checkOuterMember <code>true</code> if we should also add {@link #FLAG_OUTERMEMBER_OF_SELECTED}
     1474     * @return The flag.
     1475     */
     1476    public static int computeFlags(OsmPrimitive primitive, boolean checkOuterMember) {
     1477        if (primitive.isDisabled()) {
     1478            return FLAG_DISABLED;
     1479        } else if (primitive.isSelected()) {
     1480            return FLAG_SELECTED;
     1481        } else if (checkOuterMember && primitive.isOuterMemberOfSelected()) {
     1482            return FLAG_OUTERMEMBER_OF_SELECTED;
     1483        } else if (primitive.isMemberOfSelected()) {
     1484            return FLAG_MEMBER_OF_SELECTED;
     1485        } else {
     1486            return FLAG_NORMAL;
     1487        }
     1488    }
     1489
    14701490    private class ComputeStyleListWorker implements Callable<List<StyleRecord>>, Visitor {
    14711491        private final List<? extends OsmPrimitive> input;
     
    15131533        @Override
    15141534        public void visit(Node n) {
    1515             if (n.isDisabled()) {
    1516                 add(n, FLAG_DISABLED);
    1517             } else if (n.isSelected()) {
    1518                 add(n, FLAG_SELECTED);
    1519             } else if (n.isMemberOfSelected()) {
    1520                 add(n, FLAG_MEMBER_OF_SELECTED);
    1521             } else {
    1522                 add(n, FLAG_NORMAL);
    1523             }
     1535            add(n, computeFlags(n, false));
    15241536        }
    15251537
    15261538        @Override
    15271539        public void visit(Way w) {
    1528             if (w.isDisabled()) {
    1529                 add(w, FLAG_DISABLED);
    1530             } else if (w.isSelected()) {
    1531                 add(w, FLAG_SELECTED);
    1532             } else if (w.isOuterMemberOfSelected()) {
    1533                 add(w, FLAG_OUTERMEMBER_OF_SELECTED);
    1534             } else if (w.isMemberOfSelected()) {
    1535                 add(w, FLAG_MEMBER_OF_SELECTED);
    1536             } else {
    1537                 add(w, FLAG_NORMAL);
    1538             }
     1540            add(w, computeFlags(w, true));
    15391541        }
    15401542
    15411543        @Override
    15421544        public void visit(Relation r) {
    1543             if (r.isDisabled()) {
    1544                 add(r, FLAG_DISABLED);
    1545             } else if (r.isSelected()) {
    1546                 add(r, FLAG_SELECTED);
    1547             } else if (r.isOuterMemberOfSelected()) {
    1548                 add(r, FLAG_OUTERMEMBER_OF_SELECTED);
    1549             } else if (r.isMemberOfSelected()) {
    1550                 add(r, FLAG_MEMBER_OF_SELECTED);
    1551             } else {
    1552                 add(r, FLAG_NORMAL);
    1553             }
     1545            add(r, computeFlags(r, true));
    15541546        }
    15551547
     
    16311623        BBox bbox = bounds.toBBox();
    16321624        getSettings(renderVirtualNodes);
     1625        boolean benchmark = Main.isTraceEnabled() || Main.pref.getBoolean("mappaint.render.benchmark", false);
    16331626
    16341627        data.getReadLock().lock();
     
    16371630
    16381631            long timeStart = 0, timePhase1 = 0, timeFinished;
    1639             if (Main.isTraceEnabled()) {
     1632            if (benchmark) {
    16401633                timeStart = System.currentTimeMillis();
    16411634                System.err.print("BENCHMARK: rendering ");
     
    16581651            helper.process(new CompositeList<>(nodes, ways));
    16591652
    1660             if (Main.isTraceEnabled()) {
     1653            if (benchmark) {
    16611654                timePhase1 = System.currentTimeMillis();
    16621655                System.err.print("phase 1 (calculate styles): " + Utils.getDurationString(timePhase1 - timeStart));
     
    16761669            }
    16771670
    1678             if (Main.isTraceEnabled()) {
     1671            if (benchmark) {
    16791672                timeFinished = System.currentTimeMillis();
    16801673                System.err.println("; phase 2 (draw): " + Utils.getDurationString(timeFinished - timePhase1) +
Note: See TracChangeset for help on using the changeset viewer.