mirror of https://github.com/lukechilds/node.git
You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
51 lines
1.4 KiB
51 lines
1.4 KiB
10 years ago
|
/**
|
||
|
* @fileoverview Rule to flag variable leak in CatchClauses in IE 8 and earlier
|
||
|
* @author Ian Christian Myers
|
||
|
*/
|
||
|
|
||
|
"use strict";
|
||
|
|
||
|
//------------------------------------------------------------------------------
|
||
|
// Rule Definition
|
||
|
//------------------------------------------------------------------------------
|
||
|
|
||
|
module.exports = function(context) {
|
||
|
|
||
|
//--------------------------------------------------------------------------
|
||
|
// Helpers
|
||
|
//--------------------------------------------------------------------------
|
||
|
|
||
|
function paramIsShadowing(scope, name) {
|
||
|
var found = scope.variables.some(function(variable) {
|
||
|
return variable.name === name;
|
||
|
});
|
||
|
|
||
|
if (found) {
|
||
|
return true;
|
||
|
}
|
||
|
|
||
|
if (scope.upper) {
|
||
|
return paramIsShadowing(scope.upper, name);
|
||
|
}
|
||
|
|
||
|
return false;
|
||
|
}
|
||
|
|
||
|
//--------------------------------------------------------------------------
|
||
|
// Public API
|
||
|
//--------------------------------------------------------------------------
|
||
|
|
||
|
return {
|
||
|
|
||
|
"CatchClause": function(node) {
|
||
|
var scope = context.getScope();
|
||
|
|
||
|
if (paramIsShadowing(scope, node.param.name)) {
|
||
|
context.report(node, "Value of '{{name}}' may be overwritten in IE 8 and earlier.",
|
||
|
{ name: node.param.name });
|
||
|
}
|
||
|
}
|
||
|
};
|
||
|
|
||
|
};
|