Fixed two jslint warnings

zdennis · Aug 15, 2011 · 27d220d · 27d220d
1 parent 4ee9438
commit 27d220d
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/js/adapters/prototype-adapter.src.js b/js/adapters/prototype-adapter.src.js
@@ -14,7 +14,6 @@
  */
 
 // JSLint options:
-/*jslint forin: true */
 /*global Effect, Class, Highcharts, Event, $, $A */
 
 // Adapter interface between prototype and the Highcarts charting library
@@ -186,7 +185,7 @@ return {
 				event = HighchartsAdapter.addNS(event);
 			}
 			$(el).stopObserving(event, handler);
-		} if (window == el) {
+		} if (window === el) {
 			Event.stopObserving(el, event, handler);
 		} else {
 			HighchartsAdapter._extend(el);

diff --git a/js/highcharts.src.js b/js/highcharts.src.js
@@ -10,7 +10,6 @@
  */
 
 // JSLint options:
-/*jslint forin: true */
 /*global document, window, navigator, setInterval, clearInterval, clearTimeout, setTimeout, location, jQuery, $ */
 
 (function () {
@@ -8082,7 +8081,9 @@ function Chart(options, callback) {
 		var ONREADYSTATECHANGE = 'onreadystatechange',
 			COMPLETE = 'complete';
 		// Note: in spite of JSLint's complaints, win == win.top is required
+		/*jslint eqeq: true*/
 		if (!hasSVG && win == win.top && doc.readyState !== COMPLETE) {
+		/*jslint eqeq: false*/
 			doc.attachEvent(ONREADYSTATECHANGE, function () {
 				doc.detachEvent(ONREADYSTATECHANGE, firstRender);
 				if (doc.readyState === COMPLETE) {

diff --git a/js/parts/Chart.js b/js/parts/Chart.js
@@ -4114,7 +4114,9 @@ function Chart(options, callback) {
 		var ONREADYSTATECHANGE = 'onreadystatechange',
 			COMPLETE = 'complete';
 		// Note: in spite of JSLint's complaints, win == win.top is required
+		/*jslint eqeq: true*/
 		if (!hasSVG && win == win.top && doc.readyState !== COMPLETE) {
+		/*jslint eqeq: false*/
 			doc.attachEvent(ONREADYSTATECHANGE, function () {
 				doc.detachEvent(ONREADYSTATECHANGE, firstRender);
 				if (doc.readyState === COMPLETE) {

diff --git a/js/parts/Intro.js b/js/parts/Intro.js
@@ -10,7 +10,6 @@
  */
 
 // JSLint options:
-/*jslint forin: true */
 /*global document, window, navigator, setInterval, clearInterval, clearTimeout, setTimeout, location, jQuery, $ */
 
 (function () {