From da8786b8fdf382c27e2c96015aa6f8a47ef696ae Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Ho=C5=99e=C5=88ovsk=C3=BD?=
 <martin.horenovsky@gmail.com>
Date: Sun, 9 Apr 2017 21:20:58 +0200
Subject: [PATCH] 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)
---
 include/internal/catch_run_context.hpp | 2 --
 1 file changed, 2 deletions(-)

diff --git a/include/internal/catch_run_context.hpp b/include/internal/catch_run_context.hpp
index fa326a0f..e6dbd38c 100644
--- 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();