d9b8758f47
PR-URL: https://github.com/nodejs/node/pull/6132 Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: thefourtheye <thechargingvolcano@gmail.com>
55 lines
1.7 KiB
JavaScript
55 lines
1.7 KiB
JavaScript
/**
|
|
* @fileoverview Rule to flag when IIFE is not wrapped in parens
|
|
* @author Ilya Volodin
|
|
* @copyright 2013 Ilya Volodin. All rights reserved.
|
|
*/
|
|
|
|
"use strict";
|
|
|
|
//------------------------------------------------------------------------------
|
|
// Rule Definition
|
|
//------------------------------------------------------------------------------
|
|
|
|
module.exports = function(context) {
|
|
|
|
var style = context.options[0] || "outside";
|
|
|
|
/**
|
|
* Check if the node is wrapped in ()
|
|
* @param {ASTNode} node node to evaluate
|
|
* @returns {boolean} True if it is wrapped
|
|
* @private
|
|
*/
|
|
function wrapped(node) {
|
|
var previousToken = context.getTokenBefore(node),
|
|
nextToken = context.getTokenAfter(node);
|
|
|
|
return previousToken && previousToken.value === "(" &&
|
|
nextToken && nextToken.value === ")";
|
|
}
|
|
|
|
return {
|
|
|
|
"CallExpression": function(node) {
|
|
if (node.callee.type === "FunctionExpression") {
|
|
var callExpressionWrapped = wrapped(node),
|
|
functionExpressionWrapped = wrapped(node.callee);
|
|
|
|
if (!callExpressionWrapped && !functionExpressionWrapped) {
|
|
context.report(node, "Wrap an immediate function invocation in parentheses.");
|
|
} else if (style === "inside" && !functionExpressionWrapped) {
|
|
context.report(node, "Wrap only the function expression in parens.");
|
|
} else if (style === "outside" && !callExpressionWrapped) {
|
|
context.report(node, "Move the invocation into the parens that contain the function.");
|
|
}
|
|
}
|
|
}
|
|
};
|
|
|
|
};
|
|
|
|
module.exports.schema = [
|
|
{
|
|
"enum": ["outside", "inside", "any"]
|
|
}
|
|
];
|