remove redundant enumerable: true property on class methods

This commit is contained in:
Sebastian McKenzie
2015-01-30 10:44:33 +11:00
parent 579db9107f
commit bbbc9c0c5e
14 changed files with 7 additions and 17 deletions

View File

@@ -231,7 +231,7 @@ Class.prototype.pushMethod = function (node) {
}
util.pushMutatorMap(mutatorMap, methodName, kind, node.computed, node);
util.pushMutatorMap(mutatorMap, methodName, "enumerable", node.computed, t.literal(false));
util.pushMutatorMap(mutatorMap, methodName, "enumerable", node.computed, false);
};
/**

View File

@@ -121,7 +121,12 @@ exports.buildDefineProperties = function (mutatorMap) {
map.writable = t.literal(true);
}
map.enumerable = map.enumerable || t.literal(true);
if (map.enumerable === false) {
delete map.enumerable;
} else {
map.enumerable = t.literal(true);
}
map.configurable = t.literal(true);
each(map, function (node, key) {

View File

@@ -12,7 +12,6 @@ var Foo = (function () {
value: _asyncToGenerator(function* () {
var wat = yield bar();
}),
enumerable: false,
writable: true,
configurable: true
}

View File

@@ -12,7 +12,6 @@ var Foo = (function () {
value: _bluebird.coroutine(function* () {
var wat = yield bar();
}),
enumerable: false,
writable: true,
configurable: true
}

View File

@@ -31,7 +31,6 @@ var Test = (function (Foo) {
_get(Object.getPrototypeOf(Test), "foo", this).apply(this, arguments);
(_get2 = _get(Object.getPrototypeOf(Test), "foo", this)).call.apply(_get2, [this, "test"].concat(_slice.call(arguments)));
},
enumerable: false,
writable: true,
configurable: true
}
@@ -43,7 +42,6 @@ var Test = (function (Foo) {
_get(Object.getPrototypeOf(Test.prototype), "test", this).apply(this, arguments);
(_get2 = _get(Object.getPrototypeOf(Test.prototype), "test", this)).call.apply(_get2, [this, "test"].concat(_slice.call(arguments)));
},
enumerable: false,
writable: true,
configurable: true
}

View File

@@ -19,7 +19,6 @@ var Test = (function (Foo) {
value: function test() {
return _get(Object.getPrototypeOf(Test), "wow", this).call(this);
},
enumerable: false,
writable: true,
configurable: true
}

View File

@@ -10,7 +10,6 @@ var Test = (function () {
get: function () {
return 5 + 5;
},
enumerable: false,
set: function (val) {
this._test = val;
},

View File

@@ -10,7 +10,6 @@ var Test = (function () {
get: function () {
return 5 + 5;
},
enumerable: false,
configurable: true
}
});

View File

@@ -10,7 +10,6 @@ var Test = (function () {
value: function test() {
return 5 + 5;
},
enumerable: false,
writable: true,
configurable: true
}

View File

@@ -10,7 +10,6 @@ var Test = (function () {
set: function (val) {
this._test = val;
},
enumerable: false,
configurable: true
}
});

View File

@@ -18,7 +18,6 @@ var BaseView = (function () {
value: function foo() {
this.autoRender = true;
},
enumerable: false,
writable: true,
configurable: true
}

View File

@@ -8,13 +8,11 @@ var A = (function () {
_prototypeProperties(A, {
a: {
value: function a() {},
enumerable: false,
writable: true,
configurable: true
},
b: {
get: function () {},
enumerable: false,
set: function (b) {},
configurable: true
}

View File

@@ -12,14 +12,12 @@ var Foo = (function () {
get: function () {
return _defineProperty(this, "bar", complex()).bar;
},
enumerable: false,
configurable: true
}
}, bar, {
get: function () {
return _defineProperty(this, bar, complex())[bar];
},
enumerable: false,
configurable: true
}));

View File

@@ -10,7 +10,6 @@ var Test = (function () {
get: function () {
throw new Error("wow");
},
enumerable: false,
configurable: true
}
});