Bug 8809 – (D1 Only) Cannot statically bind to base class method overridden by derived class

Status
RESOLVED
Resolution
WONTFIX
Severity
normal
Priority
P2
Component
dmd
Product
D
Version
D1 (retired)
Platform
All
OS
All
Creation time
2012-10-12T20:30:12Z
Last change time
2019-10-10T11:54:35Z
Keywords
pull, rejects-valid
Assigned to
No Owner
Creator
hsteoh

Comments

Comment #0 by hsteoh — 2012-10-12T20:30:12Z
class B { struct S { S delegate() saveImpl; S save() { return saveImpl(); } } S eval() { // BUG: B.eval doesn't statically bind to B.eval, but ends up in C.eval, causing stack overflow return S(() => B.eval()); // return S(() => typeof(this).eval()); // this doesn't work either } } class C : B { override S eval() { auto s = super.eval(); auto t = s.save; // stack overflow, 'cos B.eval binds to C.eval return t; } } void main() { auto c = new C; auto s = c.eval(); } Basically, there is no way in the base class method to statically bind to the un-overridden method; no matter what is specified (this.eval(), B.eval(), typeof(this).eval(), etc.), it always ends up in C.eval, causing infinite recursion when C.eval calls s.save. One workaround is to rename B.eval to B.evalImpl, and make B.eval a wrapper that calls B.evalImpl. Then the delegate can be made to call evalImpl directly. But this is quite ugly, since one would have to do this for every base class method that needs to be statically bound in this way. One would expect the language (or compiler) should produce a static binding to B.eval when the code explicitly asks for B.eval.
Comment #1 by k.hara.pg — 2012-10-14T03:02:36Z
https://github.com/D-Programming-Language/dmd/pull/1181 And, this is also a problem in website.
Comment #2 by github-bugzilla — 2012-10-14T08:31:12Z
Comment #3 by github-bugzilla — 2012-10-27T06:10:26Z
Commits pushed to master at https://github.com/D-Programming-Language/dmd https://github.com/D-Programming-Language/dmd/commit/364bcddf80daf3fcf82237276f0d898cc68d91a2 fix Issue 8809 - Cannot statically bind to base class method overridden by derived class Collect DotType resolution code to TypeClass::dotExp obj.Type.foo() calls Type.foo statically. In the non-static member function, Type.foo() is implicitly translated to this.Type.foo(), and same as well. https://github.com/D-Programming-Language/dmd/commit/4c11aea5ddd866e1c86d10bdfb4d0acb8469b611 Merge pull request #1181 from 9rnsr/fix8809 Issue 8809 - Cannot statically bind to base class method overridden by derived class
Comment #4 by yebblies — 2012-10-27T06:18:38Z
Fixed D2