← Back to team overview

dolfin team mailing list archive

Re: [Branch ~dolfin-core/dolfin/main] Rev 4702: Add castto T* in Array class

 

On Fri, Apr 30, 2010 at 12:46:14PM +0200, Johannes Ring wrote:
> On Fri, Apr 30, 2010 at 12:32 PM, Mehdi Nikbakht - CITG
> <M.Nikbakht@xxxxxxxxxx> wrote:
> > Hi,
> >
> > I don't think it is hardy related. I experienced the same in Lucid.
>
> Maybe it is architecture related then? Are you on 32 or 64 bit? Both
> our 64 bit linux slaves works fine [1]. It fails on my Ubuntu Hardy 32
> bit laptop.

I'll revert the change. The cast is not very important.

--
Anders


> [1] http://fenics.org:8080/dolfin

> >>     t += 0.2;
> >>   }
> >>
> >> === modified file 'dolfin/common/Array.h'
> >> --- dolfin/common/Array.h       2010-03-11 18:48:12 +0000
> >> +++ dolfin/common/Array.h       2010-04-29 15:23:41 +0000
> >> @@ -4,7 +4,7 @@
> >>  // Modified by Anders Logg, 2010.
> >>  //
> >>  // First added:  2009-12-06
> >> -// Last changed: 2010-03-11
> >> +// Last changed: 2010-04-29
> >>
> >>  #ifndef __DOLFIN_ARRAY_H
> >>  #define __DOLFIN_ARRAY_H
> >> @@ -123,6 +123,14 @@
> >>     boost::shared_array<T> data()
> >>     { return x; }
> >>
> >> +    /// Cast to array (const version)
> >> +    operator const T*() const
> >> +    { return x.get(); }
> >> +
> >> +    /// Cast to array (non-const version)
> >> +    operator T*()
> >> +    { return x.get(); }
> >> +
> >>   private:
> >>
> >>     // Length of array
> >>
> >>
> >>
> >
> > _______________________________________________
> > Mailing list: https://launchpad.net/~dolfin
> > Post to     : dolfin@xxxxxxxxxxxxxxxxxxx
> > Unsubscribe : https://launchpad.net/~dolfin
> > More help   : https://help.launchpad.net/ListHelp
> >
> >
>
> _______________________________________________
> Mailing list: https://launchpad.net/~dolfin
> Post to     : dolfin@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~dolfin
> More help   : https://help.launchpad.net/ListHelp

Attachment: signature.asc
Description: Digital signature


References