From b304b679e50be863f4f532ff176617f06056ef62 Mon Sep 17 00:00:00 2001
From: Yann Collet <cyan@fb.com>
Date: Tue, 9 Oct 2018 17:56:59 -0700
Subject: [PATCH] use #if BACKTRACE_ENABLE directly

as suggested by @terrelln
---
 programs/fileio.c | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/programs/fileio.c b/programs/fileio.c
index 1eb9a1e0a..7cbaab7da 100644
--- a/programs/fileio.c
+++ b/programs/fileio.c
@@ -166,14 +166,20 @@ static void clearHandler(void)
 #  define BACKTRACE_ENABLE 0
 #endif
 
-#if !defined(BACKTRACE_ENABLE)          \
-   && ((defined(__linux__) && defined(__GLIBC__)) \
-       || (defined(__APPLE__) && defined(__MACH__)) )
-#  define BACKTRACE_ENABLE 1
+#if !defined(BACKTRACE_ENABLE)
+/* automatic detector : backtrace enabled by default on linux+glibc and osx */
+#  if (defined(__linux__) && defined(__GLIBC__)) \
+     || (defined(__APPLE__) && defined(__MACH__))
+#    define BACKTRACE_ENABLE 1
+#  else
+#    define BACKTRACE_ENABLE 0
+#  endif
 #endif
 
+/* note : after this point, BACKTRACE_ENABLE is necessarily defined */
+
 
-#if defined(BACKTRACE_ENABLE) && (BACKTRACE_ENABLE>=1)
+#if BACKTRACE_ENABLE
 
 #include <execinfo.h>   /* backtrace, backtrace_symbols */
 
@@ -216,7 +222,7 @@ static void ABRThandler(int sig) {
 
 void FIO_addAbortHandler()
 {
-#if defined(BACKTRACE_ENABLE) && (BACKTRACE_ENABLE>=1)
+#if BACKTRACE_ENABLE
     signal(SIGABRT, ABRThandler);
     signal(SIGFPE, ABRThandler);
     signal(SIGILL, ABRThandler);
-- 
2.47.2