Re: [eigen] Some compilation warning |

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

*To*: eigen@xxxxxxxxxxxxxxxxxxx*Subject*: Re: [eigen] Some compilation warning*From*: Márton Danóczy <marton78@xxxxxxxxx>*Date*: Mon, 2 Mar 2015 14:16:18 +0100*Dkim-signature*: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :content-type; bh=ZmMKdGnUKAfxvcIahGqBnGdwbLFF2+IuI5V1XbpyVbs=; b=rxiBH8evEbxaPDv8EGOHRBrsk7ozdsyA+QJUMydzcZsO9kphPv0sAXk3eVWVdHs5Rd nSa3oHtsdxwqZ8I9uUhHGsGtyJOHuGkBelfwknxlv6abXefFinYXlXU5DXlZrz/Kfa1y GLObZ/SM0BT/QQQCgOPDwHbS/fmY8XKF/pquzUZxdvuW6pdOlUd5jxZeI0AlDFOq0jRf xnVJBr/pJiHgKcmWmVXtX3SFNYwvbPv3Hz2ZKq2dUeCTq8HMCbYYe5qQ6Dvj5B7N9FOw eUPRtUjqM8LPjkvAZfmpwu6ZiL4DVg97YO0EIZclf8NP2fHNgZWHk2kdhlihxl9qkcjq Ntcg==

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 withclang++ -Weverything -std=c++11I 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

**Follow-Ups**:**Re: [eigen] Some compilation warning***From:*Gael Guennebaud

**Re: [eigen] Some compilation warning***From:*Christoph Hertzberg

**References**:**[eigen] Some compilation warning***From:*Mathieu Dutour

**Messages sorted by:**[ date | thread ]- Prev by Date:
**[eigen] Some compilation warning** - Next by Date:
**Re: [eigen] Some compilation warning** - Previous by thread:
**[eigen] Some compilation warning** - Next by thread:
**Re: [eigen] Some compilation warning**

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