This forum has been archived. All content is frozen. Please use KDE Discuss instead.

Clang warning (trivial patch)

Tags: None
(comma "," separated)
duetosymmetry
Registered Member
Posts
18
Karma
0

Clang warning (trivial patch)

Sat Apr 14, 2012 6:14 pm
Clang warned me to fix the following. On an unrelated note, I see that SparseLDLT has been renamed SparseLDLTLegacy; what has superseded it?

Code: Select all
--- unsupported/Eigen/src/SparseExtra/SparseLU.h   2012-02-10 14:52:25.000000000 -0500
+++ unsupported/Eigen/src/SparseExtra/SparseLU.h   2012-04-14 14:05:16.000000000 -0400
@@ -104,7 +104,7 @@
     void setOrderingMethod(int m)
     {
       eigen_assert( (m&~OrderingMask) == 0 && m!=0 && "invalid ordering method");
-      m_flags = m_flags&~OrderingMask | m&OrderingMask;
+      m_flags = (m_flags&~OrderingMask) | (m&OrderingMask);
     }
 
     int orderingMethod() const
User avatar
ggael
Moderator
Posts
3447
Karma
19
OS

Re: Clang warning (trivial patch)

Sun Apr 15, 2012 6:03 am
SparseLU does not exists anymore. Check the updated sparse tutorial:

http://eigen.tuxfamily.org/dox-devel/Tu ... parse.html
duetosymmetry
Registered Member
Posts
18
Karma
0

Re: Clang warning (trivial patch)

Mon Apr 16, 2012 6:11 pm
I see. I am still using the stable (3.0.5) branch, not the latest from hg. Thanks for the response -- I will have to update my code for the next version to use SimplicialLDLt.


Bookmarks



Who is online

Registered users: Bing [Bot], claydoh, Google [Bot], rblackwell, Yahoo [Bot]