Bug 451 - Warning with clang: missing parentheses in SparseLU.h
: Warning with clang: missing parentheses in SparseLU.h
Status: RESOLVED FIXED
Product: Eigen
Classification: Unclassified
Component: Unsupported modules
: 3.0
: All Linux
: Normal minor
Assigned To: Nobody
:
:
:
:
:
  Show dependency treegraph
 
Reported: 2012-04-22 19:36 UTC by romain.bossart
Modified: 2012-04-29 17:46 UTC (History)
1 user (show)



Attachments

Description romain.bossart 2012-04-22 19:36:16 UTC
Hi, this kind of warning could easily be suppressed with clang's advice.

/usr/include/eigen3/unsupported/Eigen/src/SparseExtra/SparseLU.h:107:24: note:
place parentheses around the '&' expression to silence this warning
      m_flags = m_flags&~OrderingMask | m&OrderingMask;
                       ^
                (                    )


Thanks in advance,
Best best regards!

Romain
Comment 1 Jitse Niesen 2012-04-29 17:46:18 UTC
Fixed in changeset d7a07bd320f9 . I later realized that the SparseLU module is
removed in the upcoming 3.1 release, so perhaps not the most urgent bug fix.

Note You need to log in before you can comment on or make changes to this bug.