ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

code cleanup - remove unused variables and declare local variables, p=avlee, r=me (closes #2326)

Browse Source 
git-svn-id: http://svn.openlayers.org/trunk/openlayers@9766 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf
This commit is contained in:
Éric Lemoine
2009-10-27 12:15:22 +00:00
parent 316136b346
commit ab42111aac
4 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/OpenLayers/BaseTypes/Bounds.js
View File

@@ -255,8 +255,6 @@ OpenLayers.Bounds = OpenLayers.Class({
origin = this.getCenterLonLat(); origin = this.getCenterLonLat();
} }
var bounds = [];
var origx,origy; var origx,origy;
// get origin coordinates // get origin coordinates

1
lib/OpenLayers/Control/Panel.js
View File

@@ -189,7 +189,6 @@ OpenLayers.Control.Panel = OpenLayers.Class(OpenLayers.Control, {
// since they need to pass through. // since they need to pass through.
for (var i=0, len=controls.length; i<len; i++) { for (var i=0, len=controls.length; i<len; i++) {
var element = document.createElement("div"); var element = document.createElement("div");
var textNode = document.createTextNode(" ");
controls[i].panel_div = element; controls[i].panel_div = element;
if (controls[i].title != "") { if (controls[i].title != "") {
controls[i].panel_div.title = controls[i].title; controls[i].panel_div.title = controls[i].title;

5
lib/OpenLayers/Control/ZoomBox.js
View File

@@ -51,12 +51,13 @@ OpenLayers.Control.ZoomBox = OpenLayers.Class(OpenLayers.Control, {
*/ */
zoomBox: function (position) { zoomBox: function (position) {
if (position instanceof OpenLayers.Bounds) { if (position instanceof OpenLayers.Bounds) {
var bounds;
if (!this.out) { if (!this.out) {
var minXY = this.map.getLonLatFromPixel( var minXY = this.map.getLonLatFromPixel(
new OpenLayers.Pixel(position.left, position.bottom)); new OpenLayers.Pixel(position.left, position.bottom));
var maxXY = this.map.getLonLatFromPixel( var maxXY = this.map.getLonLatFromPixel(
new OpenLayers.Pixel(position.right, position.top)); new OpenLayers.Pixel(position.right, position.top));
var bounds = new OpenLayers.Bounds(minXY.lon, minXY.lat, bounds = new OpenLayers.Bounds(minXY.lon, minXY.lat,
maxXY.lon, maxXY.lat); maxXY.lon, maxXY.lat);
} else { } else {
var pixWidth = Math.abs(position.right-position.left); var pixWidth = Math.abs(position.right-position.left);
@@ -70,7 +71,7 @@ OpenLayers.Control.ZoomBox = OpenLayers.Class(OpenLayers.Control, {
var xmax = center.lon + (extent.getWidth()/2)*zoomFactor; var xmax = center.lon + (extent.getWidth()/2)*zoomFactor;
var ymin = center.lat - (extent.getHeight()/2)*zoomFactor; var ymin = center.lat - (extent.getHeight()/2)*zoomFactor;
var ymax = center.lat + (extent.getHeight()/2)*zoomFactor; var ymax = center.lat + (extent.getHeight()/2)*zoomFactor;
var bounds = new OpenLayers.Bounds(xmin, ymin, xmax, ymax); bounds = new OpenLayers.Bounds(xmin, ymin, xmax, ymax);
} }
// always zoom in/out // always zoom in/out
var lastZoom = this.map.getZoom(); var lastZoom = this.map.getZoom();

2
lib/OpenLayers/Geometry/Collection.js
View File

@@ -382,7 +382,7 @@ OpenLayers.Geometry.Collection = OpenLayers.Class(OpenLayers.Geometry, {
distanceTo: function(geometry, options) { distanceTo: function(geometry, options) {
var edge = !(options && options.edge === false); var edge = !(options && options.edge === false);
var details = edge && options && options.details; var details = edge && options && options.details;
var result, best; var result, best, distance;
var min = Number.POSITIVE_INFINITY; var min = Number.POSITIVE_INFINITY;
for(var i=0, len=this.components.length; i<len; ++i) { for(var i=0, len=this.components.length; i<len; ++i) {
result = this.components[i].distanceTo(geometry, options); result = this.components[i].distanceTo(geometry, options);