Changeset 5373

Timestamp:

12/11/07 03:32:18 (1 year ago)

Author:

fredj

Message:

Remove the drawCurve method from the renderers because it is never used. Reported by pgiraud (Closes #1138)

Files:

• trunk/openlayers/lib/OpenLayers/Renderer/Elements.js (modified) (1 diff)
• trunk/openlayers/lib/OpenLayers/Renderer/SVG.js (modified) (1 diff)
• trunk/openlayers/lib/OpenLayers/Renderer/VML.js (modified) (1 diff)

trunk/openlayers/lib/OpenLayers/Renderer/Elements.js

@@ -269 +269 @@
-
-    /**
-     * Method: drawCurve
-     * Virtual function for drawing Curve Geometry. 
-     * Should be implemented by subclasses.
-     * This method is only called by the renderer itself.
-     * 
-     * Parameters: 
-     * node - {DOMElement}
-     * geometry - {<OpenLayers.Geometry>}
-     */ 
-    drawCurve: function(node, geometry) {},
-
-    /**
-     * Method: drawSurface
-     * Virtual function for drawing Surface Geometry. 

trunk/openlayers/lib/OpenLayers/Renderer/SVG.js

@@ -439 +439 @@
-    },
-    
-    
-    /**
-     * Method: drawCurve
-     * This method is only called by the renderer itself.
-     * 
-     * Parameters: 
-     * node - {DOMElement}
-     * geometry - {<OpenLayers.Geometry>}
-     */ 
-    drawCurve: function(node, geometry) {
-        var d = null;
-        var draw = true;
-        for (var i = 0; i < geometry.components.length; i++) {
-            if ((i%3) == 0 && (i/3) == 0) {
-                var component = this.getShortString(geometry.components[i]);
-                if (!component) { draw = false; }
-                d = "M " + component;
-            } else if ((i%3) == 1) {
-                var component = this.getShortString(geometry.components[i]);
-                if (!component) { draw = false; }
-                d += " C " + component;
-            } else {
-                var component = this.getShortString(geometry.components[i]);
-                if (!component) { draw = false; }
-                d += " " + component;
-            }
-        }
-        if (draw) {
-            node.setAttributeNS(null, "d", d);
-        } else {
-            node.setAttributeNS(null, "d", "");
-        }    
-    },
-    
-    /**
-     * Method: drawSurface

trunk/openlayers/lib/OpenLayers/Renderer/VML.js

@@ -525 +525 @@
-    },
-
-
-
-    /**
-     * Method: drawCurve
-     * 
-     * Parameters:
-     * node - {DOMElement}
-     * geometry - {<OpenLayers.Geometry>}
-     */
-    drawCurve: function(node, geometry) {
-        this.setNodeDimension(node, geometry);
-
-        var resolution = this.getResolution();
-    
-        var path = "";
-        for (var i = 0; i < geometry.components.length; i++) {
-            var x = geometry.components[i].x / resolution;
-            var y = geometry.components[i].y / resolution;
-    
-            if ((i%3)==0 && (i/3)==0) {
-                path += "m";
-            } else if ((i%3)==1) {
-                path += " c";
-            }
-            path += " " + x + "," + y;
-        }
-        path += " x e";
-
-        node.path = path;
-    },
-
-    /**
-     * Method: drawSurface