Changeset 1313 in josm for trunk/src/org/openstreetmap/josm/data

Timestamp:

2009-01-20T18:26:59+01:00 (15 years ago)

Author:

stoecker

Message:

fix #2047

Location:

trunk/src/org/openstreetmap/josm/data/osm

Files:

• OsmPrimitive.java (modified) (2 diffs)
• Way.java (modified) (2 diffs)
• visitor/MapPaintVisitor.java (modified) (14 diffs)

trunk/src/org/openstreetmap/josm/data/osm/OsmPrimitive.java

@@ -56 +56 @@
         errors = null;
     }
+    /* This should not be called from outside. Fixing the UI to add relevant
+       get/set functions calling this implecitely is prefered, so we can have
+       transparent cache handling in the future. */
+    protected void clearCached()
+    {
+        mappaintVisibleCode = 0;
+        mappaintDrawnCode = 0;
+        mappaintStyle = null;
+    }
     /* end of mappaint data */
 
@@ -279 +288 @@
         tagged = osm.tagged;
         incomplete = osm.incomplete;
-        mappaintStyle = null;
+        clearCached();
+        clearErrors();
     }
 

trunk/src/org/openstreetmap/josm/data/osm/Way.java

@@ -30 +30 @@
     public Integer mappaintDrawnAreaCode = 0;
     /* end of mappaint data */
-    
+    @Override protected void clearCached()
+    {
+        super.clearCached();
+        isMappaintArea = false;
+        mappaintDrawnAreaCode = 0;
+    }
+
     public void visitNodes(Visitor v) {
         for (Node n : this.nodes)
@@ -126 +132 @@
     }
 
+    public void addNode(Node n)
+    {
+        clearCached();
+        nodes.add(n);
+    }
+
+    public void addNode(int offs, Node n)
+    {
+        clearCached();
+        nodes.add(offs, n);
+    }
+
     public Boolean isClosed() {
         int s = nodes.size();

trunk/src/org/openstreetmap/josm/data/osm/visitor/MapPaintVisitor.java

@@ -93 +93 @@
     public ElemStyle getPrimitiveStyle(OsmPrimitive osm) {
         if(!useStyleCache)
-            return (styles != null) ? (IconElemStyle)styles.get(osm) : null;
+            return (styles != null) ? styles.get(osm) : null;
 
         if(osm.mappaintStyle == null && styles != null) {
-            osm.mappaintStyle =  styles.get(osm);
+            osm.mappaintStyle = styles.get(osm);
             if(osm instanceof Way)
                 ((Way)osm).isMappaintArea = styles.isArea(osm);
         }
         return osm.mappaintStyle;
+    }
+
+    public IconElemStyle getPrimitiveNodeStyle(OsmPrimitive osm) {
+        if(!useStyleCache)
+            return (styles != null) ? styles.getIcon(osm) : null;
+
+        if(osm.mappaintStyle == null && styles != null)
+            osm.mappaintStyle = styles.getIcon(osm);
+
+        return (IconElemStyle)osm.mappaintStyle;
     }
 
@@ -417 +427 @@
     }
 
-    public void drawSelectedRelation(Relation r)
-    {
-        for (RelationMember m : r.members)
-        {
-            if (m.member != null && !m.member.incomplete && !m.member.deleted
-            && !(m.member instanceof Relation))
-            {
-                /* nodes drawn on second call */
-                if(!(m.member instanceof Node))
-                    drawSelectedMember(m.member, styles != null ? styles.get(m.member)
-                    : null, true, true);
-            }
-        }
-    }
-
     public void drawSelectedMember(OsmPrimitive osm, ElemStyle style, Boolean area,
     Boolean areaselected)
@@ -463 +458 @@
 
         r.mappaintVisibleCode = 0;
-        /* TODO implement visible handling for relations too */
 
         // TODO: is it possible to do this like the nodes/ways code?
@@ -469 +463 @@
             return;
 
