Skip to content

Commit dafe436

Browse files
committed
probably should leave asserts off unless debugging, for speed
1 parent 3c96bd1 commit dafe436

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/my_config.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
#ifndef _MY_CONFIG_HPP_
77
#define _MY_CONFIG_HPP_
88

9-
#define ENABLE_ASSERT // Abort on errors
9+
#undef ENABLE_ASSERT // Abort on errors
1010
#define ENABLE_CRASH_HANDLER // Intercept SEGV and other faults
1111
#define ENABLE_DEBUG_TRACE // (SLOW) Function tracing
1212
#define ENABLE_PTRCHECK_HISTORY 2 // Per pointer history (when in --debug2 mode)

src/my_sys.hpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -186,8 +186,8 @@ void CROAK_CLEAN(const char *fmt, ...) CHECK_FORMAT_STR(printf, 1, 2);
186186
#ifdef ENABLE_ASSERT
187187
#undef ASSERT
188188
#define ASSERT(x) \
189-
if (! (x)) { \
190-
TRACE_AND_INDENT(); \
189+
if (unlikely(! (x))) { \
190+
TRACE_NO_INDENT(); \
191191
std::stringstream ss; \
192192
ss << "Assert '" << #x << "' failed at line " << __LINE__ << ", file " << __FILE__ << ", function " \
193193
<< __FUNCTION__ << "()"; \
@@ -204,7 +204,7 @@ void CROAK_CLEAN(const char *fmt, ...) CHECK_FORMAT_STR(printf, 1, 2);
204204
// https://stackoverflow.com/questions/2193544/how-to-print-additional-information-when-assert-fails
205205
#ifdef ENABLE_ASSERT
206206
#define ASSERT_EX(left, operator, right) \
207-
if (! ((left) operator(right))) { \
207+
if (unlikely(! ((left) operator(right)))) { \
208208
TRACE_AND_INDENT(); \
209209
std::cerr << "ASSERT FAILED: " << #left << " " \
210210
<< # \

0 commit comments

Comments
 (0)