diff --git a/lib/OpenLayers/Format/GPX.js b/lib/OpenLayers/Format/GPX.js index 870ccb3d22..7aa8eab66f 100644 --- a/lib/OpenLayers/Format/GPX.js +++ b/lib/OpenLayers/Format/GPX.js @@ -162,10 +162,10 @@ OpenLayers.Format.GPX = OpenLayers.Class(OpenLayers.Format.XML, { // node is either a wpt, trk or rte // attributes are children of the form value var attributes = {}; - var attrNode = node.firstChild; + var attrNode = node.firstChild, value, name; while(attrNode) { if(attrNode.nodeType == 1) { - var value = attrNode.firstChild; + value = attrNode.firstChild; if(value.nodeType == 3 || value.nodeType == 4) { name = (attrNode.prefix) ? attrNode.nodeName.split(":")[1] : diff --git a/lib/OpenLayers/Format/KML.js b/lib/OpenLayers/Format/KML.js index 811312e7fb..76e2b65c2f 100644 --- a/lib/OpenLayers/Format/KML.js +++ b/lib/OpenLayers/Format/KML.js @@ -353,11 +353,10 @@ OpenLayers.Format.KML = OpenLayers.Class(OpenLayers.Format.XML, { var types = ["LineStyle", "PolyStyle", "IconStyle", "BalloonStyle", "LabelStyle"]; - var type, nodeList, geometry, parser; + var type, styleTypeNode, nodeList, geometry, parser; for(var i=0, len=types.length; i0; --i) { - points = rings[i].components; + var points = rings[i].components; for (var j=0, jj=points.length; j 0) { - bounds = this.adjustBounds(arguments[0]); + var bounds = this.adjustBounds(arguments[0]); var res = this.map.getResolution(); var x = Math.round((bounds.left - this.tileOrigin.lon) / (res * this.tileSize.w)); var y = Math.round((this.tileOrigin.lat - bounds.top) / (res * this.tileSize.h)); diff --git a/lib/OpenLayers/Renderer.js b/lib/OpenLayers/Renderer.js index c7bc5e04e6..582b1afe05 100644 --- a/lib/OpenLayers/Renderer.js +++ b/lib/OpenLayers/Renderer.js @@ -181,8 +181,8 @@ OpenLayers.Renderer = OpenLayers.Class({ if(style.display != "none" && style.label && rendered !== false) { var location = feature.geometry.getCentroid(); if(style.labelXOffset || style.labelYOffset) { - xOffset = isNaN(style.labelXOffset) ? 0 : style.labelXOffset; - yOffset = isNaN(style.labelYOffset) ? 0 : style.labelYOffset; + var xOffset = isNaN(style.labelXOffset) ? 0 : style.labelXOffset; + var yOffset = isNaN(style.labelYOffset) ? 0 : style.labelYOffset; var res = this.getResolution(); location.move(xOffset*res, yOffset*res); }