eslint 0.22.0 → 0.24.1

package/lib/rules/curly.js

@@ -1,109 +1,109 @@
-/**
- * @fileoverview Rule to flag statements without curly braces
- * @author Nicholas C. Zakas
- */
-"use strict";
-
-//------------------------------------------------------------------------------
-// Rule Definition
-//------------------------------------------------------------------------------
-
-module.exports = function(context) {
-
-    var multiOnly = (context.options[0] === "multi");
-    var multiLine = (context.options[0] === "multi-line");
-
-    //--------------------------------------------------------------------------
-    // Helpers
-    //--------------------------------------------------------------------------
-
-    /**
-     * Determines if a given node is a one-liner that's on the same line as it's preceding code.
-     * @param {ASTNode} node The node to check.
-     * @returns {boolean} True if the node is a one-liner that's on the same line as it's preceding code.
-     * @private
-     */
-    function isCollapsedOneLiner(node) {
-        var before = context.getTokenBefore(node),
-            last = context.getLastToken(node);
-        return before.loc.start.line === last.loc.end.line;
-    }
-
-    /**
-     * Checks the body of a node to see if it's a block statement. Depending on
-     * the rule options, reports the appropriate problems.
-     * @param {ASTNode} node The node to report if there's a problem.
-     * @param {ASTNode} body The body node to check for blocks.
-     * @param {string} name The name to report if there's a problem.
-     * @param {string} suffix Additional string to add to the end of a report.
-     * @returns {void}
-     */
-    function checkBody(node, body, name, suffix) {
-        var hasBlock = (body.type === "BlockStatement");
-
-        if (multiOnly) {
-            if (hasBlock && body.body.length === 1) {
-                context.report(node, "Unnecessary { after '{{name}}'{{suffix}}.",
-                    {
-                        name: name,
-                        suffix: (suffix ? " " + suffix : "")
-                    }
-                );
-            }
-        } else if (multiLine) {
-            if (!hasBlock && !isCollapsedOneLiner(body)) {
-                context.report(node, "Expected { after '{{name}}'{{suffix}}.",
-                    {
-                        name: name,
-                        suffix: (suffix ? " " + suffix : "")
-                    }
-                );
-            }
-        } else {
-            if (!hasBlock) {
-                context.report(node, "Expected { after '{{name}}'{{suffix}}.",
-                    {
-                        name: name,
-                        suffix: (suffix ? " " + suffix : "")
-                    }
-                );
-            }
-        }
-    }
-
-    //--------------------------------------------------------------------------
-    // Public
-    //--------------------------------------------------------------------------
-
-    return {
-
-        "IfStatement": function(node) {
-
-            checkBody(node, node.consequent, "if", "condition");
-
-            if (node.alternate && node.alternate.type !== "IfStatement") {
-                checkBody(node, node.alternate, "else");
-            }
-
-        },
-
-        "WhileStatement": function(node) {
-            checkBody(node, node.body, "while", "condition");
-        },
-
-        "DoWhileStatement": function (node) {
-            checkBody(node, node.body, "do");
-        },
-
-        "ForStatement": function(node) {
-            checkBody(node, node.body, "for", "condition");
-        }
-    };
-
-};
-
-module.exports.schema = [
-    {
-        "enum": ["all", "multi", "multi-line"]
-    }
-];
+/**
+ * @fileoverview Rule to flag statements without curly braces
+ * @author Nicholas C. Zakas
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = function(context) {
+
+    var multiOnly = (context.options[0] === "multi");
+    var multiLine = (context.options[0] === "multi-line");
+
+    //--------------------------------------------------------------------------
+    // Helpers
+    //--------------------------------------------------------------------------
+
+    /**
+     * Determines if a given node is a one-liner that's on the same line as it's preceding code.
+     * @param {ASTNode} node The node to check.
+     * @returns {boolean} True if the node is a one-liner that's on the same line as it's preceding code.
+     * @private
+     */
+    function isCollapsedOneLiner(node) {
+        var before = context.getTokenBefore(node),
+            last = context.getLastToken(node);
+        return before.loc.start.line === last.loc.end.line;
+    }
+
+    /**
+     * Checks the body of a node to see if it's a block statement. Depending on
+     * the rule options, reports the appropriate problems.
+     * @param {ASTNode} node The node to report if there's a problem.
+     * @param {ASTNode} body The body node to check for blocks.
+     * @param {string} name The name to report if there's a problem.
+     * @param {string} suffix Additional string to add to the end of a report.
+     * @returns {void}
+     */
+    function checkBody(node, body, name, suffix) {
+        var hasBlock = (body.type === "BlockStatement");
+
+        if (multiOnly) {
+            if (hasBlock && body.body.length === 1) {
+                context.report(node, "Unnecessary { after '{{name}}'{{suffix}}.",
+                    {
+                        name: name,
+                        suffix: (suffix ? " " + suffix : "")
+                    }
+                );
+            }
+        } else if (multiLine) {
+            if (!hasBlock && !isCollapsedOneLiner(body)) {
+                context.report(node, "Expected { after '{{name}}'{{suffix}}.",
+                    {
+                        name: name,
+                        suffix: (suffix ? " " + suffix : "")
+                    }
+                );
+            }
+        } else {
+            if (!hasBlock) {
+                context.report(node, "Expected { after '{{name}}'{{suffix}}.",
+                    {
+                        name: name,
+                        suffix: (suffix ? " " + suffix : "")
+                    }
+                );
+            }
+        }
+    }
+
+    //--------------------------------------------------------------------------
+    // Public
+    //--------------------------------------------------------------------------
+
+    return {
+
+        "IfStatement": function(node) {
+
+            checkBody(node, node.consequent, "if", "condition");
+
+            if (node.alternate && node.alternate.type !== "IfStatement") {
+                checkBody(node, node.alternate, "else");
+            }
+
+        },
+
+        "WhileStatement": function(node) {
+            checkBody(node, node.body, "while", "condition");
+        },
+
+        "DoWhileStatement": function (node) {
+            checkBody(node, node.body, "do");
+        },
+
+        "ForStatement": function(node) {
+            checkBody(node, node.body, "for", "condition");
+        }
+    };
+
+};
+
+module.exports.schema = [
+    {
+        "enum": ["all", "multi", "multi-line"]
+    }
+];

