Unexpected exception are no longer ignored by default

What an embarassing bug, I'll have to look into why it wasn't caught by any of Catch's self-tests. Fixes #885 and closes #887 (duplicate)
catchorg · Apr 9, 2017 · da8786b · da8786b
1 parent 5577322
commit da8786b
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/include/internal/catch_run_context.hpp b/include/internal/catch_run_context.hpp
@@ -301,11 +301,9 @@ namespace Catch {
             catch(...) {
                 // Under CATCH_CONFIG_FAST_COMPILE, unexpected exceptions under REQUIRE assertions 
                 // are reported without translation at the point of origin.
-#ifdef CATCH_CONFIG_FAST_COMPILE
                 if (m_shouldReportUnexpected) {
                     makeUnexpectedResultBuilder().useActiveException();
                 }
-#endif
             }
             m_testCaseTracker->close();
             handleUnfinishedSections();