New user self-registration is currently disabled. Please email eigen-core-team @ if you need an account.
Bug 451 - Warning with clang: missing parentheses in SparseLU.h
Warning with clang: missing parentheses in SparseLU.h
Product: Eigen
Classification: Unclassified
Component: Unsupported modules
All Linux
: Normal minor
Assigned To: Nobody
Depends on:
  Show dependency treegraph
Reported: 2012-04-22 19:36 UTC by romain.bossart
Modified: 2012-04-29 17:46 UTC (History)
1 user (show)


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!

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.