cerxes/babel
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

support duplicate constants within different block scopes - fixes #161

Browse Source
This commit is contained in:
Sebastian McKenzie 2014-11-15 02:49:49 +11:00
parent 682806f1ca
commit 58fac2e2be
1 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
lib/6to5/transformation/transformers/constants.js
View File

@ -9,11 +9,12 @@ exports.ForOfStatement =
exports.ForStatement = function (node, parent, file) {
var constants = [];
var check = function (node, names) {
var check = function (node, names, parent) {
_.each(names, function (name) {
if (constants.indexOf(name) >= 0) {
throw file.errorWithNode(node, name + " is read-only");
}
if (!_.has(constants, name)) return;
if (t.isBlockStatement(parent) && parent !== constants[name]) return;
throw file.errorWithNode(node, name + " is read-only");
});
};
@ -21,12 +22,12 @@ exports.ForStatement = function (node, parent, file) {
return t.getIds(node, false, ["MemberExpression"]);
};
_.each(node.body, function (child) {
_.each(node.body, function (child, parent) {
if (child && t.isVariableDeclaration(child, { kind: "const" })) {
_.each(child.declarations, function (declar) {
_.each(getIds(declar), function (name) {
check(declar, [name]);
constants.push(name);
check(declar, [name], parent);
constants[name] = parent;
});
declar._ignoreConstant = true;