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

make bare super call in non-constructors illegal

Browse Source
This commit is contained in:
Sebastian McKenzie
2015-01-22 07:52:31 +11:00
parent da6c2b8d6a
commit f2a432531e
4 changed files with 16 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
lib/6to5/transformation/helpers/replace-supers.js
View File

@@ -202,6 +202,14 @@ ReplaceSupers.prototype.specHandle = function (getThisReference, node, parent) {
throw this.file.errorWithNode(node, "illegal use of bare super");
}
} else if (t.isCallExpression(node)) {
// super(); is illegal inside non-constructors
// https://esdiscuss.org/topic/super-call-in-methods
// https://twitter.com/wycats/status/544553184396836864
if (methodNode.key.name !== "constructor") {
var methodName = methodNode.key.name || "METHOD_NAME";
throw this.file.errorWithNode(node, "Direct super call is illegal in non-constructor, use super." + methodName + "() instead");
}
var callee = node.callee;
if (t.isIdentifier(callee, { name: "super" })) {
// super(); -> _get(Object.getPrototypeOf(ClassName), "MethodName", this).call(this);