Re: [eigen] still the solve() API debate

[ Thread Index | Date Index | More lists.tuxfamily.org/eigen Archives ]


2009/9/13 Hauke Heibel <hauke.heibel@xxxxxxxxxxxxxx>:
> Ok, I implemented the EIGEN_REF_TO_TEMPORARY stuff....
>
> I am just compiling it with gcc 4.4.1 and -std=c++0x. VS.net does not
> yet support c++0x features. It will come with the next version VC10.
>
> I had to change
>
> void swap(MatrixBase<OtherDerived> EIGEN_REF_TO_TEMPORARY & other);
>
> to
>
> void swap(MatrixBase<OtherDerived> EIGEN_REF_TO_TEMPORARY other);
>
> because it seems to be important for rvalue references that there are
> no spaces between the '&'s.

Hm, again, I wonder if it would be cleaner to do it via a template struct,

void swap(typename ei_ref_to_temporary<MatrixBase<OtherDerived> >::type other);

>
> It would be great, if somebody finds the time to check the patch.

The first thing I noticed is EIGE_   (missing N)

Take your time, i'm completely busy at the moment :)

Benoit



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/