-        // draw multipolygon relations including their ways
-        // other relations are only drawn when selected
-
-        // we are in the "draw selected" phase
-        // TODO: is it necessary to check for r.selected?
-        if(r.selected && selectedCall)
+        if(selectedCall)
         {
             for (RelationMember m : r.members)
             {
                 if (m.member != null && !m.member.incomplete && !m.member.deleted
-                && (drawRestriction || m.member instanceof Node))
-                {
-                    drawSelectedMember(m.member, styles != null ? styles.get(m.member) : null, true, true);
+                && m.member instanceof Node)
+                {
+                    drawSelectedMember(m.member, styles != null ? getPrimitiveStyle(m.member) : null, true, true);
                 }
             }
             return;
         }
-
-        if (drawMultipolygon && r.keys != null && "multipolygon".equals(r.keys.get("type")))
-        {
-            drawMultipolygon(r);
-            return;
-        }
-
-        if (drawRestriction && r.keys != null && "restriction".equals(r.keys.get("type")))
+        else if (drawMultipolygon && r.keys != null && "multipolygon".equals(r.keys.get("type")))
+        {
+            if(drawMultipolygon(r))
+              return;
+        }
+        else if (drawRestriction && r.keys != null && "restriction".equals(r.keys.get("type")))
         {
             drawRestriction(r);
-            return;
-        }
-
-        if(r.selected)
-            drawSelectedRelation(r);
-    }
-
+        }
+
+        if(r.selected) /* draw ways*/
+        {
+            for (RelationMember m : r.members)
+            {
+                if (m.member != null && !m.member.incomplete && !m.member.deleted
+                && m.member instanceof Way) /* nodes drawn on second call */
+                {
+                    drawSelectedMember(m.member, styles != null ? getPrimitiveStyle(m.member)
+                    : null, true, true);
+                }
+            }
+        }
+    }
 
     // this current experimental implementation will only work for standard restrictions:
@@ -724 +719 @@
         }
 
-        IconElemStyle nodeStyle = (IconElemStyle)getPrimitiveStyle(r);
+        IconElemStyle nodeStyle = getPrimitiveNodeStyle(r);
 
         if (nodeStyle == null) {
@@ -748 +743 @@
     }
 
-    public void drawMultipolygon(Relation r) {
+    public Boolean drawMultipolygon(Relation r) {
         Collection<Way> inner = new LinkedList<Way>();
         Collection<Way> outer = new LinkedList<Way>();
@@ -754 +749 @@
         Collection<Way> outerclosed = new LinkedList<Way>();
         Boolean incomplete = false;
+        Boolean drawn = false;
 
         r.clearErrors();
@@ -787 +783 @@
                             outer.add(w);
                         else if(r.selected)
-                            drawSelectedMember(m.member, styles != null ? styles.get(m.member) : null, true, true);
+                            drawSelectedMember(m.member, styles != null
+                            ? getPrimitiveStyle(m.member) : null, true, true);
                     }
                 }
@@ -798 +795 @@
         }
 
-        ElemStyle wayStyle = styles != null ? styles.get(r) : null;
+        ElemStyle wayStyle = styles != null ? getPrimitiveStyle(r) : null;
         if(styles != null && (wayStyle == null || !(wayStyle instanceof AreaElemStyle)))
         {
@@ -806 +803 @@
                    wayStyle = styles.get(w);
             }
+            r.mappaintStyle = wayStyle;
         }
 
@@ -814 +812 @@
             Collection<Way> join = new LinkedList<Way>();
 
+            drawn = true;
             for (Way w : outer)
             {
@@ -954 +953 @@
                 for (Way wOuter : outer)
                     wOuter.mappaintVisibleCode = viewid;
-                return;
+                return drawn;
             }
             for (Way wInner : inner)
             {
-                ElemStyle innerStyle = styles.get(wInner);
+                ElemStyle innerStyle = getPrimitiveStyle(wInner);
                 if(innerStyle == null)
                 {
@@ -988 +987 @@
             for (Way wOuter : outer)
             {
-                ElemStyle outerStyle = styles.get(wOuter);
+                ElemStyle outerStyle = getPrimitiveStyle(wOuter);
                 if(outerStyle == null)
                 {
@@ -1016 +1015 @@
             }
         }
-        else if(r.selected)
-            drawSelectedRelation(r);
+        return drawn;
     }