Commit 28554e63 authored by Gael Guennebaud's avatar Gael Guennebaud
Browse files

It's better to check that eigen_assert does raise an assert rather than...

It's better to check that eigen_assert does raise an assert rather than testing the definition of NDEBUG
Showing with 2 additions and 9 deletions
+2 -9
......@@ -13,15 +13,8 @@ template <typename MatrixType> void run_nesting_ops(const MatrixType& _m)
{
typename MatrixType::Nested m(_m);
#ifdef NDEBUG
const bool is_debug = false;
#else
const bool is_debug = true;
#endif
// Make really sure that we are in debug mode! We don't want any type of
// inlining for these tests to pass.
VERIFY(is_debug);
// Make really sure that we are in debug mode!
VERIFY_RAISES_ASSERT(eigen_assert(false));
// The only intention of these tests is to ensure that this code does
// not trigger any asserts or segmentation faults... more to come.
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment