KumarRobotics / imu_3dm_gx4

Driver for Lord Corporation Microstrain 3DM GX4 25
Other
32 stars 55 forks source link

use BOOST_VERIFY instead of ifdefs for advanceTo checks #15

Closed versatran01 closed 9 years ago

versatran01 commented 9 years ago

<boost/assert.hpp> also defines the macro BOOST_VERIFY. It has exactly the same behavior as BOOST_ASSERT, except that the expression that is passed to BOOST_VERIFY is always evaluated.

Boost 1.57 has a BOOST_VERIFY_MSG which we should use when Ubuntu catches up.

kartikmohta commented 9 years ago

Much better :-) On Aug 18, 2015 5:56 PM, "Chao Qu" notifications@github.com wrote:

<boost/assert.hpp> also defines the macro BOOST_VERIFY. It has exactly the same behavior as BOOST_ASSERT, except that the expression that is

passed to BOOST_VERIFY is always evaluated.

You can view, comment on, or merge this pull request online at:

https://github.com/KumarRobotics/imu_3dm_gx4/pull/15 Commit Summary

  • use BOOST_VERIFY instead of ifdefs for advanceTo checks

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/KumarRobotics/imu_3dm_gx4/pull/15.

versatran01 commented 9 years ago

Since I can't merge my own pr, it would be nice if someone else can test and verify this.