Make jshint quiet now

js/controller/AnimatedPreviewController.js

@@ -57,5 +57,5 @@
     ns.AnimatedPreviewController.prototype.updateDPI_ = function () {
         this.dpi = this.calculateDPI_();
         this.renderer.updateDPI(this.dpi);
-    }
+    };
 })();

js/controller/DrawingController.js

@@ -173,7 +173,7 @@
      */
     ns.DrawingController.prototype.wrapEvtInfo_ = function (event) {
       var evtInfo = {};
-      if (event.button == 0) {
+      if (event.button === 0) {
         evtInfo.button = Constants.LEFT_BUTTON;
       } else if (event.button == 2) {
         evtInfo.button = Constants.RIGHT_BUTTON;

js/controller/PreviewFilmController.js

@@ -53,7 +53,7 @@
         }
         this.container.append(this.createInterstitialTile_(frameCount));
 
-        var needDragndropBehavior = !!(frameCount > 1);
+        var needDragndropBehavior = (frameCount > 1) ? true : false;
         if(needDragndropBehavior) {
             this.initDragndropBehavior_();
         }
@@ -64,7 +64,7 @@
      */
     ns.PreviewFilmController.prototype.createInterstitialTile_ = function (tileNumber) {
         var interstitialTile = document.createElement("div");
-        interstitialTile.className = "interstitial-tile"
+        interstitialTile.className = "interstitial-tile";
         interstitialTile.setAttribute("data-tile-type", "interstitial");
         interstitialTile.setAttribute("data-inject-drop-tile-at", tileNumber);
 
@@ -206,7 +206,7 @@
             canvasPreviewDeleteAction.setAttribute('rel', 'tooltip');
             canvasPreviewDeleteAction.setAttribute('data-placement', 'right');
             canvasPreviewDeleteAction.setAttribute('title', 'Delete this frame');
-            canvasPreviewDeleteAction.className = "tile-action delete-frame-action"
+            canvasPreviewDeleteAction.className = "tile-action delete-frame-action";
             canvasPreviewDeleteAction.addEventListener('click', this.onDeleteButtonClick_.bind(this, tileNumber));
             previewTileRoot.appendChild(canvasPreviewDeleteAction);
             actionContainer.appendChild(canvasPreviewDeleteAction);

js/drawingtools/BaseTool.js

@@ -46,27 +46,27 @@
 		
 		var pixels = [];
 		var dx = Math.abs(x1-x0);
-	   	var dy = Math.abs(y1-y0);
-	   	var sx = (x0 < x1) ? 1 : -1;
-	   	var sy = (y0 < y1) ? 1 : -1;
-	   	var err = dx-dy;
+		var dy = Math.abs(y1-y0);
+		var sx = (x0 < x1) ? 1 : -1;
+		var sy = (y0 < y1) ? 1 : -1;
+		var err = dx-dy;
 
-	   	while(true){
+		while(true){
 
-	    	// Do what you need to for this
-	    	pixels.push({"col": x0, "row": y0});
+			// Do what you need to for this
+			pixels.push({"col": x0, "row": y0});
 
-		    if ((x0==x1) && (y0==y1)) break;
-		    var e2 = 2*err;
-		    if (e2>-dy){
-		    	err -= dy;
-		    	x0  += sx;
-		    }
-		    if (e2 < dx) {
-		    	err += dx;
-		    	y0  += sy;
-		    }
-	    }
-	    return pixels;
-     };
+			if ((x0==x1) && (y0==y1)) break;
+			var e2 = 2*err;
+			if (e2>-dy){
+				err -= dy;
+				x0  += sx;
+			}
+			if (e2 < dx) {
+				err += dx;
+				y0  += sy;
+			}
+		}
+		return pixels;
+	};
 })();

js/drawingtools/Move.js

@@ -7,7 +7,7 @@
 	var ns = $.namespace("pskl.drawingtools");
 
 	ns.Move = function() {
-		this.toolId = "tool-move"
+		this.toolId = "tool-move";
 		this.helpText = "Move tool";
 		
 		// Stroke's first point coordinates (set in applyToolAt)
@@ -40,7 +40,7 @@
 				} else {
 					color = Constants.TRANSPARENT_COLOR;
 				}
