Anders Logg wrote:
> On Sat, May 16, 2009 at 05:08:18PM +0200, DOLFIN wrote:
> 
>> One or more new changesets pushed to the primary dolfin repository.
>> A short summary of the last three changesets is included below.
>>
>> changeset:   6163:dff00dceec7cb1e9a8c5b89aa33519d555315ba5
>> tag:         tip
>> user:        "Garth N. Wells <[email protected]>"
>> date:        Sat May 16 16:06:30 2009 +0100
>> Fix very hard to find bug in dolfin_utils.
> 
> --- a/site-packages/dolfin_utils/wrappers/functionspace.py      Fri
> May 15 15:21:10 2009 +0200
> +++ b/site-packages/dolfin_utils/wrappers/functionspace.py      Sat
> May 16 16:06:30 2009 +0100
> @@ -46,7 +46,7 @@
>  //  }
>  // 
>  
> -  virtual ~%(classname)s()
> +  ~%(classname)s()
>    {
>    }
> 
> Was it just this? 

Yes.

I started looking at the bug but couldn't figure out
> what was wrong. I thought it was always safe to make a destructor
> virtual.
>

I didn't know either that it was a problem. I reduced the problem to

    const FunctionSpace* V = new CoefficientSpace_f;
    delete V;

which failed when the destructor was virtual (there were no other 
virtual member functions).

Garth

> Anyway, seems to work now.
> 
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> DOLFIN-dev mailing list
> [email protected]
> http://www.fenics.org/mailman/listinfo/dolfin-dev


_______________________________________________
DOLFIN-dev mailing list
[email protected]
http://www.fenics.org/mailman/listinfo/dolfin-dev

Reply via email to