Re: [eigen] 3.2.3 for the end of the week.

[Billy Araujo <billyaraujo@xxxxxxxxx>]

Yes. This cast seems to resolve it. No more warnings.

Note: No need to remove the term before the ||

Regards,

Billy.

On Sat, Dec 13, 2014 at 4:55 PM, Christoph Hertzberg <chtz@xxxxxxxxxxxxxxxxxxxxxxxx> wrote:

On 13.12.2014 14:17, Billy Araujo wrote:

If you are taking out warnings there is also:

2>e:\libraries\eigen\eigen-3.2.3\eigen\src/Core/ProductBase.h(91): warning
C4308: negative integral constant converted to unsigned type
2>
  e:\libraries\eigen\eigen-3.2.3\eigen\src/SparseCore/SparseDenseProduct.h(260)
: see reference to class template instantiation
'Eigen::ProductBase<Derived,Lhs,Rhs>' being compiled

2>e:\libraries\eigen\eigen-3.2.3\eigen\src/Core/ProductBase.h(91): warning
C4307: '*' : integral constant overflow
2>e:\libraries\eigen\eigen-3.2.3\eigen\src/Core/ProductBase.h(91): warning
C4308: negative integral constant converted to unsigned type

Do these go away if you cast the result of sizeof to int, i.e.:

    typedef typename internal::conditional<(BasePlainObject::SizeAtCompileTime==Dynamic) || (BasePlainObject::SizeAtCompileTime*int(sizeof(Scalar)) < int(EIGEN_STACK_ALLOCATION_LIMIT)),
                                           BasePlainObject, DynPlainObject>::type PlainObject;

NB: We could even remove the term before the || then (which works as long as Eigen::Dynamic<0).

Christoph



--
----------------------------------------------
Dipl.-Inf., Dipl.-Math. Christoph Hertzberg
Cartesium 0.049
Universität Bremen
Enrique-Schmidt-Straße 5
28359 Bremen

Tel: +49 (421) 218-64252
----------------------------------------------