From: <kli...@us...> - 2011-01-08 21:05:56
|
Revision: 12380 http://swig.svn.sourceforge.net/swig/?rev=12380&view=rev Author: klickverbot Date: 2011-01-08 21:05:49 +0000 (Sat, 08 Jan 2011) Log Message: ----------- [D] Fixed a bug in the loop breaking code for directors leading to a superclass implementation erroneously being called. The situation in which this would previously happen is illustrated in the new "director_alternating" test case. Currently broken for C# and Java. Thanks to Jimmy Cao for reporting this. Modified Paths: -------------- trunk/Examples/test-suite/common.mk trunk/Source/Modules/d.cxx Added Paths: ----------- trunk/Examples/test-suite/d/director_alternating_runme.2.d trunk/Examples/test-suite/director_alternating.i trunk/Examples/test-suite/director_alternating_runme.1.d trunk/Examples/test-suite/python/director_alternating_runme.py Modified: trunk/Examples/test-suite/common.mk =================================================================== --- trunk/Examples/test-suite/common.mk 2011-01-08 01:28:09 UTC (rev 12379) +++ trunk/Examples/test-suite/common.mk 2011-01-08 21:05:49 UTC (rev 12380) @@ -154,6 +154,7 @@ derived_nested \ destructor_reprotected \ director_abstract \ + director_alternating \ director_basic \ director_classes \ director_classic \ Added: trunk/Examples/test-suite/d/director_alternating_runme.2.d =================================================================== --- trunk/Examples/test-suite/d/director_alternating_runme.2.d (rev 0) +++ trunk/Examples/test-suite/d/director_alternating_runme.2.d 2011-01-08 21:05:49 UTC (rev 12380) @@ -0,0 +1,7 @@ +module director_alternating_runme; + +import director_alternating.director_alternating; + +void main() { + assert(getBar().id() == idFromGetBar()); +} Added: trunk/Examples/test-suite/director_alternating.i =================================================================== --- trunk/Examples/test-suite/director_alternating.i (rev 0) +++ trunk/Examples/test-suite/director_alternating.i 2011-01-08 21:05:49 UTC (rev 12380) @@ -0,0 +1,36 @@ +// Checks if calls to a method being defined in the base class, not +// overridden in the subclass, but again overridden in a class derived from +// the first subclass are dispatched correctly. +%module(directors="1") director_alternating; + +%feature("director") Foo; + +%inline %{ +struct Foo { + virtual ~Foo() {} + virtual int id() { + return 0; + } +}; + +struct Bar : Foo {}; + +struct Baz : Bar { + virtual int id() { + return 2; + } +}; + +// Note that even though the return value is of type Bar*, it really points to +// an instance of Baz (in which id() has been overridden). +Bar *getBar() { + static Baz baz; + return &baz; +} + +// idFromGetBar() obviously is equivalent to getBar()->id() in C++ – this +// should be true from the target language as well. +int idFromGetBar() { + return getBar()->id(); +} +%} Added: trunk/Examples/test-suite/director_alternating_runme.1.d =================================================================== --- trunk/Examples/test-suite/director_alternating_runme.1.d (rev 0) +++ trunk/Examples/test-suite/director_alternating_runme.1.d 2011-01-08 21:05:49 UTC (rev 12380) @@ -0,0 +1,7 @@ +module director_alternating_runme; + +import director_alternating.director_alternating; + +void main() { + assert(getBar().id() == idFromGetBar()); +} Added: trunk/Examples/test-suite/python/director_alternating_runme.py =================================================================== --- trunk/Examples/test-suite/python/director_alternating_runme.py (rev 0) +++ trunk/Examples/test-suite/python/director_alternating_runme.py 2011-01-08 21:05:49 UTC (rev 12380) @@ -0,0 +1,5 @@ +from director_alternating import * + +id = getBar().id() +if id != idFromGetBar(): + raise RuntimeError, "Got wrong id: " + str(id) Modified: trunk/Source/Modules/d.cxx =================================================================== --- trunk/Source/Modules/d.cxx 2011-01-08 01:28:09 UTC (rev 12379) +++ trunk/Source/Modules/d.cxx 2011-01-08 21:05:49 UTC (rev 12380) @@ -2630,6 +2630,8 @@ if (!static_flag) { Printf(imcall, "cast(void*)swigCPtr"); } + + String *proxy_param_types = NewString(""); // Write the parameter list for the proxy function declaration and the // wrapper function call. @@ -2704,10 +2706,13 @@ "No dtype typemap defined for %s\n", SwigType_str(pt, 0)); } - if (gencomma >= 2) + if (gencomma >= 2) { Printf(function_code, ", "); + Printf(proxy_param_types, ", "); + } gencomma = 2; Printf(function_code, "%s %s", proxy_type, param_name); + Append(proxy_param_types, proxy_type); Delete(proxy_type); } @@ -2772,9 +2777,11 @@ String *excode = NewString(""); if (!Cmp(return_type, "void")) - Printf(excode, "if (this.classinfo == %s.classinfo) %s; else %s", proxy_class_name, imcall, ex_imcall); + Printf(excode, "if (swigIsMethodOverridden!(%s delegate(%s), %s function(%s), %s)()) %s; else %s", + return_type, proxy_param_types, return_type, proxy_param_types, proxy_function_name, ex_imcall, imcall); else - Printf(excode, "((this.classinfo == %s.classinfo) ? %s : %s)", proxy_class_name, imcall, ex_imcall); + Printf(excode, "((swigIsMethodOverridden!(%s delegate(%s), %s function(%s), %s)()) ? %s : %s)", + return_type, proxy_param_types, return_type, proxy_param_types, proxy_function_name, ex_imcall, imcall); Clear(imcall); Printv(imcall, excode, NIL); @@ -2788,6 +2795,8 @@ Swig_warning(WARN_D_TYPEMAP_DOUT_UNDEF, input_file, line_number, "No dout typemap defined for %s\n", SwigType_str(t, 0)); } + + Delete(proxy_param_types); // The whole function body is now in stored tm (if there was a matching type // map, of course), so simply append it to the code buffer. The braces are @@ -3418,7 +3427,7 @@ // Only emit it if the proxy class has at least one method. if (first_class_dmethod < curr_class_dmethod) { Printf(proxy_class_body_code, "\n"); - Printf(proxy_class_body_code, "private bool swigIsMethodOverridden(DelegateType, FunctionType, alias fn)() {\n"); + Printf(proxy_class_body_code, "private bool swigIsMethodOverridden(DelegateType, FunctionType, alias fn)() %s{\n", (d_version > 1) ? "const " : ""); Printf(proxy_class_body_code, " DelegateType dg = &fn;\n"); Printf(proxy_class_body_code, " return dg.funcptr != SwigNonVirtualAddressOf!(FunctionType, fn);\n"); Printf(proxy_class_body_code, "}\n"); This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |