Small changes to code style

Mostly re-indentation or similar.

Author: anselmh

.jshintrc

@@ -1,14 +1,38 @@
 {
+  "node": true,
+  "browser": true,
+  "esnext": true,
+  "bitwise": true,
   "curly": true,
   "eqeqeq": true,
   "immed": true,
   "latedef": true,
   "newcap": true,
   "noarg": true,
-  "sub": true,
+  "regexp": true,
   "undef": true,
+  "trailing": true,
+  "smarttabs": true,
+  "sub": true,
   "boss": true,
   "eqnull": true,
-  "node": true,
-  "phantom": true
+  "phantom": true,
+
+  "globals": {
+    "jQuery": true,
+    "$": true,
+    "Modernizr": true,
+
+    "module": true,
+    "require": true,
+    "requirejs": true,
+    "define": true,
+
+    "sinon": true,
+    "describe": true,
+    "it": true,
+    "expect": true,
+    "beforeEach": true,
+    "afterEach": true
+  }
 }

Gruntfile.js

@@ -1,13 +1,16 @@
 /*
  * grunt-phantomcss
- * https://github.com/chrisgladd/grunt-phantomcss
+ * https://github.com/anselmh/grunt-phantomcss
  *
  * Copyright (c) 2013 Chris Gladd
+ * Copyright (c) since 2014 Anselm Hannemann
+ *
  * Licensed under the MIT license.
  */
 
 'use strict';
-module.exports = function(grunt) {
+
+module.exports = function (grunt) {
     // Project configuration.
     grunt.initConfig({
         jshint: {
@@ -22,7 +25,7 @@ module.exports = function(grunt) {
             },
         },
 
-        // Before generating any new files, remove any 
+        // Before generating any new files, remove any
         // previously-created files.
         clean: {
             tests: ['tmp'],

phantomjs/runner.js

@@ -1,65 +1,80 @@
+/*
+* grunt-phantomcss
+* https://github.com/anselmh/grunt-phantomcss
+*
+* Copyright (c) 2013 Chris Gladd
+* Copyright (c) since 2014 Anselm Hannemann
+*
+* Licensed under the MIT license.
+*/
+
+'use-strict';
+
+// Get node fileSystem module and define the separator module
 var fs = require('fs');
 var s = fs.separator;
 
 // Parse arguments passed in from the grunt task
 var args = JSON.parse(phantom.args[0]);
 
+// Get viewport arguments (width | height)
 var viewportSize = {
-    width: args.viewportSize[0],
-    height: args.viewportSize[1]
+  width: args.viewportSize[0],
+  height: args.viewportSize[1]
 };
 
 // Messages are sent to the parent by appending them to the tempfile
 var sendMessage = function() {
-    fs.write(args.tempFile, JSON.stringify(Array.prototype.slice.call(arguments)) + '\n', 'a');
+  fs.write(args.tempFile, JSON.stringify(Array.prototype.slice.call(arguments)) + '\n', 'a');
 };
 
 // Initialise CasperJs
 var phantomCSSPath = args.phantomCSSPath;
+
 phantom.casperPath = phantomCSSPath+s+'CasperJs';
 phantom.injectJs(phantom.casperPath+s+'bin'+s+'bootstrap.js');
 
 var casper = require('casper').create({
-    viewportSize: viewportSize,
-    logLevel: args.logLevel,
-    verbose: true
+  viewportSize: viewportSize,
+  logLevel: args.logLevel,
+  verbose: true
 });
 
 // Require and initialise PhantomCSS module
 var phantomcss = require(phantomCSSPath+s+'phantomcss.js');
 
 phantomcss.init({
-    screenshotRoot: args.screenshots,
-    failedComparisonsRoot: args.failures,
-    libraryRoot: phantomCSSPath, // Give absolute path, otherwise PhantomCSS fails
-    mismatchTolerance: args.mismatchTolerance,
+  screenshotRoot: args.screenshots,
+  failedComparisonsRoot: args.failures,
+  libraryRoot: phantomCSSPath, // Give absolute path, otherwise PhantomCSS fails
+  mismatchTolerance: args.mismatchTolerance, // defaults to 0.05
 
-    onFail: function(test) {
-        sendMessage('onFail', test);
-    },
-    onPass: function(test) {
-        sendMessage('onPass', test);
-    },
-    onTimeout: function(test) {
-        sendMessage('onTimeout', test);
-    },
-    onComplete: function(allTests, noOfFails, noOfErrors) {
-        sendMessage('onComplete', allTests, noOfFails, noOfErrors);
-    }
+  onFail: function(test) {
+    sendMessage('onFail', test);
+  },
+  onPass: function(test) {
+    sendMessage('onPass', test);
+  },
+  onTimeout: function(test) {
+    sendMessage('onTimeout', test);
+  },
+  onComplete: function(allTests, noOfFails, noOfErrors) {
+    sendMessage('onComplete', allTests, noOfFails, noOfErrors);
+  }
 });
 
 // Run the test scenarios
 args.test.forEach(function(testSuite) {
-    require(testSuite);
+  require(testSuite);
 });
 
 // End tests and compare screenshots
 casper.then(function() {
-    phantomcss.compareAll();
+  phantomcss.compareAll();
 })
 .then(function() {
-    casper.test.done();
+  casper.test.done();
 })
 .run(function() {
-    phantom.exit();
+  phantom.exit();
 });

tasks/phantomcss.js

@@ -1,10 +1,13 @@
-/**
+/*
  * grunt-phantomcss
- * https://github.com/chrisgladd/grunt-phantomcss
+ * https://github.com/anselmh/grunt-phantomcss
  *
  * Copyright (c) 2013 Chris Gladd
+ * Copyright (c) since 2014 Anselm Hannemann
+ *
  * Licensed under the MIT license.
  */
+
 'use strict';
 
 var path = require('path');
@@ -17,12 +20,13 @@ module.exports = function(grunt) {
     grunt.registerMultiTask('phantomcss', 'CSS Regression Testing', function() {
         var done = this.async();
 
+        // Variable object to set default values for options
         var options = this.options({
             screenshots: 'screenshots',
             results: 'results',
             viewportSize: [1280, 800],
             mismatchTolerance: 0.05,
-            logLevel: 'error'
+            logLevel: 'error' // notice | warn | error
         });
 
         // Timeout ID for message checking loop