-				frame.setPixel(col, row, color)
+				frame.setPixel(col, row, color);
 			}
 		}
 	};

js/drawingtools/VerticalMirrorPen.js

@@ -5,9 +5,9 @@
 		this.toolId = "tool-vertical-mirror-pen";
 		this.helpText = "vertical mirror pen tool";
 
-		this.swap = null
+		this.swap = null;
 		this.mirroredPreviousCol = null;
-	    this.mirroredPreviousRow = null;
+		this.mirroredPreviousRow = null;
 	};
 
 	pskl.utils.inherit(ns.VerticalMirrorPen, ns.SimplePen);

js/drawingtools/selectiontools/BaseSelect.js

@@ -103,7 +103,7 @@
 				} else {
 					color = Constants.TRANSPARENT_COLOR;
 				}
-				overlayFrame.setPixel(col, row, color)
+				overlayFrame.setPixel(col, row, color);
 			}
 		}
 	};

js/drawingtools/selectiontools/RectangleSelect.js

@@ -45,7 +45,7 @@
 	 * @override
 	 */
 	ns.RectangleSelect.prototype.onSelectEnd_ = function (col, row, color, frame, overlay) {
-		this.onSelect_(col, row, color, frame, overlay)
+		this.onSelect_(col, row, color, frame, overlay);
 	};
 
 })();

js/model/Frame.js

@@ -8,7 +8,7 @@
 	};
 
 	ns.Frame.createEmpty = function (width, height) {
-    	var pixels = ns.Frame.createEmptyPixelGrid_(width, height);
+		var pixels = ns.Frame.createEmptyPixelGrid_(width, height);
 		return new ns.Frame(pixels);
 	};
 
@@ -20,7 +20,7 @@
 				columnArray.push(Constants.TRANSPARENT_COLOR);
 			}
 			pixels[columnIndex] = columnArray;
-    	}
+		}
 		return pixels;
 	};
 
@@ -36,7 +36,7 @@
 	 * Returns a copy of the pixels used by the frame
 	 */
 	ns.Frame.prototype.getPixels = function () {
-		return this.clonePixels_(this.pixels)
+		return this.clonePixels_(this.pixels);
 	};
 
 	/**

js/model/FrameSheet.js

@@ -70,9 +70,9 @@
 	 */
 	ns.FrameSheet.prototype.deserialize = function (serialized) {
 		try {
-		 	this.load(JSON.parse(serialized));
+			this.load(JSON.parse(serialized));
 		} catch (e) {
-	 		throw "Could not load serialized framesheet : " + e.message
+			throw "Could not load serialized framesheet : " + e.message;
 		}	
 	};
 
@@ -83,20 +83,20 @@
 	 * @param {String} serialized
 	 */
 	ns.FrameSheet.prototype.load = function (framesheet) {
-	 	this.frames = [];
-	 	for (var i = 0 ; i < framesheet.length ; i++) {
-	 		var frameCfg = framesheet[i];
-	 		this.addFrame(new ns.Frame(frameCfg));
-	 	}
+		this.frames = [];
+		for (var i = 0 ; i < framesheet.length ; i++) {
+			var frameCfg = framesheet[i];
+			this.addFrame(new ns.Frame(frameCfg));
+		}
 
-	 	if (this.hasFrameAtIndex(0)) {
-		 	this.height = this.getFrameByIndex(0).getHeight();
-		 	this.width = this.getFrameByIndex(0).getWidth();
-		 	this.setCurrentFrameIndex(0);
-	 		$.publish(Events.FRAME_SIZE_CHANGED);
-	 	}
+		if (this.hasFrameAtIndex(0)) {
+			this.height = this.getFrameByIndex(0).getHeight();
+			this.width = this.getFrameByIndex(0).getWidth();
+			this.setCurrentFrameIndex(0);
+			$.publish(Events.FRAME_SIZE_CHANGED);
+		}
 
-	 	$.publish(Events.FRAMESHEET_RESET);
+		$.publish(Events.FRAMESHEET_RESET);
 	};
 
 	
