hgweb: fix jshint issues in mercurial.js
authorAnton Shestakov <av6@dwimlabs.net>
Sat, 11 Nov 2017 14:14:38 +0800
changeset 35042 5f82e26cbed1
parent 35041 6370ed6516a6
child 35043 5d4369079c86
hgweb: fix jshint issues in mercurial.js Everything is pretty self-explanatory except the last hunk, where jshint complains: "Misleading line break before '||'; readers may interpret this as an expression boundary." There is a tweakable called "laxbreak" that allows line breaks before operators, but I think it's fine to simply join this one line and avoid extra config for now (we can cook up and add a sensible .jshintrc later).
mercurial/templates/static/mercurial.js
--- a/mercurial/templates/static/mercurial.js	Tue Oct 17 15:55:40 2017 +0200
+++ b/mercurial/templates/static/mercurial.js	Sat Nov 11 14:14:38 2017 +0800
@@ -42,13 +42,13 @@
 		this.columns = 0;
 		document.getElementById('nodebgs').innerHTML = '';
 		document.getElementById('graphnodes').innerHTML = '';
-	}
+	};
 
 	this.scale = function(height) {
 		this.bg_height = height;
 		this.box_size = Math.floor(this.bg_height / 1.2);
 		this.cell_height = this.box_size;
-	}
+	};
 
 	this.setColor = function(color, bg, fg) {
 
@@ -78,7 +78,7 @@
 		this.ctx.fillStyle = s;
 		return s;
 
-	}
+	};
 
 	this.edge = function(x0, y0, x1, y1, color, width) {
 
@@ -90,7 +90,7 @@
 		this.ctx.lineTo(x1, y1);
 		this.ctx.stroke();
 
-	}
+	};
 
 	this.render = function(data) {
 
@@ -127,7 +127,7 @@
 				}
 
 				if (start == this.columns && start > end) {
-					var fold = true;
+					fold = true;
 				}
 
 				x0 = this.cell[0] + this.box_size * start + this.box_size / 2;
@@ -142,8 +142,8 @@
 
 			// Draw the revision node in the right column
 
-			column = node[0]
-			color = node[1]
+			column = node[0];
+			color = node[1];
 
 			radius = this.box_size / 8;
 			x = this.cell[0] + this.box_size * column + this.box_size / 2;
@@ -159,7 +159,7 @@
 		document.getElementById('nodebgs').innerHTML += backgrounds;
 		document.getElementById('graphnodes').innerHTML += nodedata;
 
-	}
+	};
 
 }
 
@@ -228,7 +228,7 @@
 			return shortdate(once);
 		}
 
-		for (unit in scales){
+		for (var unit in scales){
 			var s = scales[unit];
 			var n = Math.floor(delta / s);
 			if ((n >= 2) || (s == 1)){
@@ -273,7 +273,8 @@
 
     function setLinewrap(enable) {
         var nodes = document.getElementsByClassName('sourcelines');
-        for (var i = 0; i < nodes.length; i++) {
+        var i;
+        for (i = 0; i < nodes.length; i++) {
             if (enable) {
                 nodes[i].classList.add('wrap');
             } else {
@@ -282,7 +283,7 @@
         }
 
         var links = document.getElementsByClassName('linewraplink');
-        for (var i = 0; i < links.length; i++) {
+        for (i = 0; i < links.length; i++) {
             links[i].innerHTML = enable ? 'on' : 'off';
         }
     }
@@ -354,8 +355,7 @@
 
         var scrollHeight = document.documentElement.scrollHeight;
         var clientHeight = document.documentElement.clientHeight;
-        var scrollTop = document.body.scrollTop
-            || document.documentElement.scrollTop;
+        var scrollTop = document.body.scrollTop || document.documentElement.scrollTop;
 
         if (scrollHeight - (scrollTop + clientHeight) < 50) {
             updateInitiated = true;