Changeset 4623 in josm for trunk/src/org/openstreetmap/josm/gui

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)

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• MapView.java (modified) (4 diffs)
• layer/OsmDataLayer.java (modified) (2 diffs)
• mappaint/ElemStyles.java (modified) (8 diffs)

trunk/src/org/openstreetmap/josm/gui/MapView.java

@@ -45 +45 @@
 import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
 import org.openstreetmap.josm.data.osm.visitor.paint.PaintColors;
+import org.openstreetmap.josm.data.osm.visitor.paint.relations.MultipolygonCache;
 import org.openstreetmap.josm.gui.layer.GpxLayer;
 import org.openstreetmap.josm.gui.layer.Layer;
@@ -51 +52 @@
 import org.openstreetmap.josm.gui.layer.markerlayer.MarkerLayer;
 import org.openstreetmap.josm.gui.layer.markerlayer.PlayHeadMarker;
-import org.openstreetmap.josm.gui.mappaint.MapPaintStyles;
 import org.openstreetmap.josm.tools.AudioPlayer;
 import org.openstreetmap.josm.tools.BugReportExceptionHandler;
@@ -238 +238 @@
             }
         });
+        
+        // Add Multipolygon cache to layer and zoom listeners
+        addLayerChangeListener(MultipolygonCache.getInstance());
+        addZoomChangeListener(MultipolygonCache.getInstance());
     }
 
@@ -843 +847 @@
         Main.pref.removePreferenceChangeListener(this);
         DataSet.removeSelectionListener(repaintSelectionChangedListener);
-        MapPaintStyles.getStyles().clearMultipolygonsCache(this);
+        MultipolygonCache.getInstance().clear(this);
     }
 

trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

@@ -63 +63 @@
 import org.openstreetmap.josm.data.osm.visitor.paint.MapRendererFactory;
 import org.openstreetmap.josm.data.osm.visitor.paint.Rendering;
+import org.openstreetmap.josm.data.osm.visitor.paint.relations.MultipolygonCache;
 import org.openstreetmap.josm.data.projection.Projection;
 import org.openstreetmap.josm.data.validation.TestError;
@@ -207 +208 @@
         conflicts = new ConflictCollection();
         data.addDataSetListener(new DataSetListenerAdapter(this));
+        data.addDataSetListener(MultipolygonCache.getInstance());
         DataSet.addSelectionListener(this);
     }

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);
-    }
 }