@@ -148,8 +148,8 @@
 
 	ns.FrameSheet.prototype.swapFrames = function(indexFrame1, indexFrame2) {
 		if(isNaN(indexFrame1) || isNaN(indexFrame1) ||
-		   (!this.hasFrameAtIndex(indexFrame1) && !this.hasFrameAtIndex(indexFrame2))) {
-		 	throw "Bad indexes for swapFrames Framesheet function.";
+			(!this.hasFrameAtIndex(indexFrame1) && !this.hasFrameAtIndex(indexFrame2))) {
+			throw "Bad indexes for swapFrames Framesheet function.";
 		}
 		if(indexFrame1 == indexFrame2) {
 			return;

js/rendering/CanvasRenderer.js

@@ -33,4 +33,4 @@
 		var height = this.frame.getHeight() * this.dpi;
 		return pskl.CanvasUtils.createCanvas(width, height);
 	};
-})()
+})();

js/rendering/DrawingLoop.js

@@ -50,9 +50,12 @@
 	};
 
 	ns.DrawingLoop.prototype.getRequestAnimationFrameShim_ = function () {
-		var requestAnimationFrame = window.requestAnimationFrame || window.mozRequestAnimationFrame ||  
-		window.webkitRequestAnimationFrame || window.msRequestAnimationFrame || function (callback) {window.setTimeout(callback, 1000/60)};
+		var requestAnimationFrame = window.requestAnimationFrame ||
+									window.mozRequestAnimationFrame ||  
+									window.webkitRequestAnimationFrame || 
+									window.msRequestAnimationFrame || 
+									function (callback) { window.setTimeout(callback, 1000/60); };
 
 		return requestAnimationFrame;
-	}
-})()
+	};
+})();

js/rendering/FrameRenderer.js

@@ -8,7 +8,7 @@
 		};
 		renderingOptions = $.extend(true, {}, this.defaultRenderingOptions, renderingOptions);
 
-		if(container == undefined) {
+		if(container === undefined) {
 			throw "Bad FrameRenderer initialization. <container> undefined.";
 		}
 		
@@ -78,15 +78,15 @@
 	/**
 	 * Transform a screen pixel-based coordinate (relative to the top-left corner of the rendered
 	 * frame) into a sprite coordinate in column and row.
-     * @public
-     */
-    ns.FrameRenderer.prototype.convertPixelCoordinatesIntoSpriteCoordinate = function(coords) {
-    	var cellSize = this.dpi + this.gridStrokeWidth;
-    	return {
-    	  "col" : (coords.x - coords.x % cellSize) / cellSize,
-          "row" : (coords.y - coords.y % cellSize) / cellSize
-    	};
-    };
+	 * @public
+	 */
+	ns.FrameRenderer.prototype.convertPixelCoordinatesIntoSpriteCoordinate = function(coords) {
+		var cellSize = this.dpi + this.gridStrokeWidth;
+		return {
+			"col" : (coords.x - coords.x % cellSize) / cellSize,
+			"row" : (coords.y - coords.y % cellSize) / cellSize
+		};
+	};
 
 	/**
 	 * @private
@@ -103,15 +103,15 @@
 		ctx.lineWidth = Constants.GRID_STROKE_WIDTH;
 		ctx.strokeStyle = Constants.GRID_STROKE_COLOR;
 		for(var c=1; c < col; c++) {			
-	        ctx.moveTo(this.getFramePos_(c), 0);
-	        ctx.lineTo(this.getFramePos_(c), height);
-	        ctx.stroke();
+			ctx.moveTo(this.getFramePos_(c), 0);
+			ctx.lineTo(this.getFramePos_(c), height);
+			ctx.stroke();
 		}
 		
 		for(var r=1; r < row; r++) {
-	        ctx.moveTo(0, this.getFramePos_(r));
-	        ctx.lineTo(width, this.getFramePos_(r));
-	        ctx.stroke();
+			ctx.moveTo(0, this.getFramePos_(r));
+			ctx.lineTo(width, this.getFramePos_(r));
+			ctx.stroke();
 		}
 	};
 

js/rendering/SpritesheetRenderer.js

@@ -19,7 +19,7 @@
 	};
 
 	/**
-	 * TODO : (Julz) Mutualize with code already present in FrameRenderer
+	 * TODO(juliandescottes): Mutualize with code already present in FrameRenderer
 	 */
 	ns.SpritesheetRenderer.prototype.drawFrameInCanvas_ = function (frame, canvas, offsetWidth, offsetHeight) {
 		var context = canvas.getContext('2d');
@@ -41,7 +41,7 @@
 			var height = this.framesheet.getHeight();
 			return pskl.CanvasUtils.createCanvas(width, height);
 		} else {
-			throw "Cannot render empty Spritesheet"
+			throw "Cannot render empty Spritesheet";
 		}
 	};
 