package/lib/rules/default-case.js

@@ -1,66 +1,66 @@
-/**
- * @fileoverview require default case in switch statements
- * @author Aliaksei Shytkin
- */
-"use strict";
-
-var COMMENT_VALUE = "no default";
-
-//------------------------------------------------------------------------------
-// Rule Definition
-//------------------------------------------------------------------------------
-
-module.exports = function(context) {
-
-    //--------------------------------------------------------------------------
-    // Helpers
-    //--------------------------------------------------------------------------
-
-    /**
-     * Shortcut to get last element of array
-     * @param  {*[]} collection Array
-     * @returns {*} Last element
-     */
-    function last(collection) {
-        return collection[collection.length - 1];
-    }
-
-    //--------------------------------------------------------------------------
-    // Public
-    //--------------------------------------------------------------------------
-
-    return {
-
-        "SwitchStatement": function(node) {
-
-            if (!node.cases.length) {
-                // skip check of empty switch because there is no easy way
-                // to extract comments inside it now
-                return;
-            }
-
-            var hasDefault = node.cases.some(function(v) {
-                return v.test === null;
-            });
-
-            if (!hasDefault) {
-
-                var comment;
-                var comments;
-
-                var lastCase = last(node.cases);
-                comments = context.getComments(lastCase).trailing;
-
-                if (comments.length) {
-                    comment = last(comments);
-                }
-
-                if (!comment || comment.value.trim() !== COMMENT_VALUE) {
-                    context.report(node, "Expected a default case.");
-                }
-            }
-        }
-    };
-};
-
-module.exports.schema = [];
+/**
+ * @fileoverview require default case in switch statements
+ * @author Aliaksei Shytkin
+ */
+"use strict";
+
+var COMMENT_VALUE = "no default";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = function(context) {
+
+    //--------------------------------------------------------------------------
+    // Helpers
+    //--------------------------------------------------------------------------
+
+    /**
+     * Shortcut to get last element of array
+     * @param  {*[]} collection Array
+     * @returns {*} Last element
+     */
+    function last(collection) {
+        return collection[collection.length - 1];
+    }
+
+    //--------------------------------------------------------------------------
+    // Public
+    //--------------------------------------------------------------------------
+
+    return {
+
+        "SwitchStatement": function(node) {
+
+            if (!node.cases.length) {
+                // skip check of empty switch because there is no easy way
+                // to extract comments inside it now
+                return;
+            }
+
+            var hasDefault = node.cases.some(function(v) {
+                return v.test === null;
+            });
+
+            if (!hasDefault) {
+
+                var comment;
+                var comments;
+
+                var lastCase = last(node.cases);
+                comments = context.getComments(lastCase).trailing;
+
+                if (comments.length) {
+                    comment = last(comments);
+                }
+
+                if (!comment || comment.value.trim() !== COMMENT_VALUE) {
+                    context.report(node, "Expected a default case.");
+                }
+            }
+        }
+    };
+};
+
+module.exports.schema = [];

