Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to C++11 or C++14 compile flag? #30

Open
stevenwaslander opened this issue Dec 23, 2015 · 3 comments
Open

Switch to C++11 or C++14 compile flag? #30

stevenwaslander opened this issue Dec 23, 2015 · 3 comments
Assignees

Comments

@stevenwaslander
Copy link

Only issue is to 3DMG-imu driver, which needed three constants changed to constexpr (as they are floats and doubles).

@stevenwaslander
Copy link
Author

Note, I can't seem to compile with C++0x anymore, had to switch my local version to C++11 with the changes to the imu driver.

@StanJBrown
Copy link
Member

What file is causing this? I have not been getting an error on my end.

On Wed, Dec 23, 2015 at 11:24 PM, Steven Waslander <[email protected]

wrote:

Assigned #30 #30 to @StanJBrown
https://github.com/StanJBrown.


Reply to this email directly or view it on GitHub
#30 (comment).

@stevenwaslander
Copy link
Author

I'm assuming something upgraded in terms of my compiler, as the imu package was throwing errors to do with const vs constexpr of constant doubles (which are not of type "integral"). I fixed this by changing const to constexpr in three places in the 3dmgx2.h and 3dmgx2.cc (for the const doubles G, KF_K_1, KF_K_2). This is only possible with a change to the CXX_FLAGS to C++11 from C++0x in CMakeLists.txt for both IMU and MCPTAM packages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants