summaryrefslogtreecommitdiff
path: root/tools/node_modules/eslint/lib/rules/no-case-declarations.js
diff options
context:
space:
mode:
Diffstat (limited to 'tools/node_modules/eslint/lib/rules/no-case-declarations.js')
-rw-r--r--tools/node_modules/eslint/lib/rules/no-case-declarations.js57
1 files changed, 57 insertions, 0 deletions
diff --git a/tools/node_modules/eslint/lib/rules/no-case-declarations.js b/tools/node_modules/eslint/lib/rules/no-case-declarations.js
new file mode 100644
index 0000000000..e801c6bb6e
--- /dev/null
+++ b/tools/node_modules/eslint/lib/rules/no-case-declarations.js
@@ -0,0 +1,57 @@
+/**
+ * @fileoverview Rule to flag use of an lexical declarations inside a case clause
+ * @author Erik Arvidsson
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = {
+ meta: {
+ docs: {
+ description: "disallow lexical declarations in case clauses",
+ category: "Best Practices",
+ recommended: true
+ },
+
+ schema: []
+ },
+
+ create(context) {
+
+ /**
+ * Checks whether or not a node is a lexical declaration.
+ * @param {ASTNode} node A direct child statement of a switch case.
+ * @returns {boolean} Whether or not the node is a lexical declaration.
+ */
+ function isLexicalDeclaration(node) {
+ switch (node.type) {
+ case "FunctionDeclaration":
+ case "ClassDeclaration":
+ return true;
+ case "VariableDeclaration":
+ return node.kind !== "var";
+ default:
+ return false;
+ }
+ }
+
+ return {
+ SwitchCase(node) {
+ for (let i = 0; i < node.consequent.length; i++) {
+ const statement = node.consequent[i];
+
+ if (isLexicalDeclaration(statement)) {
+ context.report({
+ node,
+ message: "Unexpected lexical declaration in case block."
+ });
+ }
+ }
+ }
+ };
+
+ }
+};