js/selection/BaseSelection.js

@@ -17,7 +17,7 @@
 			movedPixel = this.pixels[i];
 			movedPixel.col += colDiff;
 			movedPixel.row += rowDiff;
-			movedPixels.push(movedPixel)
+			movedPixels.push(movedPixel);
 		}
 		this.pixels = movedPixels;
 	};
@@ -27,7 +27,7 @@
 		for(var i=0, l=this.pixels.length; i<l; i++) {
 			pixelWithCopiedColor = this.pixels[i];
 			pixelWithCopiedColor.copiedColor =
-			    targetFrame.getPixel(pixelWithCopiedColor.col, pixelWithCopiedColor.row);
+				targetFrame.getPixel(pixelWithCopiedColor.col, pixelWithCopiedColor.row);
 		}
 		this.hasPastedContent = true;
 	};

js/utils/CanvasUtils.js

@@ -1,22 +1,22 @@
 (function () {
-    var ns = $.namespace("pskl");
+	var ns = $.namespace("pskl");
 
-    ns.CanvasUtils = {
-    	createCanvas : function (width, height, classList) {
-    		var canvas = document.createElement("canvas");
+	ns.CanvasUtils = {
+		createCanvas : function (width, height, classList) {
+			var canvas = document.createElement("canvas");
 			canvas.setAttribute("width", width);
 			canvas.setAttribute("height", height);
 
 			if (typeof classList == "string") {
-    			classList = [classList];
-    		}
-    		if (Array.isArray(classList)) {
-    			for (var i = 0 ; i < classList.length ; i++) {
-    				canvas.classList.add(classList[i]);
+				classList = [classList];
+			}
+			if (Array.isArray(classList)) {
+				for (var i = 0 ; i < classList.length ; i++) {
+					canvas.classList.add(classList[i]);
 				}
-    		}
+			}
 			
 			return canvas;
-    	}
-    };
+		}
+	};
 })();

js/utils/PixelUtils.js

@@ -101,8 +101,9 @@
             var queue = [];
             var dy = [-1, 0, 1, 0];
             var dx = [0, 1, 0, -1];
+            var targetColor;
             try {
-                var targetColor = frame.getPixel(col, row);
+                targetColor = frame.getPixel(col, row);
             } catch(e) {
                 // Frame out of bound exception.
             }
@@ -123,8 +124,8 @@
                 paintedPixels.push({"col": currentItem.col, "row": currentItem.row });
 
                 for (var i = 0; i < 4; i++) {
-                    var nextCol = currentItem.col + dx[i]
-                    var nextRow = currentItem.row + dy[i]
+                    var nextCol = currentItem.col + dx[i];
+                    var nextRow = currentItem.row + dy[i];
                     try {
                         if (frame.containsPixel(nextCol, nextRow)  && frame.getPixel(nextCol, nextRow) == targetColor) {
                             queue.push({"col": nextCol, "row": nextRow });
@@ -136,7 +137,7 @@
 
                 // Security loop breaker:
                 if(loopCount > 10 * cellCount) {
-                    console.log("loop breaker called")
+                    console.log("loop breaker called");
                     break;          
                 }
             }