remove local scope variables from global namespace.

git-svn-id: http://svn.openlayers.org/trunk/openlayers@5028 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf

lib/OpenLayers/Ajax.js

@@ -526,6 +526,6 @@ OpenLayers.Ajax.getElementsByTagNameNS  = function(parentnode, nsuri,
  */
 OpenLayers.Ajax.serializeXMLToString = function(xmldom) {
     var serializer = new XMLSerializer();
-    data = serializer.serializeToString(xmldom);
+    var data = serializer.serializeToString(xmldom);
     return data;
 };

lib/OpenLayers/Control/MouseToolbar.js

@@ -206,7 +206,7 @@ OpenLayers.Control.MouseToolbar = OpenLayers.Class(
             case "measure":
                 var distance = "";
                 if (this.measureStart) {
-                    measureEnd = this.map.getLonLatFromViewPortPx(this.mouseDragStart);
+                    var measureEnd = this.map.getLonLatFromViewPortPx(this.mouseDragStart);
                     distance = OpenLayers.Util.distVincenty(this.measureStart, measureEnd);
                     distance = Math.round(distance * 100) / 100;
                     distance = distance + "km";

lib/OpenLayers/Control/PanZoomBar.js

@@ -152,7 +152,7 @@ OpenLayers.Control.PanZoomBar = OpenLayers.Class(OpenLayers.Control.PanZoom, {
         this.sliderEvents.register("dblclick", this, this.doubleClick);
         this.sliderEvents.register("click", this, this.doubleClick);
         
-        sz = new OpenLayers.Size();
+        var sz = new OpenLayers.Size();
         sz.h = this.zoomStopHeight * this.map.getNumZoomLevels();
         sz.w = this.zoomStopWidth;
         var div = null;

lib/OpenLayers/Format/GML.js

@@ -203,7 +203,7 @@ OpenLayers.Format.GML = OpenLayers.Class(OpenLayers.Format.XML, {
             // look for <gml:pos>
             var nodeList = this.getElementsByTagNameNS(node, this.gmlns, "pos");
             if(nodeList.length > 0) {
-                coordString = nodeList[0].firstChild.nodeValue;
+                var coordString = nodeList[0].firstChild.nodeValue;
                 coordString = coordString.replace(this.regExes.trimSpace, "");
                 coords = coordString.split(this.regExes.splitSpace);
             }

lib/OpenLayers/Format/KML.js

@@ -302,7 +302,7 @@ OpenLayers.Format.KML = OpenLayers.Class(OpenLayers.Format.XML, {
             for(var i=0; i<children.length; ++i ) {
                 child = children[i];
                 if(child.nodeType == 1) {
-                    type = (child.prefix) ?
+                    var type = (child.prefix) ?
                             child.nodeName.split(":")[1] :
                             child.nodeName;
                     var parser = this.parseGeometry[type.toLowerCase()];
@@ -337,10 +337,10 @@ OpenLayers.Format.KML = OpenLayers.Class(OpenLayers.Format.XML, {
                 if(grandchildren.length == 1) {
                     grandchild = grandchildren[0];
                     if(grandchild.nodeType == 3 || grandchild.nodeType == 4) {
-                        name = (child.prefix) ?
+                        var name = (child.prefix) ?
                                 child.nodeName.split(":")[1] :
                                 child.nodeName;
-                        value = grandchild.nodeValue.replace(
+                        var value = grandchild.nodeValue.replace(
                                                 this.regExes.trimSpace, "");
                         attributes[name] = value;
                     }

lib/OpenLayers/Geometry/Polygon.js

@@ -79,7 +79,7 @@ OpenLayers.Geometry.Polygon.createRegularPolygon = function(origin, radius, side
     var rotateAngle, x, y;
     var points = [];
     for(var i=0; i<sides; ++i) {
-        rotatedAngle = angle + (i * 2 * Math.PI / sides);
+        var rotatedAngle = angle + (i * 2 * Math.PI / sides);
         x = origin.x + (radius * Math.cos(rotatedAngle));
         y = origin.y + (radius * Math.sin(rotatedAngle));
         points.push(new OpenLayers.Geometry.Point(x, y));

lib/OpenLayers/Layer/EventPane.js

@@ -136,8 +136,8 @@ OpenLayers.Layer.EventPane = OpenLayers.Class(OpenLayers.Layer, {
 
         var viewSize = this.map.getSize();
         
-        msgW = Math.min(viewSize.w, 300);
+        var msgW = Math.min(viewSize.w, 300);
-        msgH = Math.min(viewSize.h, 200);
+        var msgH = Math.min(viewSize.h, 200);
         var size = new OpenLayers.Size(msgW, msgH);
 
         var centerPx = new OpenLayers.Pixel(viewSize.w/2, viewSize.h/2);

lib/OpenLayers/Layer/GeoRSS.js

@@ -193,7 +193,7 @@ OpenLayers.Layer.GeoRSS = OpenLayers.Class(OpenLayers.Layer.Markers, {
             
             data.popupSize = this.popupSize ? this.popupSize.clone() : new OpenLayers.Size(250, 120);
             if ((title != null) && (description != null)) {
-                contentHTML = '<div class="olLayerGeoRSSClose">[x]</div>'; 
+                var contentHTML = '<div class="olLayerGeoRSSClose">[x]</div>'; 
                 contentHTML += '<div class="olLayerGeoRSSTitle">';
                 if (link) {
                     contentHTML += '<a class="link" href="'+link+'" target="_blank">';
@@ -224,7 +224,7 @@ OpenLayers.Layer.GeoRSS = OpenLayers.Class(OpenLayers.Layer.Markers, {
      * evt - {Event} 
      */
     markerClick: function(evt) {
-        sameMarkerClicked = (this == this.layer.selectedFeature);
+        var sameMarkerClicked = (this == this.layer.selectedFeature);
         this.layer.selectedFeature = (!sameMarkerClicked) ? this : null;
         for(var i=0; i < this.layer.map.popups.length; i++) {
             this.layer.map.removePopup(this.layer.map.popups[i]);

lib/OpenLayers/Layer/Markers.js

@@ -70,7 +70,7 @@ OpenLayers.Layer.Markers = OpenLayers.Class(OpenLayers.Layer, {
         OpenLayers.Layer.prototype.moveTo.apply(this, arguments);
 
         if (zoomChanged || !this.drawn) {
-            for(i=0; i < this.markers.length; i++) {
+            for(var i=0; i < this.markers.length; i++) {
                 this.drawMarker(this.markers[i]);
             }
             this.drawn = true;

lib/OpenLayers/Layer/Text.js

@@ -90,7 +90,7 @@ OpenLayers.Layer.Text = OpenLayers.Class(OpenLayers.Layer.Markers, {
                 } else {
                     var vals = currLine.split('\t');
                     var location = new OpenLayers.LonLat(0,0);
-                    var title; var url;
+                    var title, url, description;
                     var icon, iconSize, iconOffset, overflow;
                     var set = false;
                     for (var valIndex = 0; valIndex < vals.length; valIndex++) {

lib/OpenLayers/Map.js

@@ -591,7 +591,7 @@ OpenLayers.Map = OpenLayers.Class({
         // if we removed the base layer, need to set a new one
         if (setNewBaseLayer && (this.baseLayer == layer)) {
             this.baseLayer = null;
-            for(i=0; i < this.layers.length; i++) {
+            for(var i=0; i < this.layers.length; i++) {
                 var iLayer = this.layers[i];
                 if (iLayer.isBaseLayer) {
                     this.setBaseLayer(iLayer);

lib/OpenLayers/Tile.js

@@ -208,7 +208,7 @@ OpenLayers.Tile = OpenLayers.Class({
                 bottomRight.lon = 180+bottomRight.lon+180;
             }        
         }
-        bounds = new OpenLayers.Bounds(topLeft.lon, 
+        var bounds = new OpenLayers.Bounds(topLeft.lon, 
                                        bottomRight.lat, 
                                        bottomRight.lon, 
                                        topLeft.lat);  

lib/OpenLayers/Util.js

@@ -520,7 +520,7 @@ OpenLayers.Util.applyDefaults = function (to, from) {
  *          of being URL escaped (foo%3Abar). 
  */
 OpenLayers.Util.getParameterString = function(params) {
-    paramsArray = [];
+    var paramsArray = [];
     
     for (var key in params) {
       var value = params[key];
@@ -1070,8 +1070,8 @@ OpenLayers.Util.isEquivalentUrl = function(url1, url2, options) {
         ignoreHash: true
     });
 
-    urlObj1 = OpenLayers.Util.createUrlObject(url1, options);
+    var urlObj1 = OpenLayers.Util.createUrlObject(url1, options);
-    urlObj2 = OpenLayers.Util.createUrlObject(url2, options);
+    var urlObj2 = OpenLayers.Util.createUrlObject(url2, options);
 
     //compare all keys (host, port, etc)
     for(var key in urlObj1) {