Skip to content

Commit 5151aa3

Browse files
Fix the build failure on release builds (#888)
* AssertMsg(cond, msg) is a macro that is not present on release builds leaving behind a lone ';' which is a warning
1 parent ea59231 commit 5151aa3

2 files changed

Lines changed: 4 additions & 0 deletions

File tree

Src/Generic/Test/testGeneric.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,10 @@ namespace unitpp
9292
if (fInjectTeardownAssert || fInjectTeardownAbort)
9393
RestorePreviousAssertProc();
9494

95+
#ifdef DEBUG
9596
if (fInjectTeardownAssert)
9697
AssertMsg(false, "Injected teardown assert for native test infrastructure validation");
98+
#endif
9799

98100
if (fInjectTeardownAbort)
99101
{

Src/views/Test/testViews.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,8 +113,10 @@ namespace unitpp
113113
if (fInjectTeardownAssert || fInjectTeardownAbort)
114114
RestorePreviousAssertProc();
115115

116+
#ifdef DEBUG
116117
if (fInjectTeardownAssert)
117118
AssertMsg(false, "Injected teardown assert for native test infrastructure validation");
119+
#endif
118120

119121
if (fInjectTeardownAbort)
120122
{

0 commit comments

Comments
 (0)