github/coffeescript
1
1
Fork 0
mirror of https://github.com/jashkenas/coffeescript.git synced 2026-02-19 03:44:23 -05:00
Code Issues Packages Projects Releases Wiki Activity

Throwing proper error when "constructor" is not a function within a class body.

Browse Source
This commit is contained in:
Stan Angeloff
2010-04-03 21:43:42 +03:00
parent d1aaed4430
commit 9958cedd89
2 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
lib/nodes.js
View File

@@ -821,6 +821,9 @@
pvar = prop.variable; pvar = prop.variable;
if (pvar && pvar.base.value === 'constructor') { if (pvar && pvar.base.value === 'constructor') {
func = prop.value; func = prop.value;
if (!(func instanceof CodeNode)) {
throw new Error("'" + (pvar.base.value) + "' must be a function in 'class " + (this.variable.base.value) + "'.");
}
func.body.push(new ReturnNode(literal('this'))); func.body.push(new ReturnNode(literal('this')));
constructor = new AssignNode(this.variable, func); constructor = new AssignNode(this.variable, func);
} else { } else {

1
src/nodes.coffee
View File

@@ -601,6 +601,7 @@ exports.ClassNode: class ClassNode extends BaseNode
pvar: prop.variable pvar: prop.variable
if pvar and pvar.base.value is 'constructor' if pvar and pvar.base.value is 'constructor'
func: prop.value func: prop.value
throw new Error("'${ pvar.base.value }' must be a function in 'class ${ @variable.base.value }'.") if not (func instanceof CodeNode)
func.body.push(new ReturnNode(literal('this'))) func.body.push(new ReturnNode(literal('this')))
constructor: new AssignNode(@variable, func) constructor: new AssignNode(@variable, func)
else else