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>
30 lines
953 B
JavaScript
30 lines
953 B
JavaScript
/**
|
|
* @fileoverview Rule to flag comparison where left part is the same as the right
|
|
* part.
|
|
* @author Ilya Volodin
|
|
*/
|
|
|
|
"use strict";
|
|
|
|
//------------------------------------------------------------------------------
|
|
// Rule Definition
|
|
//------------------------------------------------------------------------------
|
|
|
|
module.exports = function(context) {
|
|
|
|
return {
|
|
|
|
"BinaryExpression": function(node) {
|
|
var operators = ["===", "==", "!==", "!=", ">", "<", ">=", "<="];
|
|
|
|
if (operators.indexOf(node.operator) > -1 &&
|
|
(node.left.type === "Identifier" && node.right.type === "Identifier" && node.left.name === node.right.name ||
|
|
node.left.type === "Literal" && node.right.type === "Literal" && node.left.value === node.right.value)) {
|
|
context.report(node, "Comparing to itself is potentially pointless.");
|
|
}
|
|
}
|
|
};
|
|
|
|
};
|
|
|
|
module.exports.schema = [];
|