New user self-registration is disabled due to spam. Please email eigen-core-team @ lists.tuxfamily.org if you need an account.
Before reporting a bug, please make sure that your Eigen version is up-to-date!
Bug 451 - Warning with clang: missing parentheses in SparseLU.h
Summary: Warning with clang: missing parentheses in SparseLU.h
Status: RESOLVED FIXED
Alias: None
Product: Eigen
Classification: Unclassified
Component: Unsupported modules (show other bugs)
Version: 3.0
Hardware: All Linux
: Normal minor
Assignee: Nobody
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
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.