← Back to team overview

dolfin team mailing list archive

Re: [Branch ~dolfin-core/dolfin/main] Rev 5620: - Make VariationalProblem Hierarchical

 

On Monday January 31 2011 12:07:38 noreply@xxxxxxxxxxxxx wrote:
> ------------------------------------------------------------
> revno: 5620
> committer: Anders Logg <logg@xxxxxxxxx>
> branch nick: dolfin
> timestamp: Mon 2011-01-31 21:03:50 +0100
> message:
>   - Make VariationalProblem Hierarchical
>   - Implement refinement of VariationalProblem
>   - Some shared_ptr fixes
> 
>   I suggest that all functions returning a shared_ptr are named
> foo_shared_ptr. modified:
>   dolfin/adaptivity/refine.cpp
>   dolfin/adaptivity/refine.h
>   dolfin/fem/VariationalProblem.cpp
>   dolfin/fem/VariationalProblem.h
 
You are stretching the Python interface with your Hierarchical inheritance. :) 
Also all classes that inherit Hierarchical need to be stored as shared_ptr in 
the SWIG interface. This means that also VariationalProblem need to be 
shared_ptr stored.

It looks like we end up with storing everything using shared_ptr in SWIG, 
which I guess is cool, but then would foo_shared_ptr not make too much sense. 
Maybe just foo in the Python interface?

Johan

> --
> lp:dolfin
> https://code.launchpad.net/~dolfin-core/dolfin/main
> 
> Your team DOLFIN Core Team is subscribed to branch lp:dolfin.
> To unsubscribe from this branch go to
> https://code.launchpad.net/~dolfin-core/dolfin/main/+edit-subscription

Follow ups