ceylon / ceylon-js

DEPRECATED
Apache License 2.0
54 stars 9 forks source link

error for class extension of named constructor via class alias #584

Closed sgalles closed 9 years ago

sgalles commented 9 years ago

FTR I believe that it's a regression. I think that it used to work a few days ago. This code

class Foo{
    shared new Foo2(){}
}
class Bar() => Foo.Foo2();
class Baz() extends Bar(){}

shared void run(){
    Baz();
}

executed

Bar.$$=Foo_Foo_Foo2.$$;
       ^
ReferenceError: Foo_Foo_Foo2 is not defined
chochos commented 9 years ago

not a regression, strictly speaking. The name for constructors was being generated wrong in some cases and fixing that caused some new issues apparently. We really do need more constructor tests, preferably in the language module. I already added some for the value constructors, so after merge I'll add some more.