Changeset 4623 in josm for trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyles.java

Timestamp:

2011-12-01T00:53:26+01:00 (12 years ago)

Author:

Don-vip

Message:

see #6987 - multipolygon cache aware of events requiring cache refresh (data change, zoom, layer removing)

File:

• trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyles.java (modified) (8 diffs)

trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyles.java

@@ -6 +6 @@
 import java.util.Collection;
 import java.util.Collections;
-import java.util.HashMap;
 import java.util.Iterator;
 import java.util.List;
-import java.util.Map;
 import java.util.Map.Entry;
 
@@ -17 +15 @@
 import org.openstreetmap.josm.data.osm.Way;
 import org.openstreetmap.josm.data.osm.visitor.paint.relations.Multipolygon;
+import org.openstreetmap.josm.data.osm.visitor.paint.relations.MultipolygonCache;
 import org.openstreetmap.josm.gui.NavigatableComponent;
 import org.openstreetmap.josm.gui.mappaint.StyleCache.StyleList;
@@ -31 +30 @@
     private int defaultNodesIdx, defaultLinesIdx;
     
-    private final Map<NavigatableComponent, Map<Relation, Multipolygon>> multipolygonsCache;
-
     public ElemStyles()
     {
         styleSources = new ArrayList<StyleSource>();
-        multipolygonsCache = new HashMap<NavigatableComponent, Map<Relation,Multipolygon>>();
     }
 
@@ -121 +117 @@
     }
 
-    private final Multipolygon getCachedMultipolygon(NavigatableComponent nc, Relation r) {
-        Multipolygon multipolygon = null;
-        Map<Relation, Multipolygon> map = multipolygonsCache.get(nc);
-        if (map == null) {
-            multipolygonsCache.put(nc, map = new HashMap<Relation, Multipolygon>());
-        }
-        multipolygon = map.get(r);
-        if (multipolygon == null) {
-            map.put(r, multipolygon = new Multipolygon(nc));
-            multipolygon.load(r);
-        }
-        return multipolygon;
-    }
-    
     /**
      * Create the list of styles and its valid scale range for one primitive.
@@ -179 +161 @@
                     continue;
                 }
-                Multipolygon multipolygon = getCachedMultipolygon(nc, r);
+                Multipolygon multipolygon = MultipolygonCache.getInstance().get(nc, r);
 
                 if (multipolygon.getOuterWays().contains(osm)) {
@@ -244 +226 @@
                     continue;
                 }
-                final Multipolygon multipolygon = getCachedMultipolygon(nc, ref);
+                final Multipolygon multipolygon = MultipolygonCache.getInstance().get(nc, ref);
 
                 if (multipolygon.getInnerWays().contains(osm)) {
@@ -277 +259 @@
                 if (!Utils.exists(p.a, AreaElemStyle.class)) {
                     // look at outer ways to find area style
-                    Multipolygon multipolygon = new Multipolygon(nc);
-                    multipolygon.load((Relation) osm);
+                    Multipolygon multipolygon = MultipolygonCache.getInstance().get(nc, (Relation) osm);
                     for (Way w : multipolygon.getOuterWays()) {
                         Pair<StyleList, Range> wayStyles = generateStyles(w, scale, null, false);
@@ -429 +410 @@
         styleSources.addAll(sources);
     }
-
-    public void clearMultipolygonsCache(NavigatableComponent nc) {
-        multipolygonsCache.remove(nc);
-    }
 }