eslint 6.3.0 → 6.6.0

package/lib/rules/no-duplicate-imports.js

@@ -10,8 +10,7 @@
 
 /**
  * Returns the name of the module imported or re-exported.
- *
- * @param {ASTNode} node - A node to get.
+ * @param {ASTNode} node A node to get.
  * @returns {string} the name of the module, or empty string if no name.
  */
 function getValue(node) {
@@ -24,12 +23,11 @@ function getValue(node) {
 
 /**
  * Checks if the name of the import or export exists in the given array, and reports if so.
- *
- * @param {RuleContext} context - The ESLint rule context object.
- * @param {ASTNode} node - A node to get.
- * @param {string} value - The name of the imported or exported module.
- * @param {string[]} array - The array containing other imports or exports in the file.
- * @param {string} messageId - A messageId to be reported after the name of the module
+ * @param {RuleContext} context The ESLint rule context object.
+ * @param {ASTNode} node A node to get.
+ * @param {string} value The name of the imported or exported module.
+ * @param {string[]} array The array containing other imports or exports in the file.
+ * @param {string} messageId A messageId to be reported after the name of the module
  *
  * @returns {void} No return value
  */
@@ -47,16 +45,15 @@ function checkAndReport(context, node, value, array, messageId) {
 
 /**
  * @callback nodeCallback
- * @param {ASTNode} node - A node to handle.
+ * @param {ASTNode} node A node to handle.
  */
 
 /**
  * Returns a function handling the imports of a given file
- *
- * @param {RuleContext} context - The ESLint rule context object.
- * @param {boolean} includeExports - Whether or not to check for exports in addition to imports.
- * @param {string[]} importsInFile - The array containing other imports in the file.
- * @param {string[]} exportsInFile - The array containing other exports in the file.
+ * @param {RuleContext} context The ESLint rule context object.
+ * @param {boolean} includeExports Whether or not to check for exports in addition to imports.
+ * @param {string[]} importsInFile The array containing other imports in the file.
+ * @param {string[]} exportsInFile The array containing other exports in the file.
  *
  * @returns {nodeCallback} A function passed to ESLint to handle the statement.
  */
@@ -78,10 +75,9 @@ function handleImports(context, includeExports, importsInFile, exportsInFile) {
 
 /**
  * Returns a function handling the exports of a given file
- *
- * @param {RuleContext} context - The ESLint rule context object.
- * @param {string[]} importsInFile - The array containing other imports in the file.
- * @param {string[]} exportsInFile - The array containing other exports in the file.
+ * @param {RuleContext} context The ESLint rule context object.
+ * @param {string[]} importsInFile The array containing other imports in the file.
+ * @param {string[]} exportsInFile The array containing other exports in the file.
  *
  * @returns {nodeCallback} A function passed to ESLint to handle the statement.
  */

package/lib/rules/no-else-return.js

@@ -58,7 +58,6 @@ module.exports = {
          *
          * This is not a generic function. In particular, it is assumed that the scope is a function scope or
          * a function's inner scope, and that the names can be valid identifiers in the given scope.
-         *
          * @param {string[]} names Array of variable names.
          * @param {eslint-scope.Scope} scope Function scope or a function's inner scope.
          * @returns {boolean} True if all names can be safely declared, false otherwise.
@@ -134,7 +133,6 @@ module.exports = {
 
         /**
          * Checks whether the removal of `else` and its braces is safe from variable name collisions.
-         *
          * @param {Node} node The 'else' node.
          * @param {eslint-scope.Scope} scope The scope in which the node and the whole 'if' statement is.
          * @returns {boolean} True if it is safe, false otherwise.
@@ -171,7 +169,6 @@ module.exports = {
 
         /**
          * Display the context report if rule is violated
-         *
          * @param {Node} node The 'else' node
          * @returns {void}
          */
@@ -255,7 +252,6 @@ module.exports = {
 
         /**
          * Check to see if the node is a ReturnStatement
-         *
          * @param {Node} node The node being evaluated
          * @returns {boolean} True if node is a return
          */
@@ -267,7 +263,6 @@ module.exports = {
          * Naive return checking, does not iterate through the whole
          * BlockStatement because we make the assumption that the ReturnStatement
          * will be the last node in the body of the BlockStatement.
-         *
          * @param {Node} node The consequent/alternate node
          * @returns {boolean} True if it has a return
          */
@@ -284,7 +279,6 @@ module.exports = {
         /**
          * Check to see if the node is valid for evaluation,
          * meaning it has an else.
-         *
          * @param {Node} node The node being evaluated
          * @returns {boolean} True if the node is valid
          */
@@ -296,7 +290,6 @@ module.exports = {
          * If the consequent is an IfStatement, check to see if it has an else
          * and both its consequent and alternate path return, meaning this is
          * a nested case of rule violation.  If-Else not considered currently.
-         *
          * @param {Node} node The consequent node
          * @returns {boolean} True if this is a nested rule violation
          */
@@ -309,7 +302,6 @@ module.exports = {
          * Check the consequent/body node to make sure it is not
          * a ReturnStatement or an IfStatement that returns on both
          * code paths.
-         *
          * @param {Node} node The consequent or body node
          * @returns {boolean} `true` if it is a Return/If node that always returns.
          */

package/lib/rules/no-empty-function.js

@@ -28,8 +28,7 @@ const ALLOW_OPTIONS = Object.freeze([
 
 /**
  * Gets the kind of a given function node.
- *
- * @param {ASTNode} node - A function node to get. This is one of
+ * @param {ASTNode} node A function node to get. This is one of
  *      an ArrowFunctionExpression, a FunctionDeclaration, or a
  *      FunctionExpression.
  * @returns {string} The kind of the function. This is one of "functions",
@@ -130,8 +129,7 @@ module.exports = {
          * - Not allowed by options.
          * - The body is empty.
          * - The body doesn't have any comments.
-         *
-         * @param {ASTNode} node - A function node to report. This is one of
+         * @param {ASTNode} node A function node to report. This is one of
          *      an ArrowFunctionExpression, a FunctionDeclaration, or a
          *      FunctionExpression.
          * @returns {void}

package/lib/rules/no-eval.js

@@ -22,9 +22,8 @@ const candidatesOfGlobalObject = Object.freeze([
 
 /**
  * Checks a given node is a Identifier node of the specified name.
- *
- * @param {ASTNode} node - A node to check.
- * @param {string} name - A name to check.
+ * @param {ASTNode} node A node to check.
+ * @param {string} name A name to check.
  * @returns {boolean} `true` if the node is a Identifier node of the name.
  */
 function isIdentifier(node, name) {
@@ -33,9 +32,8 @@ function isIdentifier(node, name) {
 
 /**
  * Checks a given node is a Literal node of the specified string value.
- *
- * @param {ASTNode} node - A node to check.
- * @param {string} name - A name to check.
+ * @param {ASTNode} node A node to check.
+ * @param {string} name A name to check.
  * @returns {boolean} `true` if the node is a Literal node of the name.
  */
 function isConstant(node, name) {
@@ -57,9 +55,8 @@ function isConstant(node, name) {
 /**
  * Checks a given node is a MemberExpression node which has the specified name's
  * property.
- *
- * @param {ASTNode} node - A node to check.
- * @param {string} name - A name to check.
+ * @param {ASTNode} node A node to check.
+ * @param {string} name A name to check.
  * @returns {boolean} `true` if the node is a MemberExpression node which has
  *      the specified name's property
  */
@@ -113,8 +110,7 @@ module.exports = {
          *
          * This is used in order to check whether or not `this` binding is a
          * reference to the global object.
-         *
-         * @param {ASTNode} node - A node of the scope. This is one of Program,
+         * @param {ASTNode} node A node of the scope. This is one of Program,
          *      FunctionDeclaration, FunctionExpression, and ArrowFunctionExpression.
          * @returns {void}
          */
@@ -132,7 +128,6 @@ module.exports = {
 
         /**
          * Pops a variable scope from the stack.
-         *
          * @returns {void}
          */
         function exitVarScope() {
@@ -148,8 +143,7 @@ module.exports = {
          * The location of the report is always `eval` `Identifier` (or possibly
          * `Literal`). The type of the report is `CallExpression` if the parent is
          * `CallExpression`. Otherwise, it's the given node type.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function report(node) {
@@ -171,8 +165,7 @@ module.exports = {
 
         /**
          * Reports accesses of `eval` via the global object.
-         *
-         * @param {eslint-scope.Scope} globalScope - The global scope.
+         * @param {eslint-scope.Scope} globalScope The global scope.
          * @returns {void}
          */
         function reportAccessingEvalViaGlobalObject(globalScope) {
@@ -205,8 +198,7 @@ module.exports = {
 
         /**
          * Reports all accesses of `eval` (excludes direct calls to eval).
-         *
-         * @param {eslint-scope.Scope} globalScope - The global scope.
+         * @param {eslint-scope.Scope} globalScope The global scope.
          * @returns {void}
          */
         function reportAccessingEval(globalScope) {

package/lib/rules/no-ex-assign.js

@@ -33,7 +33,7 @@ module.exports = {
 
         /**
          * Finds and reports references that are non initializer and writable.
-         * @param {Variable} variable - A variable to check.
+         * @param {Variable} variable A variable to check.
          * @returns {void}
          */
         function checkVariable(variable) {

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

@@ -40,13 +40,13 @@ module.exports = {
     },
 
     create(context) {
+        const sourceCode = context.getSourceCode();
         let scopeInfo = null;
 
         /**
          * Checks if a node is free of side effects.
          *
          * This check is stricter than it needs to be, in order to keep the implementation simple.
-         *
          * @param {ASTNode} node A node to check.
          * @returns {boolean} True if the node is known to be side-effect free, false otherwise.
          */
@@ -56,8 +56,7 @@ module.exports = {
 
         /**
          * Reports a given function node.
-         *
-         * @param {ASTNode} node - A node to report. This is a FunctionExpression or
+         * @param {ASTNode} node A node to report. This is a FunctionExpression or
          *      an ArrowFunctionExpression.
          * @returns {void}
          */
@@ -71,8 +70,13 @@ module.exports = {
                         return null;
                     }
 
-                    const firstTokenToRemove = context.getSourceCode()
+                    const firstTokenToRemove = sourceCode
                         .getFirstTokenBetween(node.parent.object, node.parent.property, astUtils.isNotClosingParenToken);
+                    const lastTokenToRemove = sourceCode.getLastToken(node.parent.parent);
+
+                    if (sourceCode.commentsExistBetween(firstTokenToRemove, lastTokenToRemove)) {
+                        return null;
+                    }
 
                     return fixer.removeRange([firstTokenToRemove.range[0], node.parent.parent.range[1]]);
                 }
@@ -84,8 +88,7 @@ module.exports = {
          * method.
          *
          * e.g. `(function() {}.bind(foo))`
-         *
-         * @param {ASTNode} node - A node to report. This is a FunctionExpression or
+         * @param {ASTNode} node A node to report. This is a FunctionExpression or
          *      an ArrowFunctionExpression.
          * @returns {boolean} `true` if the node is the callee of `.bind()` method.
          */
@@ -107,8 +110,7 @@ module.exports = {
 
         /**
          * Adds a scope information object to the stack.
-         *
-         * @param {ASTNode} node - A node to add. This node is a FunctionExpression
+         * @param {ASTNode} node A node to add. This node is a FunctionExpression
          *      or a FunctionDeclaration node.
          * @returns {void}
          */
@@ -124,8 +126,7 @@ module.exports = {
          * Removes the scope information object from the top of the stack.
          * At the same time, this reports the function node if the function has
          * `.bind()` and the `this` keywords found.
-         *
-         * @param {ASTNode} node - A node to remove. This node is a
+         * @param {ASTNode} node A node to remove. This node is a
          *      FunctionExpression or a FunctionDeclaration node.
          * @returns {void}
          */
@@ -140,8 +141,7 @@ module.exports = {
         /**
          * Reports a given arrow function if the function is callee of `.bind()`
          * method.
-         *
-         * @param {ASTNode} node - A node to report. This node is an
+         * @param {ASTNode} node A node to report. This node is an
          *      ArrowFunctionExpression.
          * @returns {void}
          */
@@ -153,7 +153,6 @@ module.exports = {
 
         /**
          * Set the mark as the `this` keyword was found in this scope.
-         *
          * @returns {void}
          */
         function markAsThisFound() {

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

@@ -49,7 +49,6 @@ module.exports = {
 
         /**
          * Check if a node is in a context where its value would be coerced to a boolean at runtime.
-         *
          * @param {ASTNode} node The node
          * @param {ASTNode} parent Its parent
          * @returns {boolean} If it is in a boolean context
@@ -67,7 +66,6 @@ module.exports = {
 
         /**
          * Check if a node has comments inside.
-         *
          * @param {ASTNode} node The node to check.
          * @returns {boolean} `true` if it has comments inside.
          */
@@ -96,7 +94,7 @@ module.exports = {
                         grandparent.callee.name === "Boolean")
                 ) {
                     context.report({
-                        node,
+                        node: parent,
                         messageId: "unexpectedNegation",
                         fix: fixer => {
                             if (hasCommentsInside(parent)) {

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

@@ -40,8 +40,7 @@ module.exports = {
 
         /**
          * Creates a new scope with a breakable statement.
-         *
-         * @param {ASTNode} node - A node to create. This is a BreakableStatement.
+         * @param {ASTNode} node A node to create. This is a BreakableStatement.
          * @returns {void}
          */
         function enterBreakableStatement(node) {
@@ -54,7 +53,6 @@ module.exports = {
 
         /**
          * Removes the top scope of the stack.
-         *
          * @returns {void}
          */
         function exitBreakableStatement() {
@@ -66,8 +64,7 @@ module.exports = {
          *
          * This ignores it if the body is a breakable statement.
          * In this case it's handled in the `enterBreakableStatement` function.
-         *
-         * @param {ASTNode} node - A node to create. This is a LabeledStatement.
+         * @param {ASTNode} node A node to create. This is a LabeledStatement.
          * @returns {void}
          */
         function enterLabeledStatement(node) {
@@ -85,8 +82,7 @@ module.exports = {
          *
          * This ignores it if the body is a breakable statement.
          * In this case it's handled in the `exitBreakableStatement` function.
-         *
-         * @param {ASTNode} node - A node. This is a LabeledStatement.
+         * @param {ASTNode} node A node. This is a LabeledStatement.
          * @returns {void}
          */
         function exitLabeledStatement(node) {
@@ -97,8 +93,7 @@ module.exports = {
 
         /**
          * Reports a given control node if it's unnecessary.
-         *
-         * @param {ASTNode} node - A node. This is a BreakStatement or a
+         * @param {ASTNode} node A node. This is a BreakStatement or a
          *      ContinueStatement.
          * @returns {void}
          */
@@ -116,7 +111,15 @@ module.exports = {
                             node: labelNode,
                             messageId: "unexpected",
                             data: labelNode,
-                            fix: fixer => fixer.removeRange([sourceCode.getFirstToken(node).range[1], labelNode.range[1]])
+                            fix(fixer) {
+                                const breakOrContinueToken = sourceCode.getFirstToken(node);
+
+                                if (sourceCode.commentsExistBetween(breakOrContinueToken, labelNode)) {
+                                    return null;
+                                }
+
+                                return fixer.removeRange([breakOrContinueToken.range[1], labelNode.range[1]]);
+                            }
                         });
                     }
                     return;

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

@@ -50,7 +50,8 @@ module.exports = {
                                 returnAssign: { type: "boolean" },
                                 ignoreJSX: { enum: ["none", "all", "single-line", "multi-line"] },
                                 enforceForArrowConditionals: { type: "boolean" },
-                                enforceForSequenceExpressions: { type: "boolean" }
+                                enforceForSequenceExpressions: { type: "boolean" },
+                                enforceForNewInMemberExpressions: { type: "boolean" }
                             },
                             additionalProperties: false
                         }
@@ -80,6 +81,8 @@ module.exports = {
             context.options[1].enforceForArrowConditionals === false;
         const IGNORE_SEQUENCE_EXPRESSIONS = ALL_NODES && context.options[1] &&
             context.options[1].enforceForSequenceExpressions === false;
+        const IGNORE_NEW_IN_MEMBER_EXPR = ALL_NODES && context.options[1] &&
+            context.options[1].enforceForNewInMemberExpressions === false;
 
         const PRECEDENCE_OF_ASSIGNMENT_EXPR = precedence({ type: "AssignmentExpression" });
         const PRECEDENCE_OF_UPDATE_EXPR = precedence({ type: "UpdateExpression" });
@@ -88,7 +91,7 @@ module.exports = {
 
         /**
          * Determines if this rule should be enforced for a node given the current configuration.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the rule should be enforced for this node.
          * @private
          */
@@ -127,7 +130,7 @@ module.exports = {
 
         /**
          * Determines if a node is surrounded by parentheses.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is parenthesised.
          * @private
          */
@@ -137,7 +140,7 @@ module.exports = {
 
         /**
          * Determines if a node is surrounded by parentheses twice.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is doubly parenthesised.
          * @private
          */
@@ -147,7 +150,7 @@ module.exports = {
 
         /**
          * Determines if a node is surrounded by (potentially) invalid parentheses.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is incorrectly parenthesised.
          * @private
          */
@@ -158,7 +161,7 @@ module.exports = {
         /**
          * Determines if a node that is expected to be parenthesised is surrounded by
          * (potentially) invalid extra parentheses.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is has an unexpected extra pair of parentheses.
          * @private
          */
@@ -168,7 +171,7 @@ module.exports = {
 
         /**
          * Determines if a node test expression is allowed to have a parenthesised assignment
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the assignment can be parenthesised.
          * @private
          */
@@ -178,7 +181,7 @@ module.exports = {
 
         /**
          * Determines if a node is in a return statement
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is in a return statement.
          * @private
          */
@@ -197,7 +200,7 @@ module.exports = {
 
         /**
          * Determines if a constructor function is newed-up with parens
-         * @param {ASTNode} newExpression - The NewExpression node to be checked.
+         * @param {ASTNode} newExpression The NewExpression node to be checked.
          * @returns {boolean} True if the constructor is called with parens.
          * @private
          */
@@ -205,12 +208,19 @@ module.exports = {
             const lastToken = sourceCode.getLastToken(newExpression);
             const penultimateToken = sourceCode.getTokenBefore(lastToken);
 
-            return newExpression.arguments.length > 0 || astUtils.isOpeningParenToken(penultimateToken) && astUtils.isClosingParenToken(lastToken);
+            return newExpression.arguments.length > 0 ||
+                (
+
+                    // The expression should end with its own parens, e.g., new new foo() is not a new expression with parens
+                    astUtils.isOpeningParenToken(penultimateToken) &&
+                    astUtils.isClosingParenToken(lastToken) &&
+                    newExpression.callee.range[1] < newExpression.range[1]
+                );
         }
 
         /**
          * Determines if a node is or contains an assignment expression
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is or contains an assignment expression.
          * @private
          */
@@ -232,7 +242,7 @@ module.exports = {
 
         /**
          * Determines if a node is contained by or is itself a return statement and is allowed to have a parenthesised assignment
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the assignment can be parenthesised.
          * @private
          */
@@ -254,8 +264,8 @@ module.exports = {
         /**
          * Determines if a node following a [no LineTerminator here] restriction is
          * surrounded by (potentially) invalid extra parentheses.
-         * @param {Token} token - The token preceding the [no LineTerminator here] restriction.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {Token} token The token preceding the [no LineTerminator here] restriction.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is incorrectly parenthesised.
          * @private
          */
@@ -338,7 +348,7 @@ module.exports = {
             function finishReport() {
                 context.report({
                     node,
-                    loc: leftParenToken.loc.start,
+                    loc: leftParenToken.loc,
                     messageId: "unexpected",
                     fix(fixer) {
                         const parenthesizedSource = sourceCode.text.slice(leftParenToken.range[1], rightParenToken.range[0]);
@@ -560,7 +570,6 @@ module.exports = {
         /**
          * Checks whether the syntax of the given ancestor of an 'in' expression inside a for-loop initializer
          * is preventing the 'in' keyword from being interpreted as a part of an ill-formed for-in loop.
-         *
          * @param {ASTNode} node Ancestor of an 'in' expression.
          * @param {ASTNode} child Child of the node, ancestor of the same 'in' expression or the 'in' expression itself.
          * @returns {boolean} True if the keyword 'in' would be interpreted as the 'in' operator, without any parenthesis.
@@ -592,7 +601,6 @@ module.exports = {
         /**
          * Starts a new reports buffering. Warnings will be stored in a buffer instead of being reported immediately.
          * An additional logic that requires multiple nodes (e.g. a whole subtree) may dismiss some of the stored warnings.
-         *
          * @returns {void}
          */
         function startNewReportsBuffering() {
@@ -887,6 +895,13 @@ module.exports = {
                     report(node.object);
                 }
 
+                if (nodeObjHasExcessParens &&
+                  !IGNORE_NEW_IN_MEMBER_EXPR &&
+                  node.object.type === "NewExpression" &&
+                  isNewExpressionWithParens(node.object)) {
+                    report(node.object);
+                }
+
                 if (node.computed && hasExcessParens(node.property)) {
                     report(node.property);
                 }

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

@@ -40,7 +40,7 @@ module.exports = {
 
         /**
          * Reports an unnecessary semicolon error.
-         * @param {Node|Token} nodeOrToken - A node or a token to be reported.
+         * @param {Node|Token} nodeOrToken A node or a token to be reported.
          * @returns {void}
          */
         function report(nodeOrToken) {
@@ -64,8 +64,7 @@ module.exports = {
         /**
          * Checks for a part of a class body.
          * This checks tokens from a specified token to a next MethodDefinition or the end of class body.
-         *
-         * @param {Token} firstToken - The first token to check.
+         * @param {Token} firstToken The first token to check.
          * @returns {void}
          */
         function checkForPartOfClassBody(firstToken) {
@@ -83,7 +82,7 @@ module.exports = {
 
             /**
              * Reports this empty statement, except if the parent node is a loop.
-             * @param {Node} node - A EmptyStatement node to be reported.
+             * @param {Node} node A EmptyStatement node to be reported.
              * @returns {void}
              */
             EmptyStatement(node) {
@@ -106,7 +105,7 @@ module.exports = {
 
             /**
              * Checks tokens from the head of this class body to the first MethodDefinition or the end of this class body.
-             * @param {Node} node - A ClassBody node to check.
+             * @param {Node} node A ClassBody node to check.
              * @returns {void}
              */
             ClassBody(node) {
@@ -115,7 +114,7 @@ module.exports = {
 
             /**
              * Checks tokens from this MethodDefinition to the next MethodDefinition or the end of this class body.
-             * @param {Node} node - A MethodDefinition node of the start point.
+             * @param {Node} node A MethodDefinition node of the start point.
              * @returns {void}
              */
             MethodDefinition(node) {

package/lib/rules/no-fallthrough.js

@@ -18,9 +18,9 @@ const DEFAULT_FALLTHROUGH_COMMENT = /falls?\s?through/iu;
 
 /**
  * Checks whether or not a given node has a fallthrough comment.
- * @param {ASTNode} node - A SwitchCase node to get comments.
- * @param {RuleContext} context - A rule context which stores comments.
- * @param {RegExp} fallthroughCommentPattern - A pattern to match comment to.
+ * @param {ASTNode} node A SwitchCase node to get comments.
+ * @param {RuleContext} context A rule context which stores comments.
+ * @param {RegExp} fallthroughCommentPattern A pattern to match comment to.
  * @returns {boolean} `true` if the node has a valid fallthrough comment.
  */
 function hasFallthroughComment(node, context, fallthroughCommentPattern) {
@@ -32,7 +32,7 @@ function hasFallthroughComment(node, context, fallthroughCommentPattern) {
 
 /**
  * Checks whether or not a given code path segment is reachable.
- * @param {CodePathSegment} segment - A CodePathSegment to check.
+ * @param {CodePathSegment} segment A CodePathSegment to check.
  * @returns {boolean} `true` if the segment is reachable.
  */
 function isReachable(segment) {
@@ -41,8 +41,8 @@ function isReachable(segment) {
 
 /**
  * Checks whether a node and a token are separated by blank lines
- * @param {ASTNode} node - The node to check
- * @param {Token} token - The token to compare against
+ * @param {ASTNode} node The node to check
+ * @param {Token} token The token to compare against
  * @returns {boolean} `true` if there are blank lines between node and token
  */
 function hasBlankLinesBetween(node, token) {