#1296 illegal destructor name: template+dflt args+virt. dtor

[Paweł Tomulik]

Hello,

the following code yields warning 512 "Illegal destructor name"

---------- %< -------------
%module testmod1
%inline %{
struct A {};
template <class X, class T = int>
struct B
{
virtual ~B() { }
};
template <class X>
struct B<X,int>
{
// swig reports "illegal destructor name" (only?) if this destructor
// is virtual
virtual ~B() { }
};
%}
%template(B_T_F) B<A,float>;
%template(B_T_I) B<A>;
---------- %< -------------

Attached code to reproduce this.

[Paweł Tomulik]

Code to reproduce bug.

[Paweł Tomulik]

I patched Source/CParse/parser.y and it seems to fix problem (see patch attached). Not sure about side effects. There was (intentional?) discrepancy in naming non-virtual and virtual destructor nodes. For virtual destructors, the parser always tried to append template specification to the name.

[William Fulton]

Thanks, patch applied for swig-2.0.10.