Re: [eigen] Some compilation warning

Hi Mathieu and all,

Great point, that bothers me, too. The convention in the standard library is to use size_t as indices, which means one has to keep in mind that looping over Eigen objects requires Eigen::DenseIndex.

Also, it would be great if rowwise() and colwise() provided begin() and end() members, in order to make Eigen usable with C++11 range-based for loops.

Thanks,

Marton

On 2 March 2015 at 11:42, Mathieu Dutour <mathieu.dutour@xxxxxxxxx> wrote:

Dear all,

why does the number of rows and columns of a matrix is stored as
"long" instead of "unsigned long" or maybe better yet "size_t" ?

When compiling an application using EIGEN with
clang++ -Weverything -std=c++11
I got many EIGEN related warnings.. For example:
SIGN_CONVERSION:
/home/mathieu/opt/eigen/Eigen/src/Core/util/Memory.h:835:26: warning: implicit conversion changes signedness: 'int' to
'unsigned int' [-Wsign-conversion]
int ways = (abcd[1] & 0xFFC00000) >> 22; // B[31:22]

DEPRECATED:
/home/mathieu/opt/eigen/Eigen/src/Core/PlainObjectBase.h:132:5: warning: dynamic exception specifications are deprecated [-Wdeprecated]
EIGEN_MAKE_ALIGNED_OPERATOR_NEW_IF(NeedsToAlign)

and some other warnings.
Is there an interest in correcting those?

Mathieu