eslint 0.22.0 → 0.24.1

package/lib/rules/no-extra-bind.js

@@ -1,81 +1,81 @@
-/**
- * @fileoverview Rule to flag unnecessary bind calls
- * @author Bence Dányi <bence@danyi.me>
- * @copyright 2014 Bence Dányi. All rights reserved.
- */
-"use strict";
-
-//------------------------------------------------------------------------------
-// Rule Definition
-//------------------------------------------------------------------------------
-
-module.exports = function(context) {
-
-    var scope = [{
-        depth: -1,
-        found: 0
-    }];
-
-    /**
-     * Get the topmost scope
-     * @returns {Object} The topmost scope
-     */
-    function getTopScope() {
-        return scope[scope.length - 1];
-    }
-
-    /**
-     * Increment the depth of the top scope
-     * @returns {void}
-     */
-    function incrementScopeDepth() {
-        var top = getTopScope();
-        top.depth++;
-    }
-
-    /**
-     * Decrement the depth of the top scope
-     * @returns {void}
-     */
-    function decrementScopeDepth() {
-        var top = getTopScope();
-        top.depth--;
-    }
-
-    return {
-        "CallExpression": function(node) {
-            if (node.arguments.length === 1 &&
-                node.callee.type === "MemberExpression" &&
-                node.callee.property.name === "bind" &&
-                /FunctionExpression$/.test(node.callee.object.type)) {
-                scope.push({
-                    call: node,
-                    depth: -1,
-                    found: 0
-                });
-            }
-        },
-        "CallExpression:exit": function(node) {
-            var top = getTopScope();
-            if (top.call === node && top.found === 0) {
-                context.report(node, "The function binding is unnecessary.");
-                scope.pop();
-            }
-        },
-        "ArrowFunctionExpression": incrementScopeDepth,
-        "ArrowFunctionExpression:exit": decrementScopeDepth,
-        "FunctionExpression": incrementScopeDepth,
-        "FunctionExpression:exit": decrementScopeDepth,
-        "FunctionDeclaration": incrementScopeDepth,
-        "FunctionDeclaration:exit": decrementScopeDepth,
-        "ThisExpression": function() {
-            var top = getTopScope();
-            if (top.depth === 0) {
-                top.found++;
-            }
-        }
-    };
-
-};
-
-module.exports.schema = [];
+/**
+ * @fileoverview Rule to flag unnecessary bind calls
+ * @author Bence Dányi <bence@danyi.me>
+ * @copyright 2014 Bence Dányi. All rights reserved.
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = function(context) {
+
+    var scope = [{
+        depth: -1,
+        found: 0
+    }];
+
+    /**
+     * Get the topmost scope
+     * @returns {Object} The topmost scope
+     */
+    function getTopScope() {
+        return scope[scope.length - 1];
+    }
+
+    /**
+     * Increment the depth of the top scope
+     * @returns {void}
+     */
+    function incrementScopeDepth() {
+        var top = getTopScope();
+        top.depth++;
+    }
+
+    /**
+     * Decrement the depth of the top scope
+     * @returns {void}
+     */
+    function decrementScopeDepth() {
+        var top = getTopScope();
+        top.depth--;
+    }
+
+    return {
+        "CallExpression": function(node) {
+            if (node.arguments.length === 1 &&
+                node.callee.type === "MemberExpression" &&
+                node.callee.property.name === "bind" &&
+                /FunctionExpression$/.test(node.callee.object.type)) {
+                scope.push({
+                    call: node,
+                    depth: -1,
+                    found: 0
+                });
+            }
+        },
+        "CallExpression:exit": function(node) {
+            var top = getTopScope();
+            if (top.call === node && top.found === 0) {
+                context.report(node, "The function binding is unnecessary.");
+                scope.pop();
+            }
+        },
+        "ArrowFunctionExpression": incrementScopeDepth,
+        "ArrowFunctionExpression:exit": decrementScopeDepth,
+        "FunctionExpression": incrementScopeDepth,
+        "FunctionExpression:exit": decrementScopeDepth,
+        "FunctionDeclaration": incrementScopeDepth,
+        "FunctionDeclaration:exit": decrementScopeDepth,
+        "ThisExpression": function() {
+            var top = getTopScope();
+            if (top.depth === 0) {
+                top.found++;
+            }
+        }
+    };
+
+};
+
+module.exports.schema = [];

package/lib/rules/no-extra-boolean-cast.js