package/lib/rules/dot-location.js

@@ -1,63 +1,63 @@
-/**
- * @fileoverview Validates newlines before and after dots
- * @author Greg Cochard
- * @copyright 2015 Greg Cochard
- */
-
-"use strict";
-
-//------------------------------------------------------------------------------
-// Rule Definition
-//------------------------------------------------------------------------------
-
-module.exports = function (context) {
-
-    var config = context.options[0],
-        // default to onObject if no preference is passed
-        onObject = config === "object" || !config;
-
-    /**
-     * Checks whether two tokens are on the same line.
-     * @param {Object} left The leftmost token.
-     * @param {Object} right The rightmost token.
-     * @returns {boolean} True if the tokens are on the same line, false if not.
-     * @private
-     */
-    function isSameLine(left, right) {
-        return left.loc.end.line === right.loc.start.line;
-    }
-
-    /**
-     * Reports if the dot between object and property is on the correct loccation.
-     * @param {ASTNode} obj The object owning the property.
-     * @param {ASTNode} prop The property of the object.
-     * @param {ASTNode} node The corresponding node of the token.
-     * @returns {void}
-     */
-    function checkDotLocation(obj, prop, node) {
-        var dot = context.getTokenBefore(prop);
-
-        if (dot.type === "Punctuator" && dot.value === ".") {
-            if (onObject) {
-                if (!isSameLine(obj, dot)) {
-                    context.report(node, dot.loc.start, "Expected dot to be on same line as object.");
-                }
-            } else if (!isSameLine(dot, prop)) {
-                context.report(node, dot.loc.start, "Expected dot to be on same line as property.");
-            }
-        }
-    }
-
-    /**
-     * Checks the spacing of the dot within a member expression.
-     * @param {ASTNode} node The node to check.
-     * @returns {void}
-     */
-    function checkNode(node) {
-        checkDotLocation(node.object, node.property, node);
-    }
-
-    return {
-        "MemberExpression": checkNode
-    };
-};
+/**
+ * @fileoverview Validates newlines before and after dots
+ * @author Greg Cochard
+ * @copyright 2015 Greg Cochard
+ */
+
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = function (context) {
+
+    var config = context.options[0],
+        // default to onObject if no preference is passed
+        onObject = config === "object" || !config;
+
+    /**
+     * Checks whether two tokens are on the same line.
+     * @param {Object} left The leftmost token.
+     * @param {Object} right The rightmost token.
+     * @returns {boolean} True if the tokens are on the same line, false if not.
+     * @private
+     */
+    function isSameLine(left, right) {
+        return left.loc.end.line === right.loc.start.line;
+    }
+
+    /**
+     * Reports if the dot between object and property is on the correct loccation.
+     * @param {ASTNode} obj The object owning the property.
+     * @param {ASTNode} prop The property of the object.
+     * @param {ASTNode} node The corresponding node of the token.
+     * @returns {void}
+     */
+    function checkDotLocation(obj, prop, node) {
+        var dot = context.getTokenBefore(prop);
+
+        if (dot.type === "Punctuator" && dot.value === ".") {
+            if (onObject) {
+                if (!isSameLine(obj, dot)) {
+                    context.report(node, dot.loc.start, "Expected dot to be on same line as object.");
+                }
+            } else if (!isSameLine(dot, prop)) {
+                context.report(node, dot.loc.start, "Expected dot to be on same line as property.");
+            }
+        }
+    }
+
+    /**
+     * Checks the spacing of the dot within a member expression.
+     * @param {ASTNode} node The node to check.
+     * @returns {void}
+     */
+    function checkNode(node) {
+        checkDotLocation(node.object, node.property, node);
+    }
+
+    return {
+        "MemberExpression": checkNode
+    };
+};