js linting / code fixup.

6 files changed, 99 insertions, 104 deletions

diff --git a/web/nms.gathering.org/js/nms-color-util.js b/web/nms.gathering.org/js/nms-color-util.js
index f50ee04..6b5a4b5 100644
--- a/web/nms.gathering.org/js/nms-color-util.js
+++ b/web/nms.gathering.org/js/nms-color-util.js
@@ -89,6 +89,5 @@ function getColor(x,y) {
 	var data = imageData.data;
 	if (data.length < 4)
 		return false;
-	var ret = 'rgb(' + data[0] + ',' + data[1] + ',' + data[2] + ')';
-	return ret;
+    return 'rgb(' + data[0] + ',' + data[1] + ',' + data[2] + ')';
 }
diff --git a/web/nms.gathering.org/js/nms-data.js b/web/nms.gathering.org/js/nms-data.js
index 155b5a8..14e5fed 100644
--- a/web/nms.gathering.org/js/nms-data.js
+++ b/web/nms.gathering.org/js/nms-data.js
@@ -82,7 +82,7 @@ var nmsData = nmsData || {
 nmsData._dropData = function (name) {
 	delete this[name];
 	delete this.old[name];
-}
+};
 
 nmsData.removeSource = function (name) {
 	if (this._sources[name] == undefined) {
@@ -94,7 +94,7 @@ nmsData.removeSource = function (name) {
 		clearInterval(this._sources[name]['handle']);
 	}
 	delete this._sources[name];
-}
+};
 
 /*
  * Register a source.
@@ -121,7 +121,7 @@ nmsData.registerSource = function(name, target) {
 	}
 
 	this.stats.pollSets++;
-}
+};
 
 /*
  * Add a handler (callback) for a source, using an id.
@@ -142,7 +142,7 @@ nmsData.addHandler = function(name, id, cb, cbdata) {
 	}
 	this._sources[name].cbs[id] = cbob;
 	this.updateSource(name);
-}
+};
 
 /*
  * Unregister all handlers with the "id" for all sources.
@@ -154,11 +154,11 @@ nmsData.unregisterHandlerWildcard = function(id) {
 	for (var v in nmsData._sources) {
 		this.unregisterHandler(v, id);
 	}
-}
+};
 
 nmsData.unregisterHandler = function(name, id) {
 	delete this._sources[name].cbs[id];
-}
+};
 
 /*
  * Updates a source.
@@ -181,11 +181,11 @@ nmsData.updateSource = function(name) {
 		return;
 	}
 	this._genericUpdater(name, true);
-}
+};
 
 nmsData.invalidate = function(name) {
 	this._genericUpdater(name, false);
-}
+};
 /*
  * Reset a source, deleting all data, including old.
  *
@@ -195,7 +195,7 @@ nmsData.resetSource = function(name) {
 	this[name] = {};
 	this.old[name] = {};
 	this.updateSource(name);
-}
+};
 
 /*
  * Updates nmsData[name] and nmsData.old[name], issuing any callbacks where
@@ -236,16 +236,16 @@ nmsData._genericUpdater = function(name, cacheok) {
 				nmsData[name] = data;
 				nmsMap.drawNow();
 				for (var i in nmsData._sources[name].cbs) {
-					var tmp = nmsData._sources[name].cbs[i];
-					if (tmp.cb != undefined) {
-						tmp.cb(tmp.cbdata);
+					var tmp2 = nmsData._sources[name].cbs[i];
+					if (tmp2.cb != undefined) {
+						tmp2.cb(tmp2.cbdata);
 					}
 				}
 			} else {
-				for (var i in nmsData._sources[name].cbs) {
-					var tmp = nmsData._sources[name].cbs[i];
+				for (var j in nmsData._sources[name].cbs) {
+					var tmp = nmsData._sources[name].cbs[j];
 					if (tmp.cb != undefined && tmp.fresh) {
-						nmsData._sources[name].cbs[i].fresh = false;
+						nmsData._sources[name].cbs[j].fresh = false;
 						tmp.cb(tmp.cbdata);
 					}
 				}
diff --git a/web/nms.gathering.org/js/nms-info-box.js b/web/nms.gathering.org/js/nms-info-box.js
index dc993a9..e93ab55 100644
--- a/web/nms.gathering.org/js/nms-info-box.js
+++ b/web/nms.gathering.org/js/nms-info-box.js
@@ -15,8 +15,8 @@ var nmsInfoBox = nmsInfoBox || {
   stats: {},
   _container: false, //Container window
   _window: false, //Active window (reference to _windowTypes object or false)
-  _windowTypes: [], //List of all avaliable window types
-}
+  _windowTypes: [] //List of all avaliable window types
+};
 
 /*
  * Shows a window from the _windowTypes list
@@ -30,7 +30,7 @@ nmsInfoBox.showWindow = function (windowName,argument) {
       return;
     }
   }
-}
+};
 
 /*
  * Refresh the active window
@@ -40,7 +40,7 @@ nmsInfoBox.showWindow = function (windowName,argument) {
  */
 nmsInfoBox.refresh = function() {
   nmsInfoBox._show();
-}
+};
 
 /*
  * Internal function to show the active _window and pass along any arguments
@@ -80,7 +80,7 @@ nmsInfoBox._show = function(argument) {
   }
   this._container.appendChild(panel);
   this._container.style.display = "block";
-}
+};
 
 /*
  * Hide the active window and tell it to unload
@@ -95,7 +95,7 @@ nmsInfoBox.hide = function() {
 	nmsData.unregisterHandler("switches","switchshower");
 	nmsData.unregisterHandler("smanagement","switchshower");
 	nmsData.unregisterHandler("snmp","switchshower");
-}
+};
 
 /*
  * Window type: Add Switch
@@ -130,7 +130,7 @@ nmsInfoBox._windowTypes.addSwitch = {
       data:myData,
       success: function (data, textStatus, jqXHR) {
         var result = JSON.parse(data);
-        if(result.switches_addded.length > 0) {
+        if(result.switches_addded.length > 0) { // FIXME unresolved variable switches_addded
           nmsInfoBox.hide();
         }
         nmsData.invalidate("switches");
@@ -226,9 +226,10 @@ nmsInfoBox._windowTypes.switchInfo = {
     var template = {};
 
     nmsInfoBox._editValues = {};
+    var place;
     for (var v in this.swi) {
       if (v == "placement") {
-        var place = JSON.stringify(this.swi[v]);
+        place = JSON.stringify(this.swi[v]);
         template[v] = place;
         continue;
       }
@@ -313,7 +314,7 @@ nmsInfoBox._windowTypes.switchInfo = {
 nmsInfoBox.click = function(sw)
 {
   this.showWindow("switchInfo",sw);
-}
+};
 
 /*
  * General-purpose table-maker?
@@ -343,7 +344,7 @@ nmsInfoBox._makeTable = function(content, caption) {
 		td2.innerHTML = content[v][1];
 	}
 	return table;
-}
+};
 
 /*
  * Create and return a table for comments.
@@ -386,7 +387,7 @@ nmsInfoBox._makeCommentTable = function(content) {
 		td2.innerHTML = comment["comment"];
 	}
 	return table;
-}
+};
 
 nmsInfoBox._searchSmart = function(id, sw) {
 	if (nmsData.smanagement.switches[sw].distro == id) {
@@ -418,7 +419,7 @@ nmsInfoBox._searchSmart = function(id, sw) {
 		}
 	}
 	return false;
-}
+};
 
 /*
  * FIXME: Not sure this belongs here, it's really part of the "Core" ui,
@@ -461,21 +462,21 @@ nmsInfoBox._search = function() {
 		document.getElementById("searchbox").dataset.match = '';
 		document.getElementById("searchbox").removeEventListener("keydown",nmsInfoBox._searchKeyListener,false);
 	}
-}
+};
 
 nmsInfoBox._searchKeyListener = function(e) {
 	if(e.keyCode == 13) {
 		var sw = document.getElementById("searchbox").dataset.match;
 		nmsInfoBox.showWindow("switchInfo",sw);
 	}
-}
+};
 
 
 nmsInfoBox._nullBlank = function(x) {
 	if (x == null || x == false || x == undefined)
 		return "";
 	return x;
-}
+};
 
 
 nmsInfoBox._editChange = function(sw, v) {
@@ -496,13 +497,9 @@ nmsInfoBox._editChange = function(sw, v) {
 	var myData = nmsInfoBox._editStringify(sw);
 	var out = document.getElementById("edit-output");
 	out.value = myData;
-}
+};
 
 nmsInfoBox._editStringify = function(sw) {
-	for (var key in nmsInfoBox._editValues) {
-		var val = nmsInfoBox._editValues[key];
-	}
-	nmsInfoBox._editValues['sysname'] = sw;
-	var myData = JSON.stringify([nmsInfoBox._editValues]);
-	return myData;
-}
+    nmsInfoBox._editValues['sysname'] = sw;
+    return JSON.stringify([nmsInfoBox._editValues]);
+};
diff --git a/web/nms.gathering.org/js/nms-map-handlers.js b/web/nms.gathering.org/js/nms-map-handlers.js
index b1d31f0..b98a944 100644
--- a/web/nms.gathering.org/js/nms-map-handlers.js
+++ b/web/nms.gathering.org/js/nms-map-handlers.js
@@ -84,7 +84,7 @@ function uplinkUpdater()
 		return;
 	if (!nmsData.switchstate.switches)
 		return;
-	for (sw in nmsData.switches.switches) {
+	for (var sw in nmsData.switches.switches) {
 		var uplinks=0;
 		if (nmsData.switchstate.switches[sw] == undefined || nmsData.switchstate.switches[sw].uplinks == undefined) {
 			uplinks=0;
@@ -139,9 +139,9 @@ function trafficUpdater()
 {
 	if (!nms.switches_now["switches"])
 		return;
-	for (sw in nms.switches_now["switches"]) {
+	for (var sw in nms.switches_now["switches"]) {
 		var speed = 0;
-		for (port in nms.switches_now["switches"][sw]["ports"]) {
+		for (var port in nms.switches_now["switches"][sw]["ports"]) {
 			if (/ge-0\/0\/44$/.exec(port) ||
 			    /ge-0\/0\/45$/.exec(port) ||
 			    /ge-0\/0\/46$/.exec(port) ||
@@ -177,9 +177,9 @@ function trafficTotUpdater()
 {
 	if (!nms.switches_now["switches"])
 		return;
-	for (sw in nms.switches_now["switches"]) {
+	for (var sw in nms.switches_now["switches"]) {
 		var speed = 0;
-		for (port in nms.switches_now["switches"][sw]["ports"]) {
+		for (var port in nms.switches_now["switches"][sw]["ports"]) {
 			if (!nms.switches_then["switches"][sw] ||
 			    !nms.switches_then["switches"][sw]["ports"] ||
 			    !nms.switches_then["switches"][sw]["ports"][port])
@@ -220,14 +220,14 @@ function tempUpdater()
 	if(!nmsData.switches)
 		return;
 
-	for (sw in nmsData.switches["switches"]) {
+	for ( var sw in nmsData.switches["switches"]) {
 		var t = "white";
 		var temp = "";
 
 		if(!nmsData.snmp || !nmsData.snmp.snmp[sw]["misc"] || !nmsData.snmp.snmp[sw]["misc"]["enterprises.2636.3.1.13.1.7.7.1.0.0"])
 			continue;
 
-		tempObj = nmsData.snmp.snmp[sw]["misc"]["enterprises.2636.3.1.13.1.7.7.1.0.0"];
+		var tempObj = nmsData.snmp.snmp[sw]["misc"]["enterprises.2636.3.1.13.1.7.7.1.0.0"];
 		Object.keys(tempObj).forEach(function (key) {
 			if(key == "") {
 				temp = tempObj[key] + "°C";
@@ -259,6 +259,7 @@ function pingUpdater()
 	}
 	for (var sw in nmsData.switches.switches) {
 		try {
+            var c;
 			if (nmsData.ping.switches[sw].age > 0) {
 				c = red;
 			} else {
diff --git a/web/nms.gathering.org/js/nms-map.js b/web/nms.gathering.org/js/nms-map.js
index b74626a..c6bad49 100644
--- a/web/nms.gathering.org/js/nms-map.js
+++ b/web/nms.gathering.org/js/nms-map.js
@@ -48,19 +48,20 @@ var nmsMap = nmsMap || {
 	_color: { },
 	_highlight: { },
 	_c: {}
-}
+};
 
 nmsMap._loadEvent = function(e) {
 	nmsMap._init = false;
 	nmsMap._drawAllSwitches();
-}
+};
+
 nmsMap.init = function() {
 	this._initContexts();
 	this._init = true;
 	nmsData.addHandler("switches","nmsMap",function(){nmsMap._resizeEvent();});
 	window.addEventListener('resize',nmsMap._resizeEvent,true);
 	window.addEventListener('load',nmsMap._loadEvent,true);
-}
+};
 
 nmsMap.setSwitchColor = function(sw, color) {
 	if (this._color[sw] != color) {
@@ -70,7 +71,7 @@ nmsMap.setSwitchColor = function(sw, color) {
 	} else {
 		this.stats.colorSame++;
 	}
-}
+};
 
 nmsMap.setSwitchHighlight = function(sw, highlight) {
 	if(highlight)
@@ -80,12 +81,12 @@ nmsMap.setSwitchHighlight = function(sw, highlight) {
 		this._drawSwitch(sw);
 		this.stats.highlightChange++;
 	}
-}
+};
 
 nmsMap.removeAllSwitchHighlights = function() {
 	for(var sw in this._highlight)
 		this.setSwitchHighlight(sw,false);
-}
+};
 
 nmsMap.reset = function() {
 	for (var sw in this._color) {
@@ -94,7 +95,7 @@ nmsMap.reset = function() {
 	for (var sw in this._info) {
 		nmsMap.setSwitchInfo(sw, undefined);
 	}
-}
+};
 
 nmsMap.setSwitchInfo = function(sw,info) {
 	if (this._info[sw] != info) {
@@ -104,19 +105,19 @@ nmsMap.setSwitchInfo = function(sw,info) {
 	} else {
 		this.stats.switchInfoSame++;
 	}
-}
+};
 
 nmsMap._initContext = function(name) {
 	this._c[name] = {};
 	this._c[name].c = document.getElementById(name + "Canvas");
 	this._c[name].ctx = this._c[name].c.getContext('2d');
-}
+};
 
 nmsMap._initContexts = function() {
 	for (var context in this.contexts) {
 		this._initContext(this.contexts[context]);
 	}
-}
+};
 
 nmsMap._resizeEvent = function() {
 	var width = window.innerWidth - nmsMap._c.bg.c.offsetLeft;
@@ -149,7 +150,7 @@ nmsMap._resizeEvent = function() {
 		nmsMap.drawNow();
 		nmsMap.stats.resizeEvents++;
 	}
-}
+};
 
 /*
  * Draw current time-window
@@ -177,10 +178,10 @@ nmsMap.drawNow = function ()
 	ctx.fillStyle = "white";
 	ctx.strokeStyle = "black";
 	ctx.lineWidth = nms.fontLineFactor;
-	ctx.strokeText(now, 0 + this._settings.textMargin, 25);
-	ctx.fillText(now, 0 + this._settings.textMargin, 25);
+	ctx.strokeText(now, this._settings.textMargin, 25);
+	ctx.fillText(now, this._settings.textMargin, 25);
 	ctx.restore();
-}
+};
 
 nmsMap.setNightMode = function(toggle) {
 	if (this._nightmode == toggle)
@@ -196,14 +197,14 @@ nmsMap.setNightMode = function(toggle) {
 		this._c.blur.c.style.display = "";
 	}
 	nmsMap._drawBG();
-}
+};
 
 nmsMap._drawBG = function() {
 	var imageObj = document.getElementById('source');
 	this._c.bg.ctx.drawImage(imageObj, 0, 0, nmsMap._canvas.width, nmsMap._canvas.height);
 	if(this._nightmode)
 		nmsMap._invertBG();
-}
+};
 
 nmsMap._invertBG = function() {
 	var imageData = this._c.bg.ctx.getImageData(0, 0, nmsMap._canvas.width, nmsMap._canvas.height);
@@ -215,7 +216,7 @@ nmsMap._invertBG = function() {
 		data[i + 2] = 255 - data[i + 2];
 	}
 	this._c.bg.ctx.putImageData(imageData, 0, 0);
-}
+};
 
 nmsMap._getBox = function(sw) {
 	var box = nmsData.switches.switches[sw]['placement'];
@@ -224,7 +225,7 @@ nmsMap._getBox = function(sw) {
 	box.width = parseInt(box.width);
 	box.height = parseInt(box.height);
 	return box;
-}
+};
 
 nmsMap._drawSwitchBlur = function(sw)
 {
@@ -238,7 +239,8 @@ nmsMap._drawSwitchBlur = function(sw)
 	this._c.blur.ctx.scale(this.scale, this.scale); // FIXME
 	this._c.blur.ctx.fillRect(box['x'],box['y'],box['width'],box['height']);
 	this._c.blur.ctx.restore();
-}
+};
+
 nmsMap._drawSwitch = function(sw)
 {
 	// XXX: If a handler sets a color before switches are loaded... The
@@ -257,7 +259,7 @@ nmsMap._drawSwitch = function(sw)
 	this._drawBox(this._c.switch.ctx, box['x'],box['y'],box['width'],box['height']);
 	this._c.switch.ctx.shadowBlur = 0;
 	this._drawText(this._c.text.ctx, sw,box);
-}
+};
 
 nmsMap._drawSwitchInfo = function(sw) {
 	var box = this._getBox(sw);
@@ -266,14 +268,14 @@ nmsMap._drawSwitchInfo = function(sw) {
 	} else {
 		this._drawText(this._c.textInfo.ctx, this._info[sw], box, "right");
 	}
-}
+};
 
 nmsMap._clearBox = function(ctx,box) {
 	ctx.save();
 	ctx.scale(this.scale,this.scale);
 	ctx.clearRect(box['x'], box['y'], box['width'], box['height']);
 	ctx.restore();
-}
+};
 
 nmsMap._drawText = function(ctx, text, box, align) {
 	var rotate = false;
@@ -309,7 +311,7 @@ nmsMap._drawText = function(ctx, text, box, align) {
 	ctx.strokeText(text, 0, 0);
 	ctx.fillText(text, 0, 0);
 	ctx.restore();
-}
+};
 
 nmsMap._drawAllSwitches = function() {
 	if (nmsData.switches == undefined) {
@@ -321,7 +323,7 @@ nmsMap._drawAllSwitches = function() {
 	}
 	if (this._nightmode)
 		this._drawAllBlur();
-}
+};
 
 nmsMap._drawAllBlur = function() {
 	if (nmsMap._blurDrawn == true)
@@ -330,7 +332,7 @@ nmsMap._drawAllBlur = function() {
 	for (var sw in nmsData.switches.switches) {
 		nmsMap._drawSwitchBlur(sw);
 	}
-}
+};
 
 nmsMap._drawBox = function(ctx, x, y, boxw, boxh) {
 	ctx.save();
@@ -340,12 +342,12 @@ nmsMap._drawBox = function(ctx, x, y, boxw, boxh) {
 	ctx.strokeStyle = "#000000";
 	ctx.strokeRect(x,y, boxw, boxh);
 	ctx.restore();
-}
+};
 
 nmsMap._connectSwitches = function(sw1, sw2, color1, color2) {
 	nmsMap._connectBoxes(this._getBox(sw1), this._getBox(sw2),
 			     color1, color2);
-}
+};
 
 /*
  * Draw a line between two boxes, with a gradient going from color1 to
@@ -372,13 +374,13 @@ nmsMap._connectBoxes = function(box1, box2,color1, color2) {
 	ctx.lineWidth = 5;
 	ctx.stroke();
 	ctx.restore();
-}
+};
 
 nmsMap.moveSet = function(toggle) {
 	nmsMap._moveInProgress = toggle;
 	if (!toggle)
 		nmsMap._moveStopListen();
-}
+};
 
 /*
  * onclick handler for the canvas.
@@ -395,7 +397,7 @@ nmsMap.canvasClick = function(e)
 			nmsInfoBox.click(sw);
 		}
 	}
-}
+};
 
 nmsMap._clearOld = function(box) {
 	if (box) {
@@ -405,7 +407,7 @@ nmsMap._clearOld = function(box) {
 		nmsMap._c.top.ctx.clearRect(box['x'] - 5, box['y'] - 5, box['width'] + 10, box['height'] + 10);
 		nmsMap._c.top.ctx.restore();
 	}
-}
+};
 
 nmsMap._moveMove = function(e) {
 	nmsMap._moveX = (e.pageX - e.target.offsetLeft) / nmsMap.scale;
@@ -423,13 +425,13 @@ nmsMap._moveMove = function(e) {
 	nmsMap._c.top.ctx.fillStyle = "red";
 	nmsMap._drawBox(nmsMap._c.top.ctx, box['x'], box['y'], box['width'], box['height']);
 	nmsMap._c.top.ctx.restore();
-}
+};
 
 nmsMap._moveSubmit = function() {
 	var data = {
 		sysname: nmsMap._moving,
 		placement: nmsMap._moveOldBox
-	}
+	};
 	var myData = JSON.stringify([data]);
 	$.ajax({
 		type: "POST",
@@ -440,11 +442,12 @@ nmsMap._moveSubmit = function() {
 			nmsData.invalidate("switches");
 		}
 	});
-}
+};
+
 nmsMap._moveStopListen = function() {
 	nmsMap._c.input.c.removeEventListener('mousemove',nmsMap._moveMove, true);
 	nmsMap._c.input.c.removeEventListener('mouseup',nmsMap._moveDone, true);
-}
+};
 
 nmsMap._moveDone = function(e) {
 	nmsMap._moveStopListen();
@@ -453,7 +456,7 @@ nmsMap._moveDone = function(e) {
 	}
 	nmsMap._moveSubmit();
 	nmsMap._clearOld(nmsMap._moveOldBox);
-}
+};
 
 nmsMap._moveStart = function(sw, e)
 {
@@ -464,7 +467,7 @@ nmsMap._moveStart = function(sw, e)
 	nmsMap._moveBox = nmsData.switches.switches[sw].placement;
 	nmsMap._c.input.c.addEventListener('mousemove',nmsMap._moveMove,true);
 	nmsMap._c.input.c.addEventListener('mouseup',nmsMap._moveDone,true);
-}
+};
 
 
 /*
diff --git a/web/nms.gathering.org/js/nms.js b/web/nms.gathering.org/js/nms.js
index 8268732..8f93367 100644
--- a/web/nms.gathering.org/js/nms.js
+++ b/web/nms.gathering.org/js/nms.js
@@ -160,27 +160,27 @@ function stringToEpoch(t)
 function epochToString(t)
 {
 	// Adjust for timezone when converting from epoch (UTC) to string (local)
-	var d = new Date(parseInt(t) * parseInt(1000));
-	var timezoneOffset = d.getTimezoneOffset() * -60;
+	var date = new Date(parseInt(t) * parseInt(1000));
+	var timezoneOffset = date.getTimezoneOffset() * -60;
 	t = t - timezoneOffset;
 
-	var d = new Date(parseInt(t) * parseInt(1000));
-	var str = d.getFullYear() + "-";
-	if (parseInt(d.getMonth()) < 9)
+    date = new Date(parseInt(t) * parseInt(1000));
+	var str = date.getFullYear() + "-";
+	if (parseInt(date.getMonth()) < 9)
 		str += "0";
-	str += (parseInt(d.getMonth())+1) + "-";
-	if (d.getDate() < 10)
+	str += (parseInt(date.getMonth())+1) + "-";
+	if (date.getDate() < 10)
 		str += "0";
-	str += d.getDate() + "T";
-	if (d.getHours() < 10)
+	str += date.getDate() + "T";
+	if (date.getHours() < 10)
 		str += "0";
-	str += d.getHours() + ":";
-	if (d.getMinutes() < 10)
+	str += date.getHours() + ":";
+	if (date.getMinutes() < 10)
 		str += "0";
-	str += d.getMinutes() + ":";
-	if (d.getSeconds() < 10)
+	str += date.getMinutes() + ":";
+	if (date.getSeconds() < 10)
 		str += "0";
-	str += d.getSeconds();
+	str += date.getSeconds();
 
 	return str;
 }
@@ -409,11 +409,7 @@ function addComment(sw,comment)
  */
 function isIn(box, x, y)
 {
-	if ((x >= box.x) && (x <= (box.x + box.width)) && (y >= box.y) && (y <= (box.y + box.height))) {
-		return true;
-	}
-	return false;
-
+    return ((x >= box.x) && (x <= (box.x + box.width)) && (y >= box.y) && (y <= (box.y + box.height)));
 }
 
 /*
@@ -486,7 +482,6 @@ function initNMS() {
 }
 
 function detectHandler() {
-	var url = document.URL;
 	for (var i in handlers) {
 		if (('#' + handlers[i].tag) == document.location.hash) {
 			setUpdater(handlers[i]);