dolfin team mailing list archive
-
dolfin team
-
Mailing list archive
-
Message #22229
Re: Typemap for std:vector<boost::shared_ptr<Foo> >
On Wednesday March 23 2011 09:59:02 Anders Logg wrote:
> On Wed, Mar 23, 2011 at 09:46:29AM -0700, Johan Hake wrote:
> > On Wednesday March 23 2011 09:45:13 Anders Logg wrote:
> > > On Wed, Mar 23, 2011 at 09:42:08AM -0700, Johan Hake wrote:
> > > > Hi!
> > > >
> > > > I got a request to implement a typemap for
> > > >
> > > > std:vector<boost::shared_ptr<Foo> >
> > > >
> > > > a while back.
> > > >
> > > > Should it be:
> > > > 1) std:vector<boost::shared_ptr<Foo> >&
> > > >
> > > > or
> > > >
> > > > 2) std:vector<boost::shared_ptr<Foo> >
> > > >
> > > > I guess copying of shared_ptr aren't that costly. But we need to be
> > > >
> > > > consistent:
> > > > Form
> > > >
> > > > use the first version and
> > > >
> > > > ErrorControl
> > > >
> > > > use the second version. I have no intention writing a typemap for
> > > > both ;)
> > >
> > > I think (1) even if the copying is not costly. So it needs to be
> > > changed in ErrorControl.
> >
> > No. It needs to be changed in Form.
>
> Why? Didn't you say that Form uses the first version?
I did! But it should be the other way around *blush*
Johan
> --
> Anders
>
> _______________________________________________
> Mailing list: https://launchpad.net/~dolfin
> Post to : dolfin@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~dolfin
> More help : https://help.launchpad.net/ListHelp
Follow ups
References