@@ -1,71 +1,71 @@
-/**
- * @fileoverview Rule to flag unnecessary double negation in Boolean contexts
- * @author Brandon Mills
- */
-
-"use strict";
-
-//------------------------------------------------------------------------------
-// Rule Definition
-//------------------------------------------------------------------------------
-
-module.exports = function(context) {
-
-    return {
-        "UnaryExpression": function (node) {
-            var ancestors = context.getAncestors(),
-                parent = ancestors.pop(),
-                grandparent = ancestors.pop();
-
-            // Exit early if it's guaranteed not to match
-            if (node.operator !== "!" ||
-                    parent.type !== "UnaryExpression" ||
-                    parent.operator !== "!") {
-                return;
-            }
-
-            // if (<bool>) ...
-            if (grandparent.type === "IfStatement") {
-                context.report(node, "Redundant double negation in an if statement condition.");
-
-            // do ... while (<bool>)
-            } else if (grandparent.type === "DoWhileStatement") {
-                context.report(node, "Redundant double negation in a do while loop condition.");
-
-            // while (<bool>) ...
-            } else if (grandparent.type === "WhileStatement") {
-                context.report(node, "Redundant double negation in a while loop condition.");
-
-            // <bool> ? ... : ...
-            } else if ((grandparent.type === "ConditionalExpression" &&
-                    parent === grandparent.test)) {
-                context.report(node, "Redundant double negation in a ternary condition.");
-
-            // for (...; <bool>; ...) ...
-            } else if ((grandparent.type === "ForStatement" &&
-                    parent === grandparent.test)) {
-                context.report(node, "Redundant double negation in a for loop condition.");
-
-            // !<bool>
-            } else if ((grandparent.type === "UnaryExpression" &&
-                    grandparent.operator === "!")) {
-                context.report(node, "Redundant multiple negation.");
-
-            // Boolean(<bool>)
-            } else if ((grandparent.type === "CallExpression" &&
-                    grandparent.callee.type === "Identifier" &&
-                    grandparent.callee.name === "Boolean")) {
-                context.report(node, "Redundant double negation in call to Boolean().");
-
-            // new Boolean(<bool>)
-            } else if ((grandparent.type === "NewExpression" &&
-                    grandparent.callee.type === "Identifier" &&
-                    grandparent.callee.name === "Boolean")) {
-                context.report(node, "Redundant double negation in Boolean constructor call.");
-            }
-        }
-    };
-
-};
-
-module.exports.schema = [];
+/**
+ * @fileoverview Rule to flag unnecessary double negation in Boolean contexts
+ * @author Brandon Mills
+ */
+
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = function(context) {
+
+    return {
+        "UnaryExpression": function (node) {
+            var ancestors = context.getAncestors(),
+                parent = ancestors.pop(),
+                grandparent = ancestors.pop();
+
+            // Exit early if it's guaranteed not to match
+            if (node.operator !== "!" ||
+                    parent.type !== "UnaryExpression" ||
+                    parent.operator !== "!") {
+                return;
+            }
+
+            // if (<bool>) ...
+            if (grandparent.type === "IfStatement") {
+                context.report(node, "Redundant double negation in an if statement condition.");
+
+            // do ... while (<bool>)
+            } else if (grandparent.type === "DoWhileStatement") {
+                context.report(node, "Redundant double negation in a do while loop condition.");
+
+            // while (<bool>) ...
+            } else if (grandparent.type === "WhileStatement") {
+                context.report(node, "Redundant double negation in a while loop condition.");
+
+            // <bool> ? ... : ...
+            } else if ((grandparent.type === "ConditionalExpression" &&
+                    parent === grandparent.test)) {
+                context.report(node, "Redundant double negation in a ternary condition.");
+
+            // for (...; <bool>; ...) ...
+            } else if ((grandparent.type === "ForStatement" &&
+                    parent === grandparent.test)) {
+                context.report(node, "Redundant double negation in a for loop condition.");
+
+            // !<bool>
+            } else if ((grandparent.type === "UnaryExpression" &&
+                    grandparent.operator === "!")) {
+                context.report(node, "Redundant multiple negation.");
+
+            // Boolean(<bool>)
+            } else if ((grandparent.type === "CallExpression" &&
+                    grandparent.callee.type === "Identifier" &&
+                    grandparent.callee.name === "Boolean")) {
+                context.report(node, "Redundant double negation in call to Boolean().");
+
+            // new Boolean(<bool>)
+            } else if ((grandparent.type === "NewExpression" &&
+                    grandparent.callee.type === "Identifier" &&
+                    grandparent.callee.name === "Boolean")) {
+                context.report(node, "Redundant double negation in Boolean constructor call.");
+            }
+        }
+    };
+
+};
+
+module.exports.